WIP: [autotools / gnu tools] use "-l:" for static libraries#11713
WIP: [autotools / gnu tools] use "-l:" for static libraries#11713jngrb wants to merge 1 commit intoconan-io:developfrom
Conversation
attempt to solve conan-io#11703 TODO: detect that the linker 'ld' is the on ein use
|
|
memsharded
left a comment
memsharded
left a comment
There was a problem hiding this comment.
We will try to have some discussion around this for next iteration (1.52, as 1.51 is being shipped right now)
| elif library.startswith("lib") and library.endswith(".a"): | ||
| # link by filename | ||
| # TODO: at this point, we should only use -l: | ||
| # if we are sure that we use ld as linker |
There was a problem hiding this comment.
Probably the way to go, aligned with Conan 2.0 design, is not try to detect it, but use a conf that explicitly defines it (opt-in or opt-out depending on the default we chose). Better than tricky, and not 100% robust auto-detections.
| if not library.endswith(".lib"): | ||
| library += ".lib" | ||
| result.append(library) | ||
| elif library.startswith("lib") and library.endswith(".a"): |
There was a problem hiding this comment.
This is the legacy build systems integrations support, that are already removed in 2.0. Better not touch them.
5 participants
attempt to solve issue #11703 - tested with gcc and ld on Ubuntu and a Yocto variant
This PR is WIP:
Changelog: (Feature | Fix | Bugfix): Describe here your pull request
Docs: https://github.com/conan-io/docs/pull/XXXX
developbranch, documenting this one.Note: By default this PR will skip the slower tests and will use a limited set of python versions. Check here how to increase the testing level by writing some tags in the current PR body text.