net-p2p/feather-2.4.9::guru doesn't compile with current stabilized sys-devel/gcc:13, errors about unrecognized ‘uint8_t’ type name in the log Reproducible: Always Steps to Reproduce: 1. Install Gentoo from scratch or just upgrade sys-devel/gcc to current stable version (13.2.1_p20230826) 2. Enable GURU ebuild repository 3. Try to install net-p2p/feather::guru Actual Results: * ERROR: net-p2p/feather-2.4.9::guru failed (compile phase): * ninja -v -j8 -l12 feather failed * * Call stack: * ebuild.sh, line 136: Called src_compile * environment, line 2548: Called cmake_build 'feather' * environment, line 970: Called eninja 'feather' * environment, line 1453: Called die * The specific snippet of code: * "$@" || die -n "${*} failed" * * If you need support, post the output of `emerge --info '=net-p2p/feather-2.4.9::guru'`, * the complete build log and the output of `emerge -pqv '=net-p2p/feather-2.4.9::guru'`. * The complete build log is located at '/var/tmp/portage/net-p2p/feather-2.4.9/temp/build.log'. * The ebuild environment file is located at '/var/tmp/portage/net-p2p/feather-2.4.9/temp/environment'. * Working directory: '/var/tmp/portage/net-p2p/feather-2.4.9/work/feather-2.4.9_build' * S: '/var/tmp/portage/net-p2p/feather-2.4.9/work/feather-2.4.9' * * The following package has failed to build, install, or execute postinst: * * (net-p2p/feather-2.4.9:0/0::guru, ebuild scheduled for merge), Log file: * '/var/tmp/portage/net-p2p/feather-2.4.9/temp/build.log' * Expected Results: Software successful installation
Created attachment 872108 [details] build.log with error
Please put the output of 'emerge --info' in a comment.
Created attachment 872127 [details] emerge --info output
Probably "#include <cstdint>" should be added.
(In reply to Mike from comment #4) > Probably "#include <cstdint>" should be added. Yep, can be avoided by including the header (sorry not familiar enough with cpp, patch may be incorrect)
Created attachment 872385 [details, diff] <cstdint> included patch
I can submit the patch to the upstream developer if you want
this isn't the first time something like this has happened
(In reply to Adam from comment #8) > this isn't the first time something like this has happened Sorry, couldn't find how a similar situations were resolved. Of course you can do what you think is best for this situation, thanks anyway.
(In reply to Amayak Bo from comment #9) > (In reply to Adam from comment #8) > > this isn't the first time something like this has happened > > Sorry, couldn't find how a similar situations were resolved. Of course you > can do what you think is best for this situation, thanks anyway. I think it might have been a related package like p2pool or monero that also depends on epee (by proxy). There was a problem where it wasn't including some c++ stdlib. I think I fixed it by talking to the developer in IRC. On closer inspection it seems like these changes have already been made: https://github.com/feather-wallet/monero/commit/f983ac77805a494ea4a05a00398c553e1359aefd
Perhaps I've failed to update the monero sub-repository commit in the ebuild? It seemed correct at the time. I will make a feather-2.4.9-r1 with an updated monero commit, see if that works. It compiles for me but I can't reproduce the inital issue myself.
(In reply to Amayak Bo from comment #5) > (In reply to Mike from comment #4) > > Probably "#include <cstdint>" should be added. > > Yep, can be avoided by including the header (sorry not familiar enough with > cpp, patch may be incorrect) For future reference, the log says what to do: /var/tmp/portage/net-p2p/feather-2.4.9/work/feather-2.4.9/monero/contrib/epee/include/storages/parserse_base_utils.h:32:1: note: ‘uint8_t’ is defined in header ‘<cstdint>’; did you forget to ‘#include <cstdint>’? 31 | #include <boost/utility/string_ref_fwd.hpp> +++ |+#include <cstdint> 32 | #include <string>
(In reply to Adam from comment #11) > I will make a feather-2.4.9-r1 with an updated monero commit, see if that works. Thank you, can confirm feather-2.4.9-r1 installed successful.
