[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PULL v2 22/25] linux-user: Implement signals for openr
From: |
Alex Bennée |
Subject: |
Re: [Qemu-devel] [PULL v2 22/25] linux-user: Implement signals for openrisc |
Date: |
Sat, 07 Jul 2018 08:04:13 +0100 |
User-agent: |
mu4e 1.1.0; emacs 26.1.50 |
Stafford Horne <address@hidden> writes:
> Hello,
>
> On Wed, Jul 4, 2018, 7:34 AM Philippe Mathieu-Daudé <address@hidden> wrote:
>
>> Hi Stafford,
>>
>> On 07/02/2018 12:10 PM, Stafford Horne wrote:
>> > From: Richard Henderson <address@hidden>
<snip>
>
> Sorry, I brought these in from Richards tree and saw them but thought it
> was fine to keep the annotations.
>
> I'll remove them from patches in the future.
>
> Richard, how do you usually remove these before your pr? I suppose you
> have some kind of script?
>
For reference these are my exact-steps:
We need to ensure we have added our signoff and there is no ---
ephemera left from commit history.
,----
| errors=0
| commits=0
| while read rev; do
| author=$(git show -s --format='%an <%ae>' $rev)
| body=$(git show -s --format='%B' $rev)
|
| # Check for Author signoff
| if ! grep -q "^Signed-off-by: $author" <<< "$body"; then
| errors=$((errors+1))
| echo "$rev missing signoff by $author"
| fi
|
| # Check for my signoff
| if ! grep -q "^Signed-off-by: $signoff" <<< "$body"; then
| errors=$((errors+1))
| echo "$rev missing your signoff ($signoff)"
| fi
|
| # Check for stray history
| if grep -q "^--" <<< "$body"; then
| errors=$((errors+1))
| echo "$rev may have history in it"
| fi
|
| commits=$((commits+1))
| done < <(git rev-list "origin/master..HEAD")
|
| echo "Found $errors errors over $commits commits"
`----
If we want to strip history we can run the following:
,----
| current=$(git rev-parse --abbrev-ref HEAD)
| if [ -d "${series}.pull" ]; then
| rm -rf ${series}.pull
| fi
| git format-patch origin/master..${current} -p -o ${series}.pull
| prb=${current}-pr
| echo ${prb}
| git checkout -b ${prb} origin/master
| git am ${series}.pull/*
| rm -rf ${series}.pull
`----
--
Alex Bennée
- [Qemu-devel] [PULL v2 19/25] target/openrisc: Increase the TLB size, (continued)
- [Qemu-devel] [PULL v2 19/25] target/openrisc: Increase the TLB size, Stafford Horne, 2018/07/02
- [Qemu-devel] [PULL v2 20/25] target/openrisc: Reorg tlb lookup, Stafford Horne, 2018/07/02
- [Qemu-devel] [PULL v2 21/25] target/openrisc: Add support in scripts/qemu-binfmt-conf.sh, Stafford Horne, 2018/07/02
- [Qemu-devel] [PULL v2 22/25] linux-user: Implement signals for openrisc, Stafford Horne, 2018/07/02
- Re: [Qemu-devel] [PULL v2 22/25] linux-user: Implement signals for openrisc, Philippe Mathieu-Daudé, 2018/07/03
- Re: [Qemu-devel] [PULL v2 22/25] linux-user: Implement signals for openrisc, Stafford Horne, 2018/07/03
- Re: [Qemu-devel] [PULL v2 22/25] linux-user: Implement signals for openrisc, Richard Henderson, 2018/07/04
- Re: [Qemu-devel] [PULL v2 22/25] linux-user: Implement signals for openrisc, Eric Blake, 2018/07/06
- Re: [Qemu-devel] [PULL v2 22/25] linux-user: Implement signals for openrisc, Stafford Horne, 2018/07/06
- Re: [Qemu-devel] [PULL v2 22/25] linux-user: Implement signals for openrisc, Laurent Vivier, 2018/07/07
- Re: [Qemu-devel] [PULL v2 22/25] linux-user: Implement signals for openrisc,
Alex Bennée <=
[Qemu-devel] [PULL v2 25/25] target/openrisc: Fix writes to interrupt mask register, Stafford Horne, 2018/07/02
[Qemu-devel] [PULL v2 24/25] target/openrisc: Fix delay slot exception flag to match spec, Stafford Horne, 2018/07/02
[Qemu-devel] [PULL v2 23/25] linux-user: Fix struct sigaltstack for openrisc, Stafford Horne, 2018/07/02
Re: [Qemu-devel] [PULL v2 00/25] OpenRISC updates for 3.0, Peter Maydell, 2018/07/02