Created attachment 156272 [details]
- Remove unnecessary check for configure arguments
While upgrading lang/ocaml to latest 4.02.1, I got the following failure while upgrading this port:
===> Configuring for ocaml-nox11-4.02.1
[ERROR!]Arguments to this script look like '-prefix /foo/bar', not '-prefix=/foo/bar' (note the '=').
===> Script "configure" failed unexpectedly.
It was caused by "CPUTYPE?=core2" in /etc/make.conf
With this setting, $configure_options in configure was set to
-verbose -prefix /usr/local -cc "clang -O2 -pipe -march=core2 -fno-strict-aliasing" -as "as " -aspp clang -c -partialld ld -r -as as -no-graph
Then it fails to configure.
The check in configure line 88 to 90 is incomplete as it does not consider quotes.
*** Bug 199878 has been marked as a duplicate of this bug. ***
I filed a similar bug earlier, I marked it as duplicate of this one. The patch there is equivalent to the one here.
Hmm, that's strange, this seems like a bug a reported upstream a long time ago:
Maybe you can ask to reopen the issue?
(In reply to Michael Grünewald from comment #3)
reporting fixes upstream is maintainer's responsibility:
Anyway, reopening the bug upstream will not solve the problem which is present in the ports tree, so first some action is required in the ports tree to solve this.
You can approve the submitted patch or submit a better one if you have one.
After that you should contact the upstream and propose a fix there. You can ask help n this part, but you have to be aware that it's one of the maintainer's duties.
The patch is fine, we used to have something similar but it was removed as the
upstream bug has been closed.
Upstream bug http://caml.inria.fr/mantis/view.php?id=6628 is precisely me reporting
the bug upstream and asking to have this “argument validation” fixed, with a patch
pretty similar to what you propose. They want to keep a form of validation, so
you (Guido, sunpoet) will have to detail your use-case there.
(In reply to Michael Grünewald from comment #5)
Sorry, I clearly did not check your link properly.
I will followup to it.
> Sorry, I clearly did not check your link properly.
I have the same problem and I also have set CPUTYPE in make.conf. That did not pose a problem with the prior version of the port, so I first suspected a change in the port iteself (many patches were dropped). Notice that the upstream bug "reported a long time ago" is reported against the version which was just put in ports, 4.02.1, so maybe the problem was first introduced upstream at this version. Anyway, there is no point in re-opening the upstream bug unless the patch doesn't work. I have tested the patch from upstream, same as sunpoet attached. It fixes the build for me.
A commit references this bug:
Date: Sun May 3 13:35:35 UTC 2015
New revision: 385271
- Remove incomplete check for configure arguments to fix build when CONFIGURE_ARGS contains =
Submitted by: sunpoet (myself)
Approved by: Michael Grunewald <email@example.com> (maintainer)