Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[201811][dhcp] enable building dhcpmon and dhcp6relay debug symbols #9959

Merged
merged 3 commits into from
Feb 11, 2022

Conversation

yxieca
Copy link
Contributor

@yxieca yxieca commented Feb 10, 2022

Why I did it

dhcpmon and dhcp6relay debug symbols are not built for 201811 branch

How I did it

Cherry-pick #9013 and #9862, then made changes to fit 201811/Jessie.

How to verify it

Run dhcp6 relay tests.

saiarcot895 and others added 3 commits February 10, 2022 23:15
…#9013)

This makes it possible to install the debug symbols if needed. Also install
the package into the debug version of sonic-dhcp-relay container.

Signed-off-by: Saikrishna Arcot <[email protected]>
…#9862)

Enable dbgsym package for dhcpmon.

Allow CFLAGS and LDFLAGS from environment variables to be used
in the dhcp6relay build. This makes sure that the -O2 flag from
dpkg-buildflags gets used.

Finally, enable all hardening flags in dpkg-buildflags for
dhcp6relay and dhcpmon. The change from the default set of flags is that
during linking, immediate binding of symbols is done instead of lazy
binding.

Signed-off-by: Saikrishna Arcot <[email protected]>
@yxieca yxieca requested a review from saiarcot895 February 10, 2022 23:20
@yxieca yxieca changed the title [dhcp] enable building dhcpmon and dhcp6relay debug symbols [201811][dhcp] enable building dhcpmon and dhcp6relay debug symbols Feb 10, 2022
@yxieca yxieca requested a review from kellyyeh February 10, 2022 23:26
@yxieca yxieca merged commit 76af3dc into sonic-net:201811 Feb 11, 2022
@yxieca yxieca deleted the debug branch February 11, 2022 04:57
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.

3 participants