Summary: | dev-util/android-tools-9.0.0_p3 : /.../unistd_ext.h:34:16: error: declaration of _pid_t gettid() noexcept has a different exception specifier | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | Toralf Förster <toralf> |
Component: | Current packages | Assignee: | Zac Medico <zmedico> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | ao, creideiki+gentoo-bugzilla, gmt, hlein, javran.c, klaus818, nowa, xaviermiller |
Priority: | Normal | ||
Version: | unspecified | ||
Hardware: | All | ||
OS: | Linux | ||
See Also: | https://bugs.gentoo.org/show_bug.cgi?id=691846 | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- | |
Attachments: |
emerge-info.txt
dev-util:android-tools-9.0.0_p3:20190808-092344.log emerge-history.txt environment etc.portage.tbz2 logs.tbz2 use-glibc-s-gettid-when-using-glibc-2.30.patch |
Description
Toralf Förster
![]() Created attachment 586224 [details]
emerge-info.txt
Created attachment 586226 [details]
dev-util:android-tools-9.0.0_p3:20190808-092344.log
Created attachment 586228 [details]
emerge-history.txt
Created attachment 586230 [details]
environment
Created attachment 586232 [details]
etc.portage.tbz2
Created attachment 586234 [details]
logs.tbz2
The failure here is glibc-2.30's gettid() collision: FAILED: core/libcutils/threads.cpp.o x86_64-pc-linux-gnu-g++ -std=gnu++17 -O2 -pipe -march=native -D_GNU_SOURCE -Icore/libcutils/include -Icore/include -c core/libcutils/threads.cpp -o core/libcutils/threads.cpp.o In file included from /usr/include/unistd.h:1170, from core/libcutils/threads.cpp:29: /usr/include/bits/unistd_ext.h:34:16: error: declaration of ‘__pid_t gettid() noexcept’ has a different exception specifier 34 | extern __pid_t gettid (void) __THROW; | ^~~~~~ In file included from core/libcutils/threads.cpp:17: core/libcutils/include/cutils/threads.h:40:14: note: from previous declaration ‘pid_t gettid()’ 40 | extern pid_t gettid(); | ^~~~~~ *** Bug 695906 has been marked as a duplicate of this bug. *** (In reply to Sergei Trofimovich from comment #7) > The failure here is glibc-2.30's gettid() collision: It looks like upstream merged a fix for this in https://github.com/nmeum/android-tools/pull/9 I don't have time to test it right now, maybe some other watcher can? Created attachment 595900 [details, diff] use-glibc-s-gettid-when-using-glibc-2.30.patch (In reply to Hank Leininger from comment #9) > (In reply to Sergei Trofimovich from comment #7) > > The failure here is glibc-2.30's gettid() collision: > > It looks like upstream merged a fix for this in > https://github.com/nmeum/android-tools/pull/9 > > I don't have time to test it right now, maybe some other watcher can? Here is, for the lack of a better term, a epatchuserified version of https://raw.githubusercontent.com/nmeum/android-tools/237abccb95cadd78f2c431bc34974c2598804c21/patches/core/0022-Use-glibc-s-gettid-when-using-glibc-2.30.patch which does indeed resolve the problem on my ~amd64. Ideally if the ebuild were to adopt such a thing, it might try harder than I did to use the original patch (but there was fuzz on the epatchuser side of src_prepare... depending on which ebuild phase that patchfuzz came from, maybe not worth the effort or even possible, so, gl! :) Your OCD may vary :P). Thanks for the fix. Works for me. Worked for me as well, thank you. The bug has been closed via the following commit(s): https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=ce40dac7505118d5fcff820d21e338b571ab79ed commit ce40dac7505118d5fcff820d21e338b571ab79ed Author: Tobias M <tmmonckton@gmail.com> AuthorDate: 2019-11-10 17:37:00 +0000 Commit: Zac Medico <zmedico@gentoo.org> CommitDate: 2019-12-09 04:03:32 +0000 dev-util/android-tools: 9.0.0_p3-r1 revbump for bug 691764 This patch prevents collisions with the gettid() function in glibc-2.30. Closes: https://bugs.gentoo.org/691764 Package-Manager: Portage-2.3.81, Repoman-2.3.20 Signed-off-by: Zac Medico <zmedico@gentoo.org> dev-util/android-tools/Manifest | 1 + .../android-tools/android-tools-9.0.0_p3-r1.ebuild | 135 +++++++++++++++++++++ 2 files changed, 136 insertions(+) |