[FFmpeg-devel] [Patch] Alternative OS/2 patch

[Dave Yeo]

On 11/08/07 09:16 pm, Dave Yeo wrote:
> On 11/05/07 04:11 pm, Diego Biurrun wrote:
>> On Mon, Nov 05, 2007 at 08:04:12AM -0800, Dave Yeo wrote:
>>> On 11/04/07 04:29 pm, Diego Biurrun wrote:
>>>> On Sun, Nov 04, 2007 at 02:07:54PM -0800, Dave Yeo wrote:
>>>>> ...
> ...
>>
>>>>> --- configure    (revision 10924)
>>>>> +++ configure    (working copy)
>>>>> @@ -1210,6 +1212,33 @@
>>>>> +  os/2)
>>>>> +    SLIB_CREATE_DEF_CMD="echo LIBRARY \$(FULLNAME) INITINSTANCE 
>>>>> TERMINSTANCE > \$(FULLNAME).def; \
>>>>> +        echo DESCRIPTION '\"\$(SLIBNAME_WITH_VERSION)\"' >> 
>>>>> \$(FULLNAME).def; \
>>>>> +        echo PROTMODE >> \$(FULLNAME).def; \
>>>>> +        echo CODE PRELOAD MOVEABLE DISCARDABLE >> \$(FULLNAME).def; \
>>>>> +        echo DATA PRELOAD MOVEABLE MULTIPLE NONSHARED >> 
>>>>> \$(FULLNAME).def; \
>>>>> +        echo EXPORTS >> \$(FULLNAME).def; \
>>>>> +        emxexp -o \$(OBJS) >> \$(FULLNAME).def"
>>>> Try using 'cat << EOF', that should simplify things a lot.
>>> I've never seen cat used this way and not sure how that would work.
>>
>> Then it's high time :)
>>
>> Something like
>>
>> cat << EOF > filename
>> some
>> stuff
>> EOF
>>
>> should accomplish the above without all the nasty escaping.
>>
> 
> While now having more understanding of here-documents it seems to only
> be applicable to shell scripts and I can't see anyway to easily use it
> in a makefile. The command does have to be run by the makefile to expand
> the variables and we need the objects already built to extract the
> exports from them.
> Attached patch cleans up some of the nasty escaping by putting the 
> DESCRIPTION field into a separate variable. This has the extra advantage 
> that it makes it simpler to change the description. (adding things like 
> vendor or bldlevel info).
> Also makes config.mak more readable :)
> 
>> Diego
> Dave

Here is the patch updated for various configure updates. See new message 
about other minor fixes needed for OS/2
Dave
-------------- next part --------------
A non-text attachment was scrubbed...
Name: alt_os2.diff
Type: text/x-patch
Size: 3296 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20071123/cff9db4b/attachment.bin>