Summary: | net-p2p/feather-2.4.9::guru: compilation failure | ||
---|---|---|---|
Product: | GURU | Reporter: | Amayak Bo <gentoo_bugtracker> |
Component: | Package issues | Assignee: | Adam <adam.pimentel46> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | neb.semqen.ramesses |
Priority: | Normal | ||
Version: | unspecified | ||
Hardware: | AMD64 | ||
OS: | Linux | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- | |
Attachments: |
build.log with error
emerge --info output <cstdint> included patch |
Description
Amayak Bo
2023-10-03 20:41:10 UTC
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. |