[FFmpeg-devel] [libav-devel] [PATCH] [RFC] use a wrapper script to call MS link.exe to avoid mixing with /usr/bin/link.exe

[Steve Lhomme]

On Thu, Jul 23, 2015 at 7:02 PM, Derek Buitenhuis
<derek.buitenhuis at gmail.com> wrote:
> On 7/23/2015 5:41 PM, Steve Lhomme wrote:
>> --
>> favor link over link.exe in case some wrapper script already exists
>> fallback to "link" in the path if the one next to cl is not found
>> ---
>>  configure       | 2 +-
>>  tools/mslink.sh | 9 +++++++++
>
> Should be under compat.

Fixed in the next version.

>> diff --git a/tools/mslink.sh b/tools/mslink.sh
>> new file mode 100644
>
> Broken permissions.

Not sure how I can tweak that under Windows.

> Can do without the .sh extension (see for example, makedef).
>
>> index 0000000..6f0d98f
>> --- /dev/null
>> +++ b/tools/mslink.sh
>> @@ -0,0 +1,9 @@
>> +#!/bin/sh
>> +
>> +LINK_EXE_PATH=$(dirname "`command -v cl`")/link
>
> Be consistent with the usage of $().

What's the other alternative ? dirname is always used as $(dirname
$xyz) in configure.

>
> The idea itself seems reasonable.
>
> - Derek
> _______________________________________________
> libav-devel mailing list
> libav-devel at libav.org
> https://lists.libav.org/mailman/listinfo/libav-devel