[125312] trunk/dports/multimedia/mediainfo

[Ryan Schmidt]

> On Sep 13, 2014, at 9:57 AM, Kurt Hindenburg <kurt.hindenburg at gmail.com> wrote:
> 
> 
> On 9/13/14, 1:41 AM, Ryan Schmidt wrote:
>>> On Sep 13, 2014, at 12:33 AM, khindenburg at macports.org wrote:
>>> 
>>> Revision
>>> 125312
>>> Author
>>> khindenburg at macports.org
>>> Date
>>> 2014-09-12 22:33:11 -0700 (Fri, 12 Sep 2014)
>>> Log Message
>>> 
>>> mediainfo: update to 0.7.70 - enable curl #42269
>>> Modified: trunk/dports/multimedia/mediainfo/files/patch-CLI_Compile.sh.diff (125311 => 125312)
>>> --- trunk/dports/multimedia/mediainfo/files/patch-CLI_Compile.sh.diff	2014-09-13 05:01:09 UTC (rev 125311)
>>> +++ trunk/dports/multimedia/mediainfo/files/patch-CLI_Compile.sh.diff	2014-09-13 05:33:11 UTC (rev 125312)
>>> @@ -14,7 +14,7 @@
>>>    test -e Makefile && rm Makefile
>>>    chmod u+x configure
>>>  - ./configure $*
>>> -+ ./configure "$@"
>>> ++ ./configure --with-libcurl "$@"
>>>    if test -e Makefile; then
>>>     make clean
>>>     Zen_Make
>> It looks like all arguments that get passed to CLI_Compile.sh get passed on to ./configure. So rather than patching the --with-libcurl argument into CLI_Compile.sh where it's harder to find, you might consider just adding it to the existing build.args in the Portfile.
>> 
> Hello,
>  ATM I don't see an obvious way to handle this. If you add it to the --prefix= line, you get "configure: WARNING: unrecognized options: --with-libcurl".

Oh sorry, I didn't see that the shell script is actually running three different configure scripts, and that this argument is only used for one of them. Never mind!