[PATCH dim-tools] dim: Avoid hardcoding remote name for linux-upstream

[Rodrigo Vivi]

On Wed, Jul 09, 2025 at 08:21:05AM +0200, Lukas Wunner wrote:
> On Tue, Jul 08, 2025 at 05:19:52PM -0400, Rodrigo Vivi wrote:
> > On Sat, Jul 05, 2025 at 10:51:36AM +0200, Lukas Wunner wrote:
> > > dim hardcodes "origin" as remote name for linux-upstream in a number of
> > > places.  This (only) works if "origin" is indeed Torvalds' repo.
> > > 
> > > Resolve the remote name for linux-upstream at runtime to make dim work
> > > with local repos that lack an "origin" remote or use it for something
> > > else.
> > 
> > I'm afraid this doesn't 'resolve the name' but instead force everyone
> > else to use linux-upstream as the name for Linus branch.
> 
> No, "linux-upstream" is the symbolic git repo name used in nightly.conf
> (in the drm-rerere repo).

oh! I'm sorry, I had missed that indeed.

But perhaps change that to be origin?! :)

> 
> > rdvivi at rdvivi-mobl1:~/linux/maintainer-tools$ dim blah
> > dim: Adding remote for linux-upstream repo from URLs: 
> > git://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
> > https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
> > https://kernel.googlesource.com/pub/scm/linux/kernel/git/torvalds/linux.git
> 
> Hm, are you not using one of those URLs for your "origin" remote?

yes, I am, but the thing is that the first time it tries to access the linux-upstream,
it doesn't exist. nothing to do with the origin.

> What does "git remote -v | grep origin" return?

rdvivi at rdvivi-mobl1:~/linux/drm-tip$ git remote -v | grep origin
origin	git://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git (fetch)
origin	git://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git (push)

> 
> > > Somebody please consider applying this.  It looks like I cannot push to
> > > the maintainer-tools repo despite being allowed to push to drm-misc.
> > > Or I'm doing something wrong.  Thanks!
> > 
> > Please take a look at:
> > https://gitlab.freedesktop.org/drm/maintainer-tools
> > 
> > We are now sending merge requests there instead of patches in this mailing list.
> 
> Thanks!  I didn't get that memo.
> 
> Lukas