FS#41690 - [perf] add trace support

Attached to Project: Community Packages
Opened by ... (spider007) - Monday, 25 August 2014, 14:04 GMT
Last edited by Sébastien Luttringer (seblu) - Monday, 06 October 2014, 00:23 GMT
Task Type Feature Request
Category Packages
Status Closed
Assigned To Sébastien Luttringer (seblu)
Architecture All
Severity Medium
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 1
Private No

Details

Description:

both `perf trace` and `trace` throw this error:

> trace command not available: missing audit-libs devel package at build time.
This task depends upon

Closed by  Sébastien Luttringer (seblu)
Monday, 06 October 2014, 00:23 GMT
Reason for closing:  Implemented
Additional comments about closing:  linux-tools-3.17-1
Comment by ... (spider007) - Monday, 25 August 2014, 14:17 GMT
I have confirmed that installing audit (from aur) fixes this issue and produces a perf-package with working trace functionality.
Unfortunately the audit-package is also needed to actually use this functionality; not sure what the policy is on optional dependencies on packages in aur?
Comment by Sébastien Luttringer (seblu) - Tuesday, 26 August 2014, 08:59 GMT Comment by Sébastien Luttringer (seblu) - Tuesday, 26 August 2014, 09:26 GMT
We don't have libaudit or audit in our repository. Either someone[1] will add this package in our official repository, or perf will not enable trace command.

[1] https://mailman.archlinux.org/pipermail/arch-dev-public/2014-May/026291.html
Comment by ... (spider007) - Wednesday, 27 August 2014, 10:01 GMT
Why not add a build-dependency on audit; and possibly an optdep on audit (even though it's not in the repos). Building it correctly would already enable all functionality; the only difference is perf-trace currently throws one error; and will then throw another?
Comment by Sébastien Luttringer (seblu) - Thursday, 25 September 2014, 22:20 GMT
I will not build against aur package, because we don't do that. Daniel has suggested he may push audit back to community, no news since May 2014.
Comment by Steven Honeyman (stevenhoneyman) - Thursday, 25 September 2014, 22:50 GMT
What's the issue preventing moving audit back into community? (I'm just curious)
Comment by Daniel Micay (thestinger) - Monday, 29 September 2014, 05:38 GMT
> Why not add a build-dependency on audit; and possibly an optdep on audit (even though it's not in the repos). Building it correctly would already enable all functionality; the only difference is perf-trace currently throws one error; and will then throw another?

It wouldn't be a runtime dependency. It links against the library and it won't start if it's missing.
Comment by Daniel Micay (thestinger) - Monday, 29 September 2014, 05:39 GMT
I moved audit back to community with a post-install message warning that the official kernel packages lack support for it. The utility functions used by perf do work fine without the kernel support.
Comment by ... (spider007) - Monday, 29 September 2014, 11:09 GMT
> I moved audit back to community with a post-install message warning that the official kernel packages lack support for it. The utility functions used by perf do work fine without the kernel support.

okay, nice. But perf still needs to be rebuild. It now says (even with audit installed):

> trace command not available: missing audit-libs devel package at build time.
Comment by Sébastien Luttringer (seblu) - Tuesday, 30 September 2014, 13:26 GMT
3.17 is planned for end of the week, no need to hurry.
Comment by ... (spider007) - Wednesday, 01 October 2014, 10:56 GMT
You mean perf-3.17 will be be build with audit included? That'd be great!

Loading...