Re: [LEDE-DEV] getver.sh regression

2016-07-13 Thread Jonas Gorski
On 13 July 2016 at 11:52, Álvaro Fernández Rojas wrote: > That one is working fine after a small fix: > https://gist.github.com/Noltari/0c395c57b94598bdf96e6a292c7b5888 > (You're missing one "|"). That's what I get for manually fixing up an older WIP patch ;p. I added another

Re: [LEDE-DEV] getver.sh regression

2016-07-13 Thread Álvaro Fernández Rojas
That one is working fine after a small fix: https://gist.github.com/Noltari/0c395c57b94598bdf96e6a292c7b5888 (You're missing one "|"). El 13/7/16 a las 11:45, Jonas Gorski escribió: > On 13 July 2016 at 11:16, Álvaro Fernández Rojas wrote: >> Appart from that, if there's no

Re: [LEDE-DEV] getver.sh regression

2016-07-13 Thread Jonas Gorski
On 13 July 2016 at 11:13, John Crispin wrote: > > > On 13/07/2016 11:11, Jonas Gorski wrote: >> Hi John, >> >> On 13 July 2016 at 09:39, John Crispin wrote: >>> Hi Jonas, >>> Hi Felix, >>> >>> your recent changes to getver.sh are causing a regression when not

Re: [LEDE-DEV] getver.sh regression

2016-07-13 Thread John Crispin
On 13/07/2016 11:11, Jonas Gorski wrote: > Hi John, > > On 13 July 2016 at 09:39, John Crispin wrote: >> Hi Jonas, >> Hi Felix, >> >> your recent changes to getver.sh are causing a regression when not >> building inside trunk but inside a staging tree >> >>

Re: [LEDE-DEV] getver.sh regression

2016-07-13 Thread Jonas Gorski
Hi John, On 13 July 2016 at 09:39, John Crispin wrote: > Hi Jonas, > Hi Felix, > > your recent changes to getver.sh are causing a regression when not > building inside trunk but inside a staging tree > > blogic@debian:/lede/staging$ ./scripts/getver.sh > warning: refname