Summary: | net-p2p/mktorrent-1.1_p20180912 : prefix.c: error: _WORDSIZE undeclared (first use in this function) | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | Toralf Förster <toralf> |
Component: | Current packages | Assignee: | Gentoo musl team <musl> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | holgersson, opal, proxy-maint, sam |
Priority: | Normal | ||
Version: | unspecified | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- | |
Bug Depends on: | |||
Bug Blocks: | 713786 | ||
Attachments: |
emerge-info.txt
emerge-history.txt environment etc.portage.tbz2 net-p2p:mktorrent-1.1_p20180912:20200411-073220.log temp.tbz2 mktorrent-1.1_p20210130.ebuild updated ebuild (fixed version) |
Description
Toralf Förster
2020-04-11 09:27:42 UTC
Created attachment 632128 [details]
emerge-info.txt
Created attachment 632130 [details]
emerge-history.txt
Created attachment 632132 [details]
environment
Created attachment 632134 [details]
etc.portage.tbz2
Created attachment 632136 [details]
net-p2p:mktorrent-1.1_p20180912:20200411-073220.log
Created attachment 632138 [details]
temp.tbz2
It works to change SRC_URI="https://github.com/Rudde/${PN}/archive/v${PV}.tar.gz -> ${P}.tar.gz" and change version simply to 1.1. This is what Alpine's community/mktorrent package does. Also, I'm not sure whether toolchain-funcs or the 'tc-export CC' line is necessary; it builds without, but I don't have an exotic compiler setup. Created attachment 714906 [details]
mktorrent-1.1_p20210130.ebuild
updated ebuild
(In reply to opal hart from comment #7) > It works to change > SRC_URI="https://github.com/Rudde/${PN}/archive/v${PV}.tar.gz -> > ${P}.tar.gz" and change version simply to 1.1. This is what Alpine's > community/mktorrent package does. Hi, thanks for the information. We can't fall back to 1.1 as this is the old (ancient) release from 2017 which had (other) bugs on gentoo. See https://bugs.gentoo.org/650340 and https://github.com/pobrn/mktorrent/issues/22 for details. BTW, upstream shifted to a new user on github.com, but the code looks the same. I assume you have a musl setup. Could you please test the bumped ebuild I uploaded? > Also, I'm not sure whether toolchain-funcs or the 'tc-export CC' line is > necessary; it builds without, but I don't have an exotic compiler setup. Without toolchain-funcs the Makefile calls 'cc' instead of the full compiler name, e.g. 'x86_64-pc-linux-gnu-gcc'. While this seems to work fine on my ~amd64/x86 setup it might break cross compiling. To confirm, I asked about it on IRC in #gentoo-dev-help (thanks to ionen and CyberTailor!). I'm not entirely sure about it, but as the explicit CC exporting doesn't break anything on ~amd64 I prefer to keep it. > We can't fall back to 1.1 Understood. Is that reproducible with -march=native on all x86/x86_64 platforms and any generated .torrent file? If so, I'll test it both on 1.1 and with your ebuild and see if there is an improvement. > upstream shifted to a new user on github.com, but the code looks the same. You're right, I didn't pay attention to the redirect. > I assume you have a musl setup. Yep, I'll test soon. > Without toolchain-funcs the Makefile calls 'cc' Sounds like this should be filed upstream to prefer ${CC} in the makefile. Created attachment 715053 [details]
updated ebuild (fixed version)
Fix ebuild, previous version had a copy-paste-failure as pointed out by Sam.
The bug has been referenced in the following commit(s): https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=3c56f061b5d7eaf320fb28a6004ea70aea017586 commit 3c56f061b5d7eaf320fb28a6004ea70aea017586 Author: Nils Freydank <holgersson@posteo.de> AuthorDate: 2021-07-08 09:48:31 +0000 Commit: Joonas Niilola <juippis@gentoo.org> CommitDate: 2021-07-08 09:49:09 +0000 net-p2p/mktorrent: bump snapshot versionm, respect CC Bug: https://bugs.gentoo.org/717032 Signed-off-by: Joonas Niilola <juippis@gentoo.org> net-p2p/mktorrent/Manifest | 1 + net-p2p/mktorrent/mktorrent-1.1_p20210130.ebuild | 45 ++++++++++++++++++++++++ 2 files changed, 46 insertions(+) |