Feature request: Can a parameter be given to "stg goto" so that it can go to a commit ID rather than a patch name

[dhowells]

Can "stg goto" be given a new parameter that allows you to specify the patch to go to by commit ID rather than by patch name?E.g.:

stg goto -c 1e08868e23502e59205b6208483e967b3e6c3f1a

Alternatively/additionally, it would be handy to have a way to look up the name of a patch by commit ID, maybe:

stg id -n 1e08868e23502e59205b6208483e967b3e6c3f1a

giving the name of the patch rather than its ID.

[jpgrayson]

These features seems possible to implement. We would have to linearly scan the stack's patches' commits to find id matches, which wouldn't be too costly.

My question is: why? I'm interested in the context or application where these features would be useful. At the moment, I'm unconvinced that these would be worth the implementation effort and maintenance overhead, but I'm willing to be convinced otherwise.

[dhowells]

Because git blame gives you a commit ID, and there's no direct way to turn the commit ID into an stgit patch name.

[jpgrayson]

This use case makes sense to me. Thanks for clarifying.

Add the features you suggest to goto and/or id would be okay with me. I will work on it eventually if nobody submits a PR.

[jpgrayson]

This feature was added in 1069060. Note that the patch's hash may be specified as an argument on the command line in place of a patch name; i.e. not as a command line option (no -c option).