# Bug? url.insteadOf overwrites remote.pushUrl 3 messages from 2011-10-14 to 2011-10-14. Participants: Kirill Likhodedov, Michael J Gruber. Thread: https://gitlist.dev/t/28678 ## Kirill Likhodedov, 2011-10-14 13:55 Subject: Bug? url.insteadOf overwrites remote.pushUrl Message-ID: URL: https://gitlist.dev/e/CAB6D58F-A3C9-4532-A9CC-10E43CD34E4E%40jetbrains.com ``` I've found that defining url..insteadOf overrides explicit remote..pushUrl. On the other hand, pushInsteadOf doesn't override explicit pushUrl. Is it a bug? # cat .git/config [remote "origin"] fetch = +refs/heads/*:refs/remotes/origin/* url = github.com/klikh/Test.git pushUrl = jetbrains.com/klikh/Test.git [url "http://"] insteadOf=jet # git remote -v origin github.com/klikh/Test.git (fetch) origin http://brains.com/klikh/Test.git (push) See also http://kerneltrap.org/mailarchive/git/2009/9/7/11264/thread - patch introducing pushInsteadOf & discussion ---------------------------------- Kirill Likhodedov JetBrains, Inc http://www.jetbrains.com "Develop with pleasure!" ``` ## Michael J Gruber, 2011-10-14 15:35 Subject: Re: Bug? url.insteadOf overwrites remote.pushUrl Message-ID: <4E9856BD.3010807@drmicha.warpmail.net> URL: https://gitlist.dev/e/4E9856BD.3010807%40drmicha.warpmail.net In-Reply-To: ``` Kirill Likhodedov venit, vidit, dixit 14.10.2011 15:55: > > I've found that defining url..insteadOf overrides explicit > remote..pushUrl. It doesn't really override it. It is applied to it, i.e. transforms it. > On the other hand, pushInsteadOf doesn't > override explicit pushUrl. Is it a bug? That is as described in the thread (thanks for linking to it). > # cat .git/config [remote "origin"] fetch = > +refs/heads/*:refs/remotes/origin/* url = github.com/klikh/Test.git > pushUrl = jetbrains.com/klikh/Test.git [url "http://"] insteadOf=jet > > # git remote -v origin github.com/klikh/Test.git (fetch) origin > http://brains.com/klikh/Test.git (push) The idea of "pushInsteadOf" was that, instead of having to define url and pushurl separately, you can use different rules, say github -> git://github.com (fetch) github -> https://username@github.com (push) github/ -> git@github.com: (push) used with "url = github/otheruser/repo.git" alone, without pushurl. pushurl predates pushinsteadof, and when the latter was introduced, one could have argued for or against "insteadof" being applied to pushurls. But that was necessary before, and existing behavior at the time when pushinsteadof was introduced. So, I don't see a bug, nor anything we could change now, though arguably most people use either pushinstead of or pushurl, but not both. Cheers, Michael ``` ## Kirill Likhodedov, 2011-10-14 19:16 Subject: Re: Bug? url.insteadOf overwrites remote.pushUrl Message-ID: <0656F8BA-3B41-473D-88A6-6A703C9A9CED@jetbrains.com> URL: https://gitlist.dev/e/0656F8BA-3B41-473D-88A6-6A703C9A9CED%40jetbrains.com In-Reply-To: <4E9856BD.3010807@drmicha.warpmail.net> ``` >> On the other hand, pushInsteadOf doesn't >> override explicit pushUrl. Is it a bug? > > That is as described in the thread (thanks for linking to it). > pushurl predates pushinsteadof, and when the latter was introduced, one > could have argued for or against "insteadof" being applied to pushurls. > But that was necessary before, and existing behavior at the time when > pushinsteadof was introduced. So, I don't see a bug, nor anything we > could change now, though arguably most people use either pushinstead of > or pushurl, but not both. Yeah, that's clear now. As I already answered to Junio in another thread, my question was only about insteadOf, not pushInsteadOf behavior. The behavior of pushInsteadOf is completely clear from the thread and from t5516-fetch-push.sh Sorry for making it not clear enough. Anyway, thanks for giving a one more explanation and example on how url, pushUrl and insteadOf configs work, it is always useful to go though it once again. ```