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

Build binaries with relro/pie/bind_now #281

Closed

Conversation

idorax
Copy link

@idorax idorax commented Jan 16, 2023

Note that package 'annobin-plugin-gcc' should be installed on the system to build the binaries.

Fixes: #255

@idorax idorax force-pushed the dev.huanli.20230117.255 branch 2 times, most recently from 606f3e5 to 03b39d2 Compare January 16, 2023 13:44
@idorax idorax marked this pull request as draft January 16, 2023 13:46
@idorax idorax force-pushed the dev.huanli.20230117.255 branch 2 times, most recently from 2ab526c to 02c259f Compare January 16, 2023 14:33
@idorax idorax force-pushed the dev.huanli.20230117.255 branch from 02c259f to 100eb06 Compare February 7, 2023 13:05
@comps
Copy link

comps commented Feb 7, 2023

Looks good to me, as long as annocheck doesn't find any issues when this is built. Thanks!

Note that the second export LDFLAGS="$RPM_LD_FLAGS" is useless as it's already covered by the LDFLAGS above it. The condition in the file is only for CFLAGS for i486.

@idorax idorax force-pushed the dev.huanli.20230117.255 branch from 100eb06 to 6ea6862 Compare February 7, 2023 13:43
@idorax
Copy link
Author

idorax commented Feb 7, 2023

Note that the second export LDFLAGS="$RPM_LD_FLAGS" is useless as it's already covered by the LDFLAGS above it. The condition in the file is only for CFLAGS for i486.

Removed. @comps, thanks for your review!

@idorax idorax marked this pull request as ready for review February 7, 2023 13:52
@idorax
Copy link
Author

idorax commented Feb 8, 2023

Hi @jbastian and @p3ck, would you please help to review this PR? Thanks!

@idorax
Copy link
Author

idorax commented Feb 27, 2024

Looks the priority of the related issue is very low, so just close this PR to avoid unnecessary efforts.

@idorax idorax closed this Feb 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

restraint binaries are not built with relro/pie/bind_now
2 participants