Skip to content
Snippets Groups Projects

libknot unified

Merged Ondřej Surý requested to merge libknot-unified into master

This MR removes extra shared libknot-* libraries from distribution.

There are now only three libraries visible to the public: libknot libdnssec libzscanner

Due to the cross-use of libknot-internal from libknotd and libknotus, some code might be linked twice, but it's inevitable unless we want to expose volatile API/ABI to the outside world.

The few more things:

  • Removed all public declarations from internal/* and yparser/*
  • Add #include libknot/internal/macros.h to internal/* and yparser/* (as a side effect from playing with symbol visibility)
  • Fixed library versioning and bumped libknot from SOVERSION 0 to 1, since some symbols have disappered since (due to visibility)

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Ondřej Surý Added 1 commit:

    Added 1 commit:

    • c91317d4 - Do some liburcu_* magic to fix builds on *BSD and Mac OS X
  • Ondřej Surý Added 1 commit:

    Added 1 commit:

    • 0eefe068 - Add $(AM_CPPFLAGS) to knotd_CPPFLAGS
  • Ondřej Surý Added 2 commits:

    Added 2 commits:

    • 96906731 - Change libknot/internal/ and libknot/yparser/ to convenience libraries
    • 3fb5ac10 - Use noinst_LTLIBRARIES for internal and yparser to make unified libknot.
  • Ondřej Surý Added 1 commit:

    Added 1 commit:

    • d2620a65 - Fix pkg-config file for libknot
  • Ondřej Surý Added 1 commit:

    Added 1 commit:

    • cfc84610 - Add lmdb include path to libknot_internal_la_CPPFLAGS
  • Ondřej Surý Added 1 commit:

    Added 1 commit:

    • 67796468 - Add dummy libknot_internal_la_CFLAGS to trigger build :)
  • Ondřej Surý Added 1 commit:

    Added 1 commit:

    • 08ef1c02 - Move lmdb-embedded stray files from libknot.la to libknot-internal.la
  • Ondřej Surý Added 1 commit:

    Added 1 commit:

    • 9f6da6b0 - Remove version-script support to remove additional complexity it introduces (it'…
  • Ondřej Surý Added 1 commit:

    Added 1 commit:

    • 7fb79384 - Add more debug output for used _CFLAGS and _LIBS to configure and explicitly lin…
  • Ondřej Surý Added 1 commit:

    Added 1 commit:

    • bbb5b9c7 - Explicitly link knotc with $(liburcu_LIBS)
  • Ondřej Surý Added 1 commit:

    Added 1 commit:

    • 312bc035 - Ok, now add liburcu_LIBS to tests/
  • Daniel Salzman mentioned in merge request !378 (closed)

    mentioned in merge request !378 (closed)

  • Ondřej Surý Added 4 commits:

    Added 4 commits:

    • ac223c31 - libzscanner have to be linked with libknotd instead of libknot
    • 1e66fa19 - Remove liburcu_LIBS from knotc_LDADD - will it break BSD linking or won't?
    • 13342d43 - Tidy up libknot.pc - we don't expose any symbols from linked libraries, so every…
    • a78ea6db - Tidy up libdnssec.pc, we don't expose neither GnuTLS nor jansson symbols outside…
  • Ondřej Surý Added 1 commit:

    Added 1 commit:

    • 42f29263 - Revert "Remove liburcu_LIBS from knotc_LDADD - will it break BSD linking or won't?"
  • Daniel Salzman Added 1 commit:

    Added 1 commit:

    • 16ec6cff - configure: reformat configuration summary
  • Ondřej Surý Added 1 commit:

    Added 1 commit:

    • 87e503f5 - Use hidden visibility only for shared libraries
  • Ondřej Surý Added 1 commit:

    Added 1 commit:

    • 60c4a16d - knotd uses liburcu directly
  • Ondřej Surý Added 1 commit:

    Added 1 commit:

    • 9eb8e334 - Workaround dumb dyld on Mac OS X by explicitly add liburcu_LIBS where needed (argh)
  • Ondřej Surý Added 1 commit:

    Added 1 commit:

    • 796d4a36 - Debug _lt_dar_allow_undefined on Mac OS X
  • Ondřej Surý Added 1 commit:

    Added 1 commit:

    • 72e357ed - Remove liburcu_LIBS, but this time compile this without evil .la files from liburcu
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
Please register or sign in to reply
Loading