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

curl-7.46.0 requires perl and GNU make #555

Closed
0-wiz-0 opened this Issue Dec 6, 2015 · 1 comment

Comments

Projects
None yet
2 participants
@0-wiz-0

0-wiz-0 commented Dec 6, 2015

curl up to 7.45.0 could be built without perl and GNU make (BSD make was fine).
In 7.46.0, the zsh support in scripts/ made both of these tools necessary.

gmake[1]: Entering directory '/scratch/www/curl/work/curl-7.46.0/scripts'
/usr/pkg/bin/perl zsh.pl > _curl
curl: not found
Use of uninitialized value $opts_str in scalar chomp at zsh.pl line 16.
Use of uninitialized value $opts_str in concatenation (.) or string at zsh.pl line 18.

About GNU make: The new Makefile rule is:

$(ZSH_COMPLETION_FUNCTION_FILENAME): zsh.pl
        $(PERL) $< > $@

This is an explicit rule that uses "$<". The man page for BSD make (on NetBSD) says about "$<":

             .IMPSRC   In suffix-transformation rules, the name/path of the
                       source from which the target is to be transformed (the
                       ``implied'' source); also known as `<'.  It is not
                       defined in explicit rules.

I.e., it's supported only in implicit rules.
I suggest just using "zsh.pl" instead of "$<".

About perl: perhaps there is a way to use awk or sed instead. Either way, the warnings should be fixed, and it should be passed a path to curl.

centminmod added a commit to centminmod/docker-ubuntu-nghttp2 that referenced this issue Dec 8, 2015

@bagder bagder closed this in f08fea7 Dec 8, 2015

@bagder

This comment has been minimized.

Show comment
Hide comment
@bagder

bagder Dec 8, 2015

Member

Thanks, fixed now. It doesn't work without perl now but it should at least handle it gracefully.

Member

bagder commented Dec 8, 2015

Thanks, fixed now. It doesn't work without perl now but it should at least handle it gracefully.

@lock lock bot locked as resolved and limited conversation to collaborators May 7, 2018

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.