Skip to content

Fix bpfilter.pc debug link flags#15

Merged
qdeslandes merged 1 commit into
facebook:mainfrom
qdeslandes:fix_pkgconfig_ldflags_debug
Sep 8, 2023
Merged

Fix bpfilter.pc debug link flags#15
qdeslandes merged 1 commit into
facebook:mainfrom
qdeslandes:fix_pkgconfig_ldflags_debug

Conversation

@qdeslandes
Copy link
Copy Markdown
Contributor

When building in debug mode, bpfilter (and libbpfilter) is linked to asan and ubsan. However, those link flags are not reflected in bpfilter.pc, leading to build failure in downstream projects.

Use a generator expression to add -fsanitize=address and -fsanitize=undefined to bpfilter.pc in debug mode.

When building in debug mode, bpfilter (and libbpfilter) is linked to asan
and ubsan. However, those link flags were not reflected in bpfilter.pc,
leading to build failure in downstream projects.

Use a generator expression to add -fsanitize=address and
-fsanitize=undefined to bpfilter.pc in debug mode.

Signed-off-by: Quentin Deslandes <qde@naccy.de>
@qdeslandes qdeslandes force-pushed the fix_pkgconfig_ldflags_debug branch from ad0ee2b to 3260b60 Compare September 8, 2023 18:20
@qdeslandes qdeslandes merged commit b438d33 into facebook:main Sep 8, 2023
@codecov
Copy link
Copy Markdown

codecov Bot commented Sep 8, 2023

Codecov Report

Patch and project coverage have no change.

Comparison is base (1293026) 10.70% compared to head (3260b60) 10.70%.
Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #15   +/-   ##
=======================================
  Coverage   10.70%   10.70%           
=======================================
  Files          33       33           
  Lines        2260     2260           
=======================================
  Hits          242      242           
  Misses       2018     2018           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@qdeslandes qdeslandes deleted the fix_pkgconfig_ldflags_debug branch September 8, 2023 18:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants