Message ID | 1407314534-5536-1-git-send-email-alex.bennee@linaro.org |
---|---|
State | Accepted |
Headers | show |
Dear Alex Bennée, On Wed, 6 Aug 2014 09:42:14 +0100, Alex Bennée wrote: > gdb has had AArch64 support since 7.6 which is now the default minimum > version of GDB in the tree. Older versions are now legacy. > > Signed-off-by: Alex Bennée <alex.bennee@linaro.org> Applied, thanks! Thomas
Thomas Petazzoni writes: > Dear Alex Bennée, > > On Wed, 6 Aug 2014 09:42:14 +0100, Alex Bennée wrote: >> gdb has had AArch64 support since 7.6 which is now the default minimum >> version of GDB in the tree. Older versions are now legacy. >> >> Signed-off-by: Alex Bennée <alex.bennee@linaro.org> > > Applied, thanks! How are you applying the patches? I'm curious because while my local patch shows up in the log with: commit 0b729ce2baa9903e7d3520196f9fff16cdff24fa Author: Alex Bennée <alex.bennee@linaro.org> The one in upstream is: commit 4372449a4f8e16f763c7d5a0c125ca015a7cbef6 Author: alex.bennee@linaro.org <alex.bennee@linaro.org> I'd usually assume utf-8 shenanigans tripping something up but I see the repo does have names with more accents than mine ;-) > > Thomas
Dear Alex Bennée, On Thu, 07 Aug 2014 10:46:47 +0100, Alex Bennée wrote: > > On Wed, 6 Aug 2014 09:42:14 +0100, Alex Bennée wrote: > >> gdb has had AArch64 support since 7.6 which is now the default minimum > >> version of GDB in the tree. Older versions are now legacy. > >> > >> Signed-off-by: Alex Bennée <alex.bennee@linaro.org> > > > > Applied, thanks! > > How are you applying the patches? I'm curious because while my local > patch shows up in the log with: > > commit 0b729ce2baa9903e7d3520196f9fff16cdff24fa > Author: Alex Bennée <alex.bennee@linaro.org> > > The one in upstream is: > > commit 4372449a4f8e16f763c7d5a0c125ca015a7cbef6 > Author: alex.bennee@linaro.org <alex.bennee@linaro.org> > > I'd usually assume utf-8 shenanigans tripping something up but I see the > repo does have names with more accents than mine ;-) We're using pwclient, which downloads the patch from patchwork and applies it with "git am". There's basically no chance for your local commit used to generate the patch to have the same SHA1 as the commit integrated in the official repository. Your commit applies on commit A, but it ends up applied on top of commit B. And since the SHA1 of the parent commit is part of what defines the next commit and therefore it's SHA1, there's no chance for your commit to have the same SHA1 in the official repo as in your local repo with the workflow we have. SHA1 would be preserved if we were taking pull requests and merging those pull requests without rebasing them. But basically, by using "git am", we're rebasing all patches on top of the latest master when applying them, which changes their SHA1. Best regards, Thomas
Hi Thomas, On Thu, Aug 07, 2014 at 12:00:31PM +0200, Thomas Petazzoni wrote: > On Thu, 07 Aug 2014 10:46:47 +0100, Alex Bennée wrote: > > > > On Wed, 6 Aug 2014 09:42:14 +0100, Alex Bennée wrote: > > >> gdb has had AArch64 support since 7.6 which is now the default minimum > > >> version of GDB in the tree. Older versions are now legacy. > > >> > > >> Signed-off-by: Alex Bennée <alex.bennee@linaro.org> > > > > > > Applied, thanks! > > > > How are you applying the patches? I'm curious because while my local > > patch shows up in the log with: > > > > commit 0b729ce2baa9903e7d3520196f9fff16cdff24fa > > Author: Alex Bennée <alex.bennee@linaro.org> > > > > The one in upstream is: > > > > commit 4372449a4f8e16f763c7d5a0c125ca015a7cbef6 > > Author: alex.bennee@linaro.org <alex.bennee@linaro.org> > > > > I'd usually assume utf-8 shenanigans tripping something up but I see the > > repo does have names with more accents than mine ;-) > > We're using pwclient, which downloads the patch from patchwork and > applies it with "git am". > > There's basically no chance for your local commit used to generate the > patch to have the same SHA1 as the commit integrated in the official > repository. Your commit applies on commit A, but it ends up applied on > top of commit B. And since the SHA1 of the parent commit is part of > what defines the next commit and therefore it's SHA1, there's no chance > for your commit to have the same SHA1 in the official repo as in your > local repo with the workflow we have. > > SHA1 would be preserved if we were taking pull requests and merging > those pull requests without rebasing them. But basically, by using "git > am", we're rebasing all patches on top of the latest master when > applying them, which changes their SHA1. Alex was asking about author name, not commit ID. Compare: Author: Alex Bennée <alex.bennee@linaro.org> Author: alex.bennee@linaro.org <alex.bennee@linaro.org> I should note that the sign-off line preserves the name correctly. baruch
Dear Baruch Siach, On Thu, 7 Aug 2014 13:09:15 +0300, Baruch Siach wrote: > > We're using pwclient, which downloads the patch from patchwork and > > applies it with "git am". > > > > There's basically no chance for your local commit used to generate the > > patch to have the same SHA1 as the commit integrated in the official > > repository. Your commit applies on commit A, but it ends up applied on > > top of commit B. And since the SHA1 of the parent commit is part of > > what defines the next commit and therefore it's SHA1, there's no chance > > for your commit to have the same SHA1 in the official repo as in your > > local repo with the workflow we have. > > > > SHA1 would be preserved if we were taking pull requests and merging > > those pull requests without rebasing them. But basically, by using "git > > am", we're rebasing all patches on top of the latest master when > > applying them, which changes their SHA1. > > Alex was asking about author name, not commit ID. Compare: > > Author: Alex Bennée <alex.bennee@linaro.org> > > Author: alex.bennee@linaro.org <alex.bennee@linaro.org> > > I should note that the sign-off line preserves the name correctly. Ah yes. Then, seems possibly like a patchwork issue, since from patchwork point of view, the author is alex.bennee@linaro.org. See http://patchwork.ozlabs.org/patch/376942/. However, it does seem to handle non-ASCII characters in author names if you look at http://patchwork.ozlabs.org/patch/377013/. http://patchwork.ozlabs.org/patch/371846/ is another example. If you look at http://patchwork.ozlabs.org/project/buildroot/list/?state=*, you'll see that Alex's patch is the only case in recent times where the e-mail was used instead of the author name. Not sure what happened. Thomas
Thomas Petazzoni writes: > Dear Alex Bennée, > > On Thu, 07 Aug 2014 10:46:47 +0100, Alex Bennée wrote: > >> > On Wed, 6 Aug 2014 09:42:14 +0100, Alex Bennée wrote: >> >> gdb has had AArch64 support since 7.6 which is now the default minimum >> >> version of GDB in the tree. Older versions are now legacy. >> >> >> >> Signed-off-by: Alex Bennée <alex.bennee@linaro.org> >> > >> > Applied, thanks! >> >> How are you applying the patches? I'm curious because while my local >> patch shows up in the log with: >> >> commit 0b729ce2baa9903e7d3520196f9fff16cdff24fa >> Author: Alex Bennée <alex.bennee@linaro.org> >> >> The one in upstream is: >> >> commit 4372449a4f8e16f763c7d5a0c125ca015a7cbef6 >> Author: alex.bennee@linaro.org <alex.bennee@linaro.org> >> >> I'd usually assume utf-8 shenanigans tripping something up but I see the >> repo does have names with more accents than mine ;-) > > We're using pwclient, which downloads the patch from patchwork and > applies it with "git am". > > There's basically no chance for your local commit used to generate the > patch to have the same SHA1 as the commit integrated in the official > repository. Sorry I was actually talking about the mangled Author field. The SHAs were just for reference. It looks like it gets mangled as it goes into patchwork. Compare: From: =?UTF-8?q?Alex=20Benn=C3=A9e?= <alex.bennee@linaro.org> To: buildroot@busybox.net Cc: =?UTF-8?q?Alex=20Benn=C3=A9e?= <alex.bennee@linaro.org> Subject: [PATCH V2] gdb: allow building on AArch64 systems Date: Wed, 6 Aug 2014 09:42:14 +0100 Message-Id: <1407314534-5536-1-git-send-email-alex.bennee@linaro.org> to Subject: [V2] gdb: allow building on AArch64 systems From: alex.bennee@linaro.org X-Patchwork-Id: 376942 Message-Id: <1407314534-5536-1-git-send-email-alex.bennee@linaro.org> To: buildroot@busybox.net Date: Wed, 6 Aug 2014 09:42:14 +0100
diff --git a/package/gdb/Config.in b/package/gdb/Config.in index cc2469e..bfbc8d8 100644 --- a/package/gdb/Config.in +++ b/package/gdb/Config.in @@ -1,5 +1,5 @@ comment "gdb/gdbserver needs a toolchain w/ threads, threads debug" - depends on !BR2_aarch64 && !BR2_nios2 + depends on !BR2_nios2 depends on !BR2_TOOLCHAIN_HAS_THREADS || !BR2_TOOLCHAIN_HAS_THREADS_DEBUG config BR2_PACKAGE_GDB @@ -10,7 +10,7 @@ config BR2_PACKAGE_GDB select BR2_PACKAGE_GDB_SERVER if \ (!BR2_PACKAGE_GDB_DEBUGGER && !BR2_TOOLCHAIN_EXTERNAL_GDB_SERVER_COPY) depends on BR2_TOOLCHAIN_HAS_THREADS && BR2_TOOLCHAIN_HAS_THREADS_DEBUG - depends on !BR2_aarch64 && !BR2_nios2 + depends on !BR2_nios2 help GDB, the GNU Project debugger, allows you to see what is going on `inside' another program while it executes -- or
gdb has had AArch64 support since 7.6 which is now the default minimum version of GDB in the tree. Older versions are now legacy. Signed-off-by: Alex Bennée <alex.bennee@linaro.org> --- v2: - reword commit. --- package/gdb/Config.in | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)