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

unknown command line option: --default-symver #228

Closed
dcb314 opened this issue Jan 2, 2022 · 4 comments
Closed

unknown command line option: --default-symver #228

dcb314 opened this issue Jan 2, 2022 · 4 comments

Comments

@dcb314
Copy link

dcb314 commented Jan 2, 2022

Suggest provide support for this linker flag.

@rui314
Copy link
Owner

rui314 commented Jan 2, 2022

When I was working for ld.lld, we implemented all command line options we observed in the wild for the sake of compatibility with GNU ld. ld.lld doesn't however support --default-symver, implying that the flag is used rarely. Did you need this flag for a real project? Or did you file for the sake of completeness?

@dcb314
Copy link
Author

dcb314 commented Jan 2, 2022

Did you need this flag for a real project?

Project json-glib, part of Fedora Linux and maybe some other Linux distributions, uses it.
That seems to be the only project so far.

I am busy compiling the whole of Fedora Linux. As part of that I am linking with mold.
I have done projects named [0-9][a-l]* so far, so I am about half way through.

@rui314
Copy link
Owner

rui314 commented Jan 3, 2022

Thanks. -Wl,--default-symver was added to json-glib's mason file in GNOME/json-glib@ef7fb78, and the reasoning of doing that looks legitimate. So I'll add that option to mold.

@dcb314
Copy link
Author

dcb314 commented Jan 3, 2022

So I'll add that option to mold.

Thanks. Project pesign also seems to need that flag.

@rui314 rui314 closed this as completed in 9894b31 Jan 3, 2022
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

No branches or pull requests

2 participants