C.Y.M wrote:
> Klaus Schmidinger wrote:
>> Oliver Endriss wrote:
>>> Klaus Schmidinger wrote:
>>>> VDR developer version 1.3.48 is now available at
>>>> ...
>>>> - Simplified the 'grep|awk|sed' command to retrieve the
>>>> VDR/APIVERSION to a single
>>>> 'sed' call.
>>> (1) My sed does not accept lines like this:
>>> | sed -ne '/define APIVERSION/ { s/^.*"\(.*\)".*$/\1/; p }' config.h
>>> | sed: -e expression #1, char 47: Extra characters after command
>>>
>>> This works:
>>> | sed -ne '/define APIVERSION/ { s/^.*"\(.*\)".*$/\1/p; }' config.h
>>> | 1.3.47
>>>
>>> (2) The newplugin script is broken:
>>> - references VDRVERSION, not APIVERSION
>>> - '\' must be escaped
>>> - sed problem as above
>> Just to make sure I don't make any stupid last minute error
>> here's the complete change that resulted from this.
>> Maybe somebody finds the time to look over this...
>>
>> Klaus
>>
>
> Im trying to test this from the console and I seem to have a problem with the
> following command (using sed-4.1.4-7 from debian sid):
>
> "sed -ne '/define APIVERSION/ { s/^.*"\(.*\)".*$$/\1/; p }' config.h"
>
> This returns nothing.
>
> "sed -ne '/define APIVERSION/ { s/^.*"\(.*\)".*$/\1/; p }' config.h"
>
> This returns:
>
> 1.3.48
>
> Is that right? Removing one of the "$" seems to fix it..
That's all right. The '$' needs to be doubled in the Makefile.
Klaus
