Skip to content

fix(cmake): drop -pie from global link options #50

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.

Sign up for GitHub

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

Open
hillwoodroc wants to merge 1 commit into
base: master
Choose a base branch
from
Open

fix(cmake): drop -pie from global link options #50

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion CMakeLists.txt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ set(CMAKE_CXX_STANDARD_REQUIRED ON)

# 安全编译选项
add_compile_options(-fstack-protector-strong -D_FORTITY_SOURCE=1)
add_link_options(-z noexecstack -pie -fPIC)
add_link_options(-z noexecstack -fPIC)

@lzwind lzwind Feb 7, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This compilation parameter is required for security purposes.

@lzwind lzwind Feb 7, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If the architecture for this parameter cannot be found, isolation is recommended.

@hillwoodroc hillwoodroc Feb 7, 2026

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

-pie is specifically used for generating executables; since this is merely a shared library, there is no need to use this parameter.

Refer：https://gcc.gnu.org/onlinedocs/gcc/Link-Options.html?hl=zh-CN

--pie

Produce a dynamically linked position independent executable on targets that support it. For predictable results, you must also specify the same set of options used for compilation (-fpie, -fPIE, or model suboptions) when you specify this linker option.

@lzwind lzwind Feb 7, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The parameters for fpic do seem incorrect; it's recommended to remove this unnecessary parameter. pie is indeed necessary.

@wineee wineee Feb 7, 2026

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

use DEB_BUILD_HARDENING_PIE:


# 设置安装路径
include(GNUInstallDirs)
Expand Down