[Bug 1398922] Review Request: uftrace - User-space function call tracer for C and C++ programs

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



https://bugzilla.redhat.com/show_bug.cgi?id=1398922



--- Comment #12 from Michael Schwendt <bugs.michael@xxxxxxx> ---
The fedora-review tool is equally helpful to the package maintainer as to the
reviewer.


About GPLv2 or GPLv2+, a simple grep turns up these false positives:

  $ grep "or later" * -R
  libtraceevent/trace-seq.c: * into a special buffer (@s) for later retrieval
by a sequencer
  libtraceevent/event-parse.c:    /* Save for later use. */


> 1) These BR are not needed: gcc

Wouldn't be a big issue. The default buildroot doesn't contain gcc-c++ anymore,
however. So, if gcc were not found, you would notice that with a failing build.


> 2) Unversioned so-files not in -devel subpackage.

They are dlopen()ed plugins. I'd rather adjust the INSTALL_LIB_PATH definition
before compiling uftrace and install them into a private path below $libdir
instead of directly into $libdir.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
_______________________________________________
package-review mailing list -- package-review@xxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to package-review-leave@xxxxxxxxxxxxxxxxxxxxxxx




[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]