[Bug 205600] arm's lang/perl5.20: via pkg install perl5: cc => '/nxb-bin/usr/bin/cc' in /usr/local/lib/perl5/5.20/mach/Config.pm

[bugzilla-noreply at freebsd.org]

https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=205600

--- Comment #4 from Mathieu Arnold <mat at FreeBSD.org> ---
(In reply to Mark Millard from comment #3)
> (In reply to Mathieu Arnold from comment #2)
> 
> (1) A mistake?
> 
> The ${REINPLACE_CMDNPLACE_CMD} uses seem to translate to empty strings. (I
> tried a direct "make post-build" in /usr/ports/lang/perl5.22 with CC set
> appropriately because the REINPLACE_CMDNPLACE_CMD text just looked odd. This
> was after patching the Makefile.)
> 
> Should those have been ${REINPLACE_CMD} uses instead?

Ah, yes, something funny happened there, REINPLACE_CMD is what should be there.

> (This suggests that avoiding the Makefile and directly sed'ing the Config.pm
> and Config_heavy.pl files in /usr/local/lib/perl5/5.xx/... would not make a
> great overall test as it would miss the above problem.)
> 
> 
> (2) How to test the overall behavior?
> 
> My builds of perl5 would never have ${CC} == /nxb-bin/usr/bin/cc normally
> and if I tried having that path in the CC definition in my /etc/make.conf
> the build would normally fail for lack of such a path.
> 
> My original work around to pkg installing something that was broken was to
> build the lang/perl5 port instead of using "pkg install perl5". That had no
> "/nxb-bin/usr/bin/cc" involved and worked fine. (It still works fine with
> the patch to the perl5 Makefile involved.)
> 
> I expect that the /nxb-bin/usr/bin/cc comes from how things are built to
> make official packages (as done on the official servers). I do not know how
> to set up such a build.
> 
> The closest simple test idea that I've come up with would be to temporarily
> have /nxb-bin/usr/bin/cc be a symbolic link to /usr/bin/cc during a perl5.xx
> build in my environment and have /etc/make.conf's CC reference
> /nxb-bin/usr/bin/cc at that time. Then later change /etc/make.conf's CC to
> reference /usr/bin/cc and remove /nxb-bin/usr/bin/cc from the file system
> before seeing if the devel/p5-Locale-gettext build problem repeats.
> 
> Sound sufficient?

Yes, the /nxb-bin thing is there because of how the packages are built. It will
only be there if you installed the official package.
Making a symlink is a bad idea, it hides thing.
I don't really get all that you're saying though.

I'll just decide that my patch (with REINPLACE_CMD) works and commit it.

-- 
You are receiving this mail because:
You are on the CC list for the bug.