Skip to content
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

testing/bcc: split into subpackages #9937

Closed
wants to merge 1 commit into from

Conversation

acj
Copy link
Contributor

@acj acj commented Aug 7, 2019

Now that libbpf has releases, I'm including it as part of source to simplify the build and patches. See discussion below.

@probot-autolabeler probot-autolabeler bot added the R-testing Testing repository label Aug 7, 2019
@kaey
Copy link
Contributor

kaey commented Aug 9, 2019

pkg-config files should be in -dev.
bcc-tools should also depend on py-bcc.
Everything else looks good.

@acj
Copy link
Contributor Author

acj commented Aug 10, 2019

Fixed. Thanks.

@acj
Copy link
Contributor Author

acj commented Aug 25, 2019

@maxice8 @Ikke Sorry for the direct ping, but if you have time for a review, I would love to get this merged. It's blocking work on several downstream projects.

testing/bcc/APKBUILD Outdated Show resolved Hide resolved
testing/bcc/APKBUILD Outdated Show resolved Hide resolved
@acj acj force-pushed the bcc-subpackages branch 3 times, most recently from 9fc309d to 85838a7 Compare August 27, 2019 11:44
@@ -33,6 +34,28 @@ build() {
package() {
cd "$builddir/build"
make install DESTDIR="$pkgdir"

mv "$pkgdir"/usr/share/bcc/man "$pkgdir"/usr/share/man
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tools which these man pages refer to are not in PATH, so it's better to leave them in /usr/share/bcc/man.
Furthermore in a shared space they would conflict with man pages provided by bpftrace

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fair enough. Moved them back

@algitbot
Copy link

Merged in 34aa41f by @maxice8. Thanks for your contribution!

(This pull request has been closed automatically by GitHub PR Closer. If you think that it’s not resolved yet, please add a comment.)

@algitbot algitbot closed this Aug 28, 2019
@acj acj deleted the bcc-subpackages branch July 17, 2020 00:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
R-testing Testing repository
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants