# Adding a line after the signed-off git am -s

4 messages from 2019-10-11 to 2019-10-16. Participants: Daniel Lezcano, Junio C Hamano, Beat Bolli.
Thread: https://gitlist.dev/t/52021

## Daniel Lezcano, 2019-10-11 14:43

Subject: Adding a line after the signed-off git am -s
Message-ID: <6797ef7d-5444-4c77-59e6-1d78fd3ccb0c@linaro.org>
URL: https://gitlist.dev/e/6797ef7d-5444-4c77-59e6-1d78fd3ccb0c%40linaro.org

```

Hi all,

Is there a way to specify a line to be added in the change-log after the
SOB with git-am ?

I would like to do something:

git am -s -l "Link: https://lore.kernel.org/r/<msgid>"

Which will give:

blabla

Signed-off-by: author@kairnail.org
Signed-off-by: commiter@kairnail.org
Link: https://lore.kernel.org/r/<msgid>

This way it is compatible with patchwork, git-pw, etc...

Thanks in advance

-- 
 <http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs

Follow Linaro:  <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog


```

## Junio C Hamano, 2019-10-12 01:03

Subject: Re: Adding a line after the signed-off git am -s
Message-ID: <xmqq7e5ag4g5.fsf@gitster-ct.c.googlers.com>
URL: https://gitlist.dev/e/xmqq7e5ag4g5.fsf%40gitster-ct.c.googlers.com
In-Reply-To: <6797ef7d-5444-4c77-59e6-1d78fd3ccb0c@linaro.org>

```
Daniel Lezcano <daniel.lezcano@linaro.org> writes:

> I would like to do something:
>
> git am -s -l "Link: https://lore.kernel.org/r/<msgid>"
>
> Which will give:
>
> blabla
>
> Signed-off-by: author@kairnail.org
> Signed-off-by: commiter@kairnail.org
> Link: https://lore.kernel.org/r/<msgid>
>
> This way it is compatible with patchwork, git-pw, etc...

There is the post-applypatch hook you can define after the patch
gets applied and produces a commit.

I use it to maintain the amlog notes in my repository (iow, I do not
amend the commit, but add notes to the resulting commit so that I
can tell, given a commit, which message resulted in it).  

If you want to amend the resulting commit instead, the place to do
so would be where I call "git notes --ref amlog" in the sample
script.

-- >8 -- post-applypatch hook example -- >8 --
#!/bin/sh

GIT_DIR=.git
dotest="$GIT_DIR/rebase-apply"

prec=4 &&
this=$(cat 2>/dev/null "$dotest/next") &&
msgnum=$(printf "%0${prec}d" $this) &&
test -f "$dotest/$msgnum" &&
message_id=$(sed -ne '
	/^[ 	]/{
		# Append continuation line to hold space
		H
		# Swap hold and pattern
		x
		# Remove the LF, making it a single line
		s/\n//
		# Swap hold and pattern back
		x
		# Discard the pattern and go on
		n
	}
	# Hold this new line, and look at what is in the hold space
	x
	# Is it the Message-ID line?  If so, spit it out and finish.
	/^[Mm][Ee][Ss][Ss][Aa][Gg][Ee]-[Ii][Dd]:[ 	]*/{
		s///p
		q
	}
	# Otherwise, check if this new line is empty
	x
	# Is it?  Then we are done with the header
	/^$/b end
	# Otherwise we need to hold onto this header line
	x
	# And start the next cycle
	b
: end
	# ??? do we want to check if we held onto the last message-id line
	# and process it here if we did???
	q
' "$dotest/$msgnum") &&

if	test -n "$message_id" &&
	head=$(git rev-parse --verify HEAD 2>/dev/null)
then
	echo "$head $message_id" >>"$GIT_DIR"/am.log &&
	git notes --ref amlog add -f -m "Message-Id: $message_id" "$head"
fi

```

## Beat Bolli, 2019-10-15 22:52

Subject: Re: Adding a line after the signed-off git am -s
Message-ID: <8089dfce-0183-b7fb-cab8-0a71404b6584@drbeat.li>
URL: https://gitlist.dev/e/8089dfce-0183-b7fb-cab8-0a71404b6584%40drbeat.li
In-Reply-To: <6797ef7d-5444-4c77-59e6-1d78fd3ccb0c@linaro.org>

```
On 11.10.19 16:43, Daniel Lezcano wrote:
> 
> Hi all,
> 
> Is there a way to specify a line to be added in the change-log after the
> SOB with git-am ?
> 
> I would like to do something:
> 
> git am -s -l "Link: https://lore.kernel.org/r/<msgid>"
> 
> Which will give:
> 
> blabla
> 
> Signed-off-by: author@kairnail.org
> Signed-off-by: commiter@kairnail.org
> Link: https://lore.kernel.org/r/<msgid>
> 
> This way it is compatible with patchwork, git-pw, etc...

I think something like

    git interpret-trailer --trailer Link:https://lore.kernel.irg/r/msgid
<patch | git am -

should work.


Cheers,
Beat

```

## Daniel Lezcano, 2019-10-16 08:48

Subject: Re: Adding a line after the signed-off git am -s
Message-ID: <c82cf831-a033-1ad9-139b-4cbc1eb3fb1d@linaro.org>
URL: https://gitlist.dev/e/c82cf831-a033-1ad9-139b-4cbc1eb3fb1d%40linaro.org
In-Reply-To: <8089dfce-0183-b7fb-cab8-0a71404b6584@drbeat.li>

```
On 16/10/2019 00:52, Beat Bolli wrote:
> On 11.10.19 16:43, Daniel Lezcano wrote:
>>
>> Hi all,
>>
>> Is there a way to specify a line to be added in the change-log after the
>> SOB with git-am ?
>>
>> I would like to do something:
>>
>> git am -s -l "Link: https://lore.kernel.org/r/<msgid>"
>>
>> Which will give:
>>
>> blabla
>>
>> Signed-off-by: author@kairnail.org
>> Signed-off-by: commiter@kairnail.org
>> Link: https://lore.kernel.org/r/<msgid>
>>
>> This way it is compatible with patchwork, git-pw, etc...
> 
> I think something like
> 
>     git interpret-trailer --trailer Link:https://lore.kernel.irg/r/msgid
> <patch | git am -
> 
> should work.

Thank you for the suggestion, I will try.

  -- Daniel


-- 
 <http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs

Follow Linaro:  <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog


```