Summary: | net-p2p/mldonkey-3.1.7-r2 - src/.../CryptoPP.cc: error: reference to byte is ambiguous (maybe gcc-11 related) | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | Toralf Förster <toralf> |
Component: | Current packages | Assignee: | Jesús P Rey (Chuso) <gentoo> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | esigra, phantom4, proxy-maint, sam |
Priority: | Normal | Keywords: | PullRequest |
Version: | unspecified | ||
Hardware: | All | ||
OS: | Linux | ||
See Also: | https://github.com/gentoo/gentoo/pull/21663 | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- | |
Bug Depends on: | |||
Bug Blocks: | 251464, 732706 | ||
Attachments: |
emerge-info.txt
emerge-history.txt environment etc.portage.tar.bz2 logs.tar.bz2 net-p2p:mldonkey-3.1.7-r2:20210514-060456.log temp.tar.bz2 |
Description
Toralf Förster
2021-05-14 07:17:30 UTC
Created attachment 708285 [details]
emerge-info.txt
Created attachment 708288 [details]
emerge-history.txt
Created attachment 708291 [details]
environment
Created attachment 708294 [details]
etc.portage.tar.bz2
Created attachment 708297 [details]
logs.tar.bz2
Created attachment 708300 [details]
net-p2p:mldonkey-3.1.7-r2:20210514-060456.log
Created attachment 708303 [details]
temp.tar.bz2
Is this bundling crypto++? It still compiles correctly for me with sys-devel/gcc:10. I think the issue is only triggered with sys-devel/gcc:11, which is still in ~arch. There seems to be an upstream patch for it: https://github.com/ygrek/mldonkey/commit/d9394aa I'm currently merging sys-devel/gcc:11 to try to reproduce the issue and test the patch. Answering Sam's question and looking at the comments inside CryptoPP.cc, yes, this seems to be a modified copy of crypto++: > // This file contains a subset of the Crypto++ library (version 5.2.1), with > // kind permission from Wei Dai. Please note that this file should not reflect > // on the real Crypto++ library in any way, as this file been greatly mangled to > // reduce the code-size, since this app only makes use of RSA classes (for Secure > // Identification) and the MD4 hashing class (for FileIDs). I don't know how easy (if possible) unbundling it could be without modifying the code. Looking at the Makefile, it seems this is used by DonkeySUI and could be avoided with --disable-donkeysui (with the obvious side effect of disabling SecureUserIdent for the eDonkey protocol). I think the easiest and fastest solution is testing the upstream patch. The upstream patch doesn't fix it for me. This will need more research. The bug has been closed via the following commit(s): https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=207859482e3cca4d24bbb1d96e7a0f3511725d38 commit 207859482e3cca4d24bbb1d96e7a0f3511725d38 Author: Jesus P Rey (Chuso) <gentoo@chuso.net> AuthorDate: 2021-07-15 18:56:30 +0000 Commit: Sam James <sam@gentoo.org> CommitDate: 2021-07-16 01:33:18 +0000 net-p2p/mldonkey: Patch for C++17 support. Signed-off-by: Jesus P Rey <gentoo@chuso.net> Closes: https://bugs.gentoo.org/790134 Closes: https://github.com/gentoo/gentoo/pull/21663 Signed-off-by: Sam James <sam@gentoo.org> net-p2p/mldonkey/files/cpp17-byte-namespace.patch | 63 +++++++++++++++++++++++ net-p2p/mldonkey/mldonkey-3.1.7-r2.ebuild | 2 + 2 files changed, 65 insertions(+) |