Diego Biurrun <[email protected]> writes:

> On Fri, May 27, 2011 at 09:31:56PM -0700, Dave Yeo wrote:
>> On 05/27/11 11:55 am, Diego Biurrun wrote:
>>> On Thu, May 26, 2011 at 11:08:16PM -0700, Dave Yeo wrote:
>>>> On 05/26/11 05:53 am, Diego Biurrun wrote:
>>>>> Libav should now compile out-of-the-box on OS/2 and the
>>>>> information contained in the linked page is badly outdated.
>>>>
>>>> It compiles out of the box with the correct shell ((pdk)sh). Unluckily
>>>> eCS ships with ash which has a bug where $PWD is not initially set. This
>>>> results in everything going smoothly till the end when linking
>>>> ffmpeg.exe fails with 39 missing symbols due to bad -L options.
>>>
>>> Does adding
>>>
>>>    PWD=$(pwd)
>>>
>>> to the beginning of configure fix the issue?
>>
>> Yes, doing
>> --- a/configure
>> +++ b/configure
>> @@ -14,6 +14,7 @@ export LC_ALL
>>  # make sure we are running under a compatible shell
>>  # try to make this part work with most shells
>>
>> +PWD=$(pwd)
>>  try_exec(){
>>      echo "Trying shell $1"
>>      type "$1" > /dev/null 2>&1 && exec "$@"
>>
>> allows the build to finish
>
> OK, I will submit a patch along those lines.

Don't bother.  From the spec:

PWD
    Set by the shell and by the cd utility. [...] Assignments to this
    variable may be ignored. If an application sets or unsets the value
    of PWD, the behaviors of the cd and pwd utilities are unspecified.

We don't support broken shells, especially not on systems as arcane as
OS/2.  End of story.

I also have a better fix.

-- 
Måns Rullgård
[email protected]
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to