New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
net-misc/aria2: bump to 1.35.0 #18694
net-misc/aria2: bump to 1.35.0 #18694
Conversation
Pull Request assignmentSubmitter: @crabbedhaloablution net-misc/aria2: @mgorny Linked bugsBugs linked: 704086 In order to force reassignment and/or bug reference scan, please append Docs: Code of Conduct ● Copyright policy (expl.) ● Devmanual ● GitHub PRs ● Proxy-maint guide |
Pull request CI reportReport generated at: 2020-12-17 13:25 UTC There are existing issues already. Please look into the report to make sure none of them affect the packages in question: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tests fail:
FAIL: aria2c
============================================================================
Testsuite summary for aria2 1.35.0
============================================================================
# TOTAL: 1
# PASS: 0
# SKIP: 0
# XFAIL: 0
# FAIL: 1
# XPASS: 0
# ERROR: 0
============================================================================
See test/test-suite.log
Please report to https://github.com/aria2/aria2/issues
============================================================================
I get a pass.
my test/test-suite.log isn't very verbose, is yours any better?
|
Ok, I can reproduce with all the use-flags, looks like a sandbox thing. |
I think it's network sandbox, last I checked. I think disabling the two tests should fix it (but how?) |
b4d6b9d
to
57a9a40
Compare
Fixed it. The sed is not pretty looking, could be reduced to a one-liner if you like that better, but I chose to robustify it so it is more likely to survive when upstream bumps their automake or adds another test on next release. |
Pull request CI reportReport generated at: 2020-12-17 16:00 UTC There are existing issues already. Please look into the report to make sure none of them affect the packages in question: |
net-misc/aria2/aria2-1.35.0.ebuild
Outdated
src_prepare() { | ||
default | ||
sed -i -e "s|/tmp|${T}|" test/*.cc test/*.txt || die "sed failed" | ||
# Fails with all-the-use-flags && FEATURES=network-sandbox |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I dare guess it's USE=bittorrent.
net-misc/aria2/aria2-1.35.0.ebuild
Outdated
app-arch/xz-utils | ||
virtual/pkgconfig | ||
nls? ( sys-devel/gettext ) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These tree are BDEPEND in EAPI 7.
net-misc/aria2/aria2-1.35.0.ebuild
Outdated
# ssl? -> openssl || (gnutls + (nettle || libgcrypt )) | ||
# !ssl? -> nettle || libgcrypt | ||
|
||
CDEPEND="sys-libs/zlib:0= |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
CDEPEND="sys-libs/zlib:0= | |
RDEPEND="sys-libs/zlib:0= |
net-misc/aria2/aria2-1.35.0.ebuild
Outdated
libxml2? ( >=dev-libs/libxml2-2.6.26:2= ) | ||
!libxml2? ( dev-libs/expat:0= ) )" | ||
|
||
DEPEND="${CDEPEND} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
DEPEND="${CDEPEND} | |
DEPEND=${RDEPEND} |
net-misc/aria2/aria2-1.35.0.ebuild
Outdated
virtual/pkgconfig | ||
nls? ( sys-devel/gettext ) | ||
test? ( >=dev-util/cppunit-1.12.0:0 )" | ||
RDEPEND="${CDEPEND} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
RDEPEND="${CDEPEND} | |
RDEPEND+=" |
net-misc/aria2/aria2-1.35.0.ebuild
Outdated
# xmlrpc has no explicit switch, it's turned out by any XML library | ||
# so metalink implicitly forces it on | ||
REQUIRED_USE="?? ( jemalloc tcmalloc ) | ||
metalink? ( xmlrpc )" | ||
RESTRICT="!test? ( test )" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Move these just below IUSE, please.
net-misc/aria2/aria2-1.35.0.ebuild
Outdated
-e 's|LpdMessageDispatcherTest.cc||' \ | ||
-e 's|LpdMessageReceiverTest.cc||' \ | ||
-e '/LpdMessageDispatcherTest.Po/d' \ | ||
-e '/LpdMessageReceiverTest.Po/d' \ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do you actually need this? I don't think this affects the targets we're using.
net-misc/aria2/aria2-1.35.0.ebuild
Outdated
-e 's|LpdMessageReceiverTest.cc||' \ | ||
-e '/LpdMessageDispatcherTest.Po/d' \ | ||
-e '/LpdMessageReceiverTest.Po/d' \ | ||
test/Makefile.in||die |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
test/Makefile.in||die | |
test/Makefile.in || die |
net-misc/aria2/aria2-1.35.0.ebuild
Outdated
rm -rf "${D}"/usr/share/doc/aria2 \ | ||
"${D}"/usr/share/doc/${PF}/README{,.html} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
rm -rf "${D}"/usr/share/doc/aria2 \ | |
"${D}"/usr/share/doc/${PF}/README{,.html} | |
rm -r "${ED}"/usr/share/doc/aria2 \ | |
"${ED}"/usr/share/doc/${PF}/README{,.html} || die |
Bump EAPI while we're at it. Closes: https://bugs.gentoo.org/704086 Package-Manager: Portage-3.0.12, Repoman-3.0.2 Signed-off-by: Peter Alfredsen <crabbedhaloablution@icloud.com>
57a9a40
to
b06d206
Compare
Re-pushed, all requests integrated. |
Pull request CI reportReport generated at: 2020-12-17 17:05 UTC There are existing issues already. Please look into the report to make sure none of them affect the packages in question: |
default | ||
sed -i -e "s|/tmp|${T}|" test/*.cc test/*.txt || die "sed failed" | ||
# Fails with USE=bittorrent && FEATURES=network-sandbox | ||
sed -i -E \ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm sorry for missing this previously but I don't think you're actually using extended regexp syntax here.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You're correct. I was experimenting with a one-liner to start with, but felt better with a readable solution.
Overall good work, thanks! If you want me to fix that last nit, please just confirm that I'm correct and I'll do it while merging. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks. I've had to kill that /usr/share/doc/${PN}
rm because no such directory existed, and then I merged it.
Bump EAPI while we're at it.
Closes: https://bugs.gentoo.org/704086
Package-Manager: Portage-3.0.12, Repoman-3.0.2
Signed-off-by: Peter Alfredsen crabbedhaloablution@icloud.com