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

configure: fix check_link_silent test #387

Merged
merged 1 commit into from
Jul 23, 2021
Merged

configure: fix check_link_silent test #387

merged 1 commit into from
Jul 23, 2021

Conversation

ffontaine
Copy link
Contributor

Replace -o /dev/null by -o "$tmpc".out in check_link_silent otherwise test will fail with some bugged binutils (https://sourceware.org/bugzilla/show_bug.cgi?id=19526) since version 4.13 and 35a674b:

checking what's the option to use in linker to set library name ...
cannot find an option to set library name
package/pkg-generic.mk:249: recipe for target '/home/buildroot/autobuild/instance-2/output-1/build/proxychains-ng-4.14/.stamp_configured' failed

Fixes:

Signed-off-by: Fabrice Fontaine fontaine.fabrice@gmail.com

@rofl0r
Copy link
Owner

rofl0r commented Jul 23, 2021

check_compile needs the same fix, pls update pr

Replace -o /dev/null by -o "$tmpc".out in check_link_silent and
check_compile otherwise test will fail with some bugged binutils
(https://sourceware.org/bugzilla/show_bug.cgi?id=19526) since version
4.13 and
35a674b:

checking what's the option to use in linker to set library name ...
cannot find an option to set library name
package/pkg-generic.mk:249: recipe for target '/home/buildroot/autobuild/instance-2/output-1/build/proxychains-ng-4.14/.stamp_configured' failed

Fixes:
 - http://autobuild.buildroot.org/results/9320d9b2c69882e23bbe7b30057eb8bee0c9d2e5

Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
@rofl0r rofl0r merged commit 83bfa0b into rofl0r:master Jul 23, 2021
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

Successfully merging this pull request may close these issues.

2 participants