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

Fix compiling in directories with spaces #185

Closed
wants to merge 1 commit into from
Closed

Conversation

BenBE
Copy link
Contributor

@BenBE BenBE commented Oct 13, 2014

This fix helps especially building OpenSSL on Windows when using Jenkins as the default build directory contains spaces in its pathname.

This patch requires the bsd_glob functions of Perl to be available which usually should be the case for any reasonable Perl configuration.

This fix helps especially building OpenSSL on Windows when using Jenkins as the default build directory contains spaces in its pathname.

This patch requires the bsd_glob functions of Perl to be available which usually should be the case for any reasonable Perl configuration.
@richsalz
Copy link
Contributor

@levitte you want to look at this?

@@ -1,7 +1,7 @@
#!/usr/local/bin/perl

use Fcntl;

use File::Glob ':glob';
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

According to my manual, :glob is discouraged. Please use :bsd_glob

@richsalz
Copy link
Contributor

+1

@levitte
Copy link
Member

levitte commented May 3, 2016

The build system has been entirely rebuilt Therefore, this PR doesn't make sense any more. Closing.

@levitte levitte closed this May 3, 2016
mcr pushed a commit to mcr/openssl that referenced this pull request Jun 22, 2021
Correctly verify abbreviated IPv6 SANs
mamckee pushed a commit to mamckee/openssl that referenced this pull request Dec 28, 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

Successfully merging this pull request may close these issues.

None yet

3 participants