Every time I submit a patch to gerrit using the git review command, I get two warning messages that I ignore. I just want to make sure that I can continue to safely ignore them (or else, how to get rid of them).
The first one says: Using global/system git-review config files (/etc/git-review/git-review.conf) is deprecated
The second one says: remote: Pushing to refs/publish/* is deprecated, use refs/for/* instead.
Thanks, Huji
"remote: Pushing to refs/publish/* is deprecated, use refs/for/*" is a new one in gerrit 2.15+. they kept backwards support for refs/publish since they knew git-review was using it. On Monday, 11 June 2018, 23:50:28 BST, Huji Lee huji.huji@gmail.com wrote:
Every time I submit a patch to gerrit using the git review command, I get two warning messages that I ignore. I just want to make sure that I can continue to safely ignore them (or else, how to get rid of them).
The first one says: Using global/system git-review config files (/etc/git-review/git-review.conf) is deprecated
The second one says: remote: Pushing to refs/publish/* is deprecated, use refs/for/* instead.
Thanks, Huji _______________________________________________ MediaWiki-l mailing list To unsubscribe, go to: https://lists.wikimedia.org/mailman/listinfo/mediawiki-l
Sending a copy to wikitech-l, because this is a little bit more generic than "just" MediaWiki.
Hello,
the first one means you have something in /etc/git-review/git-review.conf, which is probably unneeded. I suggest you to delete that file. On my system, it doesn't exist.
The second one is caused by Gerrit update. Upstream kept refs/publish/* working, because they know git-review is using that ref. I think that as soon as git-review developers fixes this and you will upgrade, the warning will disappear.
Best, Martin
Ășt 12. 6. 2018 v 0:51 odesĂlatel Huji Lee huji.huji@gmail.com napsal:
Every time I submit a patch to gerrit using the git review command, I get two warning messages that I ignore. I just want to make sure that I can continue to safely ignore them (or else, how to get rid of them).
The first one says: Using global/system git-review config files (/etc/git-review/git-review.conf) is deprecated
The second one says: remote: Pushing to refs/publish/* is deprecated, use refs/for/* instead.
Thanks, Huji _______________________________________________ MediaWiki-l mailing list To unsubscribe, go to: https://lists.wikimedia.org/mailman/listinfo/mediawiki-l
On Tue, Jun 12, 2018 at 5:03 AM Martin Urbanec martin.urbanec@wikimedia.cz wrote:
The second one is caused by Gerrit update. Upstream kept refs/publish/* working, because they know git-review is using that ref. I think that as soon as git-review developers fixes this and you will upgrade, the warning will disappear.
Yeah, I guess refs/publish/* is being deprecated.
Which is pretty funny, considering refs/for/* was going to be deprecated in favor of the newer refs/publish/*. Later, that was deemed to be a bad idea and so the new one was deprecated and the original retained.
Any scripts that use refs/publish/* should be updated. I assume git-review will do this in short order.
-Chad
mediawiki-l@lists.wikimedia.org