On Wed, Jun 29, 2016 at 05:59:47PM -0700, Stefan Beller wrote: > Allow a user to pass information along a push to the pre/post-receive hook > on the remote. Seems like a reasonable thing to want, and related to: http://thread.gmane.org/gmane.comp.version-control.git/285124 which was specifically interested in making "--quiet" and "--force" available, but stopped short of arbitrary data. (It looks like that topic got overlooked, and Dennis never prodded again). > When using a remote that is more than just a plain Git host (e.g. Gerrit, > Git{hub/lab}, etc) this may become more obvious: The (server backend specific) > push options can instruct the server to: > * open a pull request I know this is just a "for example", but I don't know if we would ever support opening a pull request directly via push, if only because a good pull request has actual parameters that the user needs to fill out (like the cover letter message). So you'd probably want some client tool to help the user figure out what to put in the PR, and of course that already exists, because GitHub has an HTTP API that it can talk to. Shoe-horning it into Git doesn't buy much. I'm not sure if your other examples would be better served by just using an HTTP API or not. I think Gerrit is funny in this regard because it eschews branches entirely. E.g., in a GitHub PR, you push to branch "foo", and then you open a PR using "foo" as the source. But with Gerrit, you push to the magic refs/for/master, and you have no real way to cross-reference that submission later. Whereas in Dennis's patches, it was about specific information directly related to the act of pushing. > * When pushing to Gerrit you can already attach some of this information by > adding a '%' followed by the parameter to the ref, i.e. when interacting with > Gerrit it is possible to do things like[1]: > > git push origin HEAD:refs/for/master%draft%topic=example%cc=jon.doe@xxxxxxxxxxx > > This is not appealing to our users as it looks like hacks upon hacks to make > it work. It would read better if it was spelled as: Heh. It _is_ hacks upon hacks, isn't it? ;) -Peff -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html