[Bug 1647948] Review Request: bpftrace - High-level tracing language for Linux eBPF

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

 



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

Augusto Caringi <acaringi@xxxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
              Flags|needinfo?(acaringi@redhat.c |
                   |om)                         |



--- Comment #7 from Augusto Caringi <acaringi@xxxxxxxxxx> ---
(In reply to Jerome Marchand from comment #5)
> >%cmake . \
> >        -DCMAKE_BUILD_TYPE=RelWithDebInfo \
> >        -DSYSTEM_BCC_LIBRARY:BOOL=ON \
> >        -DENABLE_TESTS:BOOL=OFF \
> >        -DBUILD_SHARED_LIBS:BOOL=OFF
> 
> Any specific reason why we don't build the dynamic libraries?

That's a good question, but the answer is: There is no dynamic libraries.

I'll explain...

By default, during the build, bpftrace creates 4 static libraries:

- libresources.a
- libparser.a
- libarch.a
- libast.a

These "private" libraries are then statically linked in bpftrace binary (which
is a dynamic linked executable regarding all the other external libraries).

Then when I was working on this package I realized that %cmake macro has a
"-DBUILD_SHARED_LIBS:BOOL=ON" which was forcing the build of these "private
static by default" libraries as dynamic libraries.

This behavior was causing some problems (bpftrace CMake files are not prepared
to handle these libraries as dynamic ones, rpmbuild was identifying new
dependencies based on these libraries and so on...)

So, this line: "-DBUILD_SHARED_LIBS:BOOL=OFF" is there just to force back the
default behavior.

-- 
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
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: https://lists.fedoraproject.org/archives/list/package-review@xxxxxxxxxxxxxxxxxxxxxxx




[Index of Archives]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite Conditions]     [KDE Users]

  Powered by Linux