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

meson: use chosen linker #8481

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

meson: use chosen linker #8481

wants to merge 1 commit into from

Conversation

dhewg
Copy link
Contributor

@dhewg dhewg commented Jan 5, 2024

meson requires setting the linker via c_ld/cpp_ld.
This now links meson based package builds with the chosen DEFAULT_LINKER.

@dhewg
meson: use chosen linker
52858e7 
meson requires setting the linker via c_ld/cpp_ld.
This now links meson based package builds with the chosen
DEFAULT_LINKER.

Signed-off-by: Andre Heider <a.heider@gmail.com>
@dhewg dhewg mentioned this pull request Jan 5, 2024
Merged
@heitbaum heitbaum added MERGE BLOCKED BUILDSYSTEM NEEDS REVIEW Issue needs review from a team member. LE 13.0 labels Jan 5, 2024
@heitbaum
Copy link
Contributor

heitbaum commented Jan 5, 2024

For review and testing with LE13. This changes the meson ld from bfd to gold/mold.

Member
@heitbaum heitbaum 13 hours ago
This makes sense. Does this mean that we have been usingbfd for all meson:target builds? What is the impact of these builds now using gold or mold? (Well specifically gold as it is the default in the LE12 image?)
Member
Author
@dhewg dhewg 12 hours ago
Yes, I believe those have been using bfd all along.
I only tested mold for my allwinner board though. While that works just fine, I neither tested gold nor any other platform
Member
@heitbaum heitbaum 11 hours ago
This one will need more testing then. I just noticed pixman on the 0.43.0 update is meson and it explicitly wanted bfd in configure, but with this change gold would be used. So let’s hold this commit back for LE12.

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
BUILDSYSTEM LE 13.0 MERGE BLOCKED NEEDS REVIEW Issue needs review from a team member.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@dhewg @heitbaum