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.

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

Jump to bottom

[SITL] suppress spurious linker warning with gcc12+ #9163

Merged
merged 2 commits into from
Jul 7, 2023
Merged

[SITL] suppress spurious linker warning with gcc12+ #9163

merged 2 commits into from
Jul 7, 2023

Conversation

stronnag
Copy link
Collaborator

@stronnag stronnag commented Jul 6, 2023

gcc 12 and later generate a mainly useless linker warning:

warning: <TARGET>.elf has a LOAD segment with RWX permissions

For the SITL, ex-MacOS, suppress this.

Tested on:

  • Arch Linux (gcc 13.1.1)
  • FreeBSD (gcc 13.0.1)
  • Cygwin (gcc 11.3)
  • MacOS (clang)

Note: If it is ever possible to build reliable firmware with cmake and gcc 12 or later, the same directive may be applied to firmware builds.

@stronnag stronnag added this to the 7.0 milestone Jul 6, 2023
@stronnag
Copy link
Collaborator Author

stronnag commented Jul 6, 2023

Well, well gcc, 11.3 / Cygwin works, 11.3 Linux doesn't.

@stronnag stronnag closed this Jul 6, 2023
@stronnag stronnag reopened this Jul 6, 2023
@DzikuVx DzikuVx merged commit 5b0410a into master Jul 7, 2023
28 checks passed
@DzikuVx DzikuVx deleted the jh_suppress_sitl_ld_rwx_warning branch July 7, 2023 07:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
7.0
Development

Successfully merging this pull request may close these issues.
2 participants
@stronnag @DzikuVx