|Summary:||devel/git: fails to build with PERL option and non-default LOCALBASE|
|Product:||Ports & Packages||Reporter:||Erick Turnquist <jhujhiti>|
|Component:||Individual Port(s)||Assignee:||Renato Botelho <garga>|
|Severity:||Affects Some People||CC:||garga, jcfyecrayz|
Description Erick Turnquist 2016-10-15 03:24:10 UTC
Created attachment 175773 [details] patch Upstream commit 259f22af explicitly sets the make variable PERL_PATH to /usr/local/bin/perl. For some reason this overrides the PERL_PATH we set in the port Makefile. The configure script provides a --with-perl, which works properly. Patch is attached.
Comment 1 John Hein 2016-11-30 14:59:38 UTC
Created attachment 177539 [details] [patch] use --with-perl to specify perl command I hit this, too. Attached is my patch. I leave it to the maintainer whether to bump PORTREVISION. If perl was not found, the build fails. But it's possible that the wrong perl could be found and the build succeeds. It's probably correct to bump PORTREVISION, but more than likely very few people are affected.
Comment 2 John Hein 2016-11-30 17:30:31 UTC
Just for further clarification... config.make.uname sets PERL_PATH to /usr/local/bin/perl. Putting PERL_PATH in the environment (MAKE_ENV) is not enough to override that Makefile variable assignment (the make var takes precedence over the env var in this case). Using --with-perl, however, overrides the default setting in config.mak.uname (in config.mak.autogen). So that's why using --with-perl is a better substitute for putting PERL_PATH in the environment.
Comment 3 John Hein 2016-12-01 14:51:58 UTC
QA info with attachment 177539 [details] applied: - portlint ok (no new warnings, no errors) - stage-qa ok (no new warnings) - poudriere testport ok (tested on stable-10/64)
Comment 4 commit-hook 2016-12-01 15:25:00 UTC
A commit references this bug: Author: garga Date: Thu Dec 1 15:24:06 UTC 2016 New revision: 427505 URL: https://svnweb.freebsd.org/changeset/ports/427505 Log: - Fix build with PERL and non-default LOCALBASE  - Fix build with DEVELOPER=yes  - Do not install ZSH completion  - ZSH installs a better version itself - Fix PLIST - Bump PORTREVISION PR: 213494  214009  214870  (based on) Submitted by: John Hein <firstname.lastname@example.org>   Michael Zuo <email@example.com> Sponsored by: Rubicon Communications (Netgate) Changes: head/devel/git/Makefile head/devel/git/pkg-plist