setlocalversion: fix detection of hg revision for untagged versions
authorThomas De Schampheleire <thomas.de_schampheleire@nokia.com>
Wed, 26 Apr 2017 12:43:11 +0000 (14:43 +0200)
committerPeter Korsgaard <peter@korsgaard.com>
Mon, 3 Jul 2017 09:40:49 +0000 (11:40 +0200)
commitec019bcf643de83d502f30d85d56dd4547b0c11b
tree5791b14c13385c5c5ba4500ebd67c702f380dea8
parent3529ee59d1954da17b6b74b51d400317c774225a
setlocalversion: fix detection of hg revision for untagged versions

By default, cut prints the entire line if the specified delimiter is not
present at all:

    $ printf "foo bar" | cut -d' ' -f2
    bar
    $ printf "foobar" | cut -d' ' -f2
    foobar

In setlocalversion, cut is presented with the output of 'hg id' which has
the format:

    "<revision> <tags-if-any>"

If the current revision is not tagged, the output of 'hg id' does not
contain the delimiter (space), cut prints the entire string, and
setlocalversion thinks the version is the tag.
As setlocalversion does not print anything for tagged versions, there is no
output overall, and no correct indication of the mercurial revision.

Fix by passing the extra cut option '--only-delimited', which suppresses
output if no delimiter is found.

This problem likely went unnoticed for so long, because the tag 'tip' (i.e.
most recent revision of the branch) is treated specially: in this case the
mercurial revision _is_ printed, i.e. the situation is treated as
'untagged'.
The problem is only seen when you are _not_ at the most recent revision in
your branch.

Signed-off-by: Thomas De Schampheleire <thomas.de_schampheleire@nokia.com>
Signed-off-by: Peter Korsgaard <peter@korsgaard.com>
support/scripts/setlocalversion