diff mbox

[MONARCH_LTS,1/2] scripts/git_hash.sh: to support monarch_lts tags

Message ID 1470213659-4120-2-git-send-email-maxim.uvarov@linaro.org
State Superseded
Headers show

Commit Message

Maxim Uvarov Aug. 3, 2016, 8:40 a.m. UTC
Signed-off-by: Maxim Uvarov <maxim.uvarov@linaro.org>

---
 scripts/git_hash.sh | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

-- 
2.7.1.250.gff4ea60

Comments

Anders Roxell Aug. 5, 2016, 6:33 p.m. UTC | #1
On 2016-08-03 11:40, Maxim Uvarov wrote:
> Signed-off-by: Maxim Uvarov <maxim.uvarov@linaro.org>

> ---

>  scripts/git_hash.sh | 2 +-

>  1 file changed, 1 insertion(+), 1 deletion(-)

> 

> diff --git a/scripts/git_hash.sh b/scripts/git_hash.sh

> index 336eb01..83123f4 100755

> --- a/scripts/git_hash.sh

> +++ b/scripts/git_hash.sh

> @@ -7,7 +7,7 @@ fi

>  ROOTDIR=${1}

>  

>  if [ -d ${ROOTDIR}/.git ]; then

> -	hash=$(git --git-dir=${ROOTDIR}/.git describe --match 'v[0-9]*\.[0-9]*\.[0-9]*\.[0-9]*'\

> +	hash=$(git --git-dir=${ROOTDIR}/.git describe --match 'v[0-9]*\.[0-9]*\.[0-9]*\.[0-9]*-monarch_lts'\


If we change the tag from v1.11.0.0-monarch_lts to v1.11.0.0_monarch_lts
we don't have to add release specific patches to specify the name.

Cheers,
Anders
Maxim Uvarov Aug. 8, 2016, 7:50 a.m. UTC | #2
On 08/05/16 21:33, Anders Roxell wrote:
> On 2016-08-03 11:40, Maxim Uvarov wrote:

>> Signed-off-by: Maxim Uvarov <maxim.uvarov@linaro.org>

>> ---

>>   scripts/git_hash.sh | 2 +-

>>   1 file changed, 1 insertion(+), 1 deletion(-)

>>

>> diff --git a/scripts/git_hash.sh b/scripts/git_hash.sh

>> index 336eb01..83123f4 100755

>> --- a/scripts/git_hash.sh

>> +++ b/scripts/git_hash.sh

>> @@ -7,7 +7,7 @@ fi

>>   ROOTDIR=${1}

>>   

>>   if [ -d ${ROOTDIR}/.git ]; then

>> -	hash=$(git --git-dir=${ROOTDIR}/.git describe --match 'v[0-9]*\.[0-9]*\.[0-9]*\.[0-9]*'\

>> +	hash=$(git --git-dir=${ROOTDIR}/.git describe --match 'v[0-9]*\.[0-9]*\.[0-9]*\.[0-9]*-monarch_lts'\

> If we change the tag from v1.11.0.0-monarch_lts to v1.11.0.0_monarch_lts

> we don't have to add release specific patches to specify the name.

>

> Cheers,

> Anders

Tags in master branch are:
v1.12.0
..
v1.13.0
...
v1.14.0


tags in monarch_lts branch are:
v1.11.0.0-monarch_lts
v1.11.1.0-monarch_lts
v1.11.2.0-monarch_lts
v1.11.3.0-monarch_lts

We need this change to not overlap with master branch.

Maxim.
Anders Roxell Aug. 9, 2016, 6:48 p.m. UTC | #3
On 8 August 2016 at 09:50, Maxim Uvarov <maxim.uvarov@linaro.org> wrote:
> On 08/05/16 21:33, Anders Roxell wrote:

>>

>> On 2016-08-03 11:40, Maxim Uvarov wrote:

>>>

>>> Signed-off-by: Maxim Uvarov <maxim.uvarov@linaro.org>

>>> ---

>>>   scripts/git_hash.sh | 2 +-

>>>   1 file changed, 1 insertion(+), 1 deletion(-)

>>>

>>> diff --git a/scripts/git_hash.sh b/scripts/git_hash.sh

>>> index 336eb01..83123f4 100755

>>> --- a/scripts/git_hash.sh

>>> +++ b/scripts/git_hash.sh

>>> @@ -7,7 +7,7 @@ fi

>>>   ROOTDIR=${1}

>>>     if [ -d ${ROOTDIR}/.git ]; then

>>> -       hash=$(git --git-dir=${ROOTDIR}/.git describe --match

>>> 'v[0-9]*\.[0-9]*\.[0-9]*\.[0-9]*'\

>>> +       hash=$(git --git-dir=${ROOTDIR}/.git describe --match

>>> 'v[0-9]*\.[0-9]*\.[0-9]*\.[0-9]*-monarch_lts'\

>>

>> If we change the tag from v1.11.0.0-monarch_lts to v1.11.0.0_monarch_lts

>> we don't have to add release specific patches to specify the name.

>>

>> Cheers,

>> Anders

>

> Tags in master branch are:

> v1.12.0

> ..

> v1.13.0

> ...

> v1.14.0


Yes the tags in the master branch will be v<digit 1>.<digit 2>.<digit
3>.<digit 4>
v1.12.0.0
..
v1.13.0.0
...
v1.14.0.0

This is what you've done from verstion v1.1.0.0, so I'm not sure why you want to
change it to only three digits going forward?

>

>

> tags in monarch_lts branch are:

> v1.11.0.0-monarch_lts

> v1.11.1.0-monarch_lts

> v1.11.2.0-monarch_lts

> v1.11.3.0-monarch_lts


What I'm suggesting are if we change  the tags in the monarch_lts branch to:
v1.11.0.0_monarch_lts
v1.11.1.0_monarch_lts

>

> We need this change to not overlap with master branch.


and we do not need to change as long as we use an underscore
"_monarch_lts" after the four digits.

Cheers,
Anders
diff mbox

Patch

diff --git a/scripts/git_hash.sh b/scripts/git_hash.sh
index 336eb01..83123f4 100755
--- a/scripts/git_hash.sh
+++ b/scripts/git_hash.sh
@@ -7,7 +7,7 @@  fi
 ROOTDIR=${1}
 
 if [ -d ${ROOTDIR}/.git ]; then
-	hash=$(git --git-dir=${ROOTDIR}/.git describe --match 'v[0-9]*\.[0-9]*\.[0-9]*\.[0-9]*'\
+	hash=$(git --git-dir=${ROOTDIR}/.git describe --match 'v[0-9]*\.[0-9]*\.[0-9]*\.[0-9]*-monarch_lts'\
 	       | tr -d "\n")
 	if [[ $(git --git-dir=${ROOTDIR}/.git diff --shortstat 2> /dev/null \
 		| tail -n1) != "" ]]; then