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

COMMANDLINE_LIBRARY fallback to $(wildcard $(GNU_DIR) #296

Closed

Conversation

mdavidsaver
Copy link
Member

Attempt to restore auto-detect with older compilers (eg. GCC 4.8 circa RHEL7). cf #254

https://epics.anl.gov/core-talk/2022/msg00381.php

This PR is not a simple revert. For simplicity, the older $(wildcard) logic is placed in an #else clause within configure/toolchain.c (which is really a makefile) where it will be run for all targets, not just the selectlinux* targets where it was previously found (cf. 42c7dbc).

As this change tests for GNU_DIR being not-empty, I don't think this would effect MSVC or RTEMS targets. It might effect linux, apple, solaris, cygwin, mingw, and vxworks. Where it could cause breakage if some (misconfigured) target has GNU_DIR set incorrectly (eg. for the host of a cross build), or if the headers are detect but the library is not in the search path.

@AppVeyorBot
Copy link

@anjohnson
Copy link
Member

Core Group 12/21: Low risk, apply.

@anjohnson anjohnson added the Approved Ready To Merge label Dec 21, 2022
@mdavidsaver
Copy link
Member Author

Applied as 52cc684

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Approved Ready To Merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants