-
Notifications
You must be signed in to change notification settings - Fork 30
fix: update build hardening flags and cmake configuration #155
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
Conversation
1. Removed redundant compiler flags from CMakeLists.txt that are now handled by debian/rules 2. Added security hardening build options in debian/rules including: - Enabling all hardening features - Adding -Wall warning flags for C/C++ - Setting secure linker flags (RELRO, NOW, noexecstack) 3. Cleaned up CMake configuration by removing duplicate flags that are now handled at Debian build level fix: 更新构建加固标志和cmake配置 1. 从CMakeLists.txt中移除了现在由debian/rules处理的冗余编译器标志 2. 在debian/rules中添加了安全加固构建选项包括: - 启用所有加固功能 - 为C/C++添加-Wall警告标志 - 设置安全链接器标志(RELRO, NOW, noexecstack) 3. 通过移除现在由Debian构建层处理的重复标志来清理CMake配置
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
deepin pr auto review关键摘要:
是否建议立即修改:
|
Reviewer's GuideThis PR consolidates security hardening and warning flags into the Debian build layer by removing redundant flags from CMakeLists.txt and introducing comprehensive build-hardening and warning options in debian/rules. File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: 18202781743, mhduiy The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
fix: 更新构建加固标志和cmake配置
Summary by Sourcery
Consolidate security and warning flags into debian/rules for build hardening and clean up redundant CMake settings
Enhancements:
Build: