fix(nspr): drop runtests.pl so nspr-dev does not need perl - #692
Merged
Conversation
The perl removal broke the build: nspr-dev ships
${libdir}/nspr/tests/runtests.pl, and the file-rdeps QA check requires a
provider of /usr/bin/perl in RDEPENDS:nspr-dev, which the distro removes.
ERROR: nspr-4.38.2-r0 do_package_qa: QA Issue:
/usr/lib/nspr/tests/runtests.pl contained in package nspr-dev requires
/usr/bin/perl, but no providers found in RDEPENDS:nspr-dev? [file-rdeps]
The script only drives the nspr test binaries, which no image runs, so removing
it keeps the QA check honest instead of silencing it.
Signed-off-by: Jan Zachmann <50990105+JanZachmann@users.noreply.github.com>
mlilien
approved these changes
Aug 21, 2026
HarryWaschkeit
approved these changes
Aug 21, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
New
nsprbbappend that deletes${libdir}/nspr/tests/runtests.plfor target builds.Reason
#690 broke the build.
RDEPENDS:nspr-dev:remove = "perl"takes perl out of the build graph, butnspr-devstill ships a perl script, so thefile-rdepsQA check fails:The recipe installs that script itself (
install -m 0755 ${S}/pr/tests/runtests.pl ${D}${libdir}/nspr/tests) and packages it through the${libdir}/nspr/tests/*glob inFILES:${PN}-dev. It only drives the nspr test binaries, which no image runs, and the recipe already deletescompile-et.plfrom${bindir}for exactly the same reason. Deleting this one file is therefore the honest fix; the alternative,INSANE_SKIP:nspr-dev += "file-rdeps", would only silence a check that is telling the truth.This is a build-machine-independent failure: it is
nspr:do_package_qa, so it hits every build whose graph contains nspr. From the same trigger (wrynose/dev/gateway-devel, build 6):rpi4-omnect-labfailed at 13:44 - the first build to reach that taskgenericx86-64hit the identical error while still runningdehndetectpassed: nspr does not appear once in its build log, so that image does not pull it intauril2build 6 errored for an unrelated reason (no nspr in its log); 6.1 is runningrpi4was still before that task at the time of writingIf CI still fails on nspr after this, the fallback is to revert the
RDEPENDS:nspr-dev:removeline from #690 and accept perl in the build and the SBOM again.