Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 731106 - dev-util/perf with EAPI=7 should use BDEPEND
Summary: dev-util/perf with EAPI=7 should use BDEPEND
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal
Assignee: Naohiro Aota
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2020-07-06 16:33 UTC by David Michael
Modified: 2021-02-06 21:33 UTC (History)
4 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description David Michael 2020-07-06 16:33:44 UTC
All of the ebuilds are using EAPI 7, but none defines BDEPEND.  This makes building the package in a sysroot fail because required tools don't get installed in the native root.

Reproducible: Always

Steps to Reproduce:
1. emerge -v perf  # with a sysroot profile

Actual Results:  
DEPEND installs tools like asciidoc in the sysroot, so the commands are not found and the build fails.

Expected Results:  
It should build and install.

At least bison, flex, and all the doc stuff should be in BDEPEND.  Probably others, but I didn't verify them.  Tested with "--rage-clean"ing packages out of the native root, then installing perf in the sysroot with --nodeps.
Comment 1 Larry the Git Cow gentoo-dev 2020-11-19 15:36:43 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=c2716da2764b18e49eecbf61c4864a5cc0da8d72

commit c2716da2764b18e49eecbf61c4864a5cc0da8d72
Author:     Guilherme Amadio <amadio@gentoo.org>
AuthorDate: 2020-11-19 15:20:33 +0000
Commit:     Guilherme Amadio <amadio@gentoo.org>
CommitDate: 2020-11-19 15:36:00 +0000

    dev-util/perf: version bump to 5.9.9
    
    Add support for libpfm
    Fix perf version command
    Fix installation of tips.txt
    Move warning about USE=-doc to pkg_pretend
    Move build dependencies to BDEPEND
    
    Closes: https://bugs.gentoo.org/588152
    Closes: https://bugs.gentoo.org/618790
    Closes: https://bugs.gentoo.org/731106
    Package-Manager: Portage-3.0.9, Repoman-3.0.2
    Signed-off-by: Guilherme Amadio <amadio@gentoo.org>

 dev-util/perf/Manifest          |   2 +
 dev-util/perf/metadata.xml      |   1 +
 dev-util/perf/perf-5.9.9.ebuild | 246 ++++++++++++++++++++++++++++++++++++++++
 3 files changed, 249 insertions(+)
Comment 2 David Michael 2020-11-19 15:42:30 UTC
Reopening since the new ebuild still does not contain BDEPEND.
Comment 3 David Michael 2020-11-19 17:22:14 UTC
It looks like BDEPEND was added later, but now patchutils should still be moved from DEPEND to BDEPEND.
Comment 4 Larry the Git Cow gentoo-dev 2021-02-06 21:33:50 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=80021431801184c01735b89a024b79f8de1f1c4a

commit 80021431801184c01735b89a024b79f8de1f1c4a
Author:     Sam James <sam@gentoo.org>
AuthorDate: 2021-02-06 21:33:45 +0000
Commit:     Sam James <sam@gentoo.org>
CommitDate: 2021-02-06 21:33:45 +0000

    dev-util/perf: move patchutils to BDEPEND
    
    Closes: https://bugs.gentoo.org/731106
    Package-Manager: Portage-3.0.14, Repoman-3.0.2
    Signed-off-by: Sam James <sam@gentoo.org>

 dev-util/perf/perf-5.9.9.ebuild | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)