RFR: 8048184 : (s) handle mercurial dev build version string

[pointo1d]

Hi Daniel ,

On 08/07/14 11:30, Daniel Fuchs wrote:
> Hi Mike,
>
> sorry for not reacting sooner.
> I applied your patch on my machine, and now get the following:
>
> sh ./get_source.sh
> ./get_source.sh: line 70: [: too many arguments
> ./get_source.sh: line 76: [: `)' expected, found -1
> ./get_source.sh: line 82: [: `)' expected, found -1
>
>
> I have added:
>
> echo hgversion="$hgversion"
> echo hgmajor="$hgmajor"
> echo hgminor="$hgminor"
> echo hgrev="$hgrev"
>
> just before line 70 and I see this:
>
> $ sh ./get_source.sh
> hgversion=3.0.1
> hgmajor=3
> hgminor=0
> -1
> hgrev=
> ./get_source.sh: line 76: [: too many arguments
> ./get_source.sh: line 82: [: `)' expected, found -1
> ./get_source.sh: line 88: [: `)' expected, found -1
>
>
> There is a strange '-1' which gets printed - and it
> seems it comes from $hgminor, whose value is:
>
> hgminor='0
> -1'
>
>
> best regards,
>
> -- daniel
>
>
> <snip>

What do you see if you run the script using bash ?

-- 
Dave Pointon FIAP MBCS

Now I saw, tho' too late, the folly of beginning a work before we count the cost and before we we judge rightly of our strength to go thro' with it - Robinson Crusoe