[MacPorts] #64179: p5-extutils-cbuilder uses -Wno-error=implicit-function-declaration
MacPorts
noreply at macports.org
Fri Dec 10 13:39:24 UTC 2021
#64179: p5-extutils-cbuilder uses -Wno-error=implicit-function-declaration
------------------------+----------------------------------
Reporter: ryandesign | Owner: (none)
Type: defect | Status: new
Priority: Normal | Milestone:
Component: ports | Version: 2.7.1
Keywords: arm64 | Port: p5-extutils-cbuilder
------------------------+----------------------------------
p5-extutils-cbuilder uses `-Wno-error=implicit-function-declaration` when
it builds things on behalf of other ports.
{{{
$ port -q contents p5.28-extutils-cbuilder | xargs grep implicit
/opt/local/lib/perl5/vendor_perl/5.28/ExtUtils/CBuilder/Platform/darwin.pm:
$cf->{ccflags} .= ($cf->{ccflags} ? ' ' : '').'-Wno-error=implicit-
function-declaration';
}}}
This was added to ExtUtils::CBuilder here:
https://github.com/Perl-Toolchain-Gang/ExtUtils-
CBuilder/commit/067d75adb44691e970bcdb39aaa250b115dd26aa
Don't we probably not want that?
We already had this issue with p5-extutils-makemaker in #61630. Upstream
added the flag, we added a patch to remove it and told upstream to remove
it, though so far they haven't, and the last comment in that issue to date
is the opinion that they like the fact that this flag will result in build
success on x86_64 and they accept the fact that it may cause misbehavior
on arm64, opining that any such misbehavior should be caught by the
module's tests. They also suggest maybe keeping the flag for x86_64 and
removing it for arm64.
Whatever we decide to do, we should probably be consistent between these
two modules. Either we should always remove the flag in cbuilder, as we
already do in makemaker, or we could change both of them to remove the
flag only when not arm64, though what would happen for an arm64/x86_64
universal build in that case is not clear to me, so it's probably safer to
always remove the flag.
We should also file a bug report with the ExtUtils::CBuilder folks, which
according to [https://github.com/Perl-Toolchain-Gang/ExtUtils-CBuilder the
project page] is the perl core developers.
--
Ticket URL: <https://trac.macports.org/ticket/64179>
MacPorts <https://www.macports.org/>
Ports system for macOS
More information about the macports-tickets
mailing list