[devel] fixup-pkgconfig падает на правильной конструкции в .pc

[devel] fixup-pkgconfig падает на правильной конструкции в .pc

[Pavlov Konstantin]

[-- Attachment #1: Type: text/plain, Size: 1014 bytes --]

Всем привет.

[builder@hasher pkgconfig]$ cat vlc-plugin.pc 
prefix=/usr
exec_prefix=/usr
libdir=/usr/lib
includedir=/usr/include

Name: VLC plugin API
Description: VLC media player plugin interface
Version: 0.9.0-test1
Cflags: -I${includedir} -I${includedir}/vlc/plugins \
    -D_FILE_OFFSET_BITS=64 \
    -D__USE_UNIX98 \
    -D_LARGEFILE64_SOURCE \
     \
    -D_REENTRANT \
    -D_THREAD_SAFE
Libs: -L${libdir} -lvlccore

builder@hasher pkgconfig]$ RPM_BUILD_ROOT="/usr/src/RPM/BUILD/vlc-0.9.0" sh
-exuf /usr/lib/rpm/fixup-pkgconfig vlc-plugin.pc
...
...
...
+ subst -p 's|^Cflags: .*|Cflags: -I${includedir}/vlc/plugins \|g' -- vlc-plugin.pc
subst: sed: -e expression #1, char 53: unterminated `s' command
subst: fork
+ return 1


С точки зрения pkg-config конструкция в Cflags вполне валидна.

Запатчить fixup-pkgconfig у меня не хватит душевных сил :(

-- 
The first half of our lives is ruined by our parents and the second half
by our children.
		-- Clarence Darrow

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 189 bytes --]

Re: [devel] fixup-pkgconfig падает на правильной конструкции в .pc

[Dmitry V. Levin]

[-- Attachment #1: Type: text/plain, Size: 1092 bytes --]

On Fri, Jul 04, 2008 at 12:19:09AM +0400, Pavlov Konstantin wrote:
> [builder@hasher pkgconfig]$ cat vlc-plugin.pc 
> prefix=/usr
> exec_prefix=/usr
> libdir=/usr/lib
> includedir=/usr/include
> 
> Name: VLC plugin API
> Description: VLC media player plugin interface
> Version: 0.9.0-test1
> Cflags: -I${includedir} -I${includedir}/vlc/plugins \
>     -D_FILE_OFFSET_BITS=64 \
>     -D__USE_UNIX98 \
>     -D_LARGEFILE64_SOURCE \
>      \
>     -D_REENTRANT \
>     -D_THREAD_SAFE
> Libs: -L${libdir} -lvlccore
> 
> builder@hasher pkgconfig]$ RPM_BUILD_ROOT="/usr/src/RPM/BUILD/vlc-0.9.0" sh
> -exuf /usr/lib/rpm/fixup-pkgconfig vlc-plugin.pc
> ...
> ...
> ...
> + subst -p 's|^Cflags: .*|Cflags: -I${includedir}/vlc/plugins \|g' -- vlc-plugin.pc
> subst: sed: -e expression #1, char 53: unterminated `s' command
> subst: fork
> + return 1
> 
> 
> С точки зрения pkg-config конструкция в Cflags вполне валидна.
> 
> Запатчить fixup-pkgconfig у меня не хватит душевных сил :(

Надо квотить, quote_sed_regexp() можно взять из /bin/shell-regexp


-- 
ldv

[-- Attachment #2: Type: application/pgp-signature, Size: 197 bytes --]

Re: [devel] fixup-pkgconfig падает на правильной конструкции в .pc

[Alexey I. Froloff]

[-- Attachment #1: Type: text/plain, Size: 267 bytes --]

* Dmitry V. Levin <ldv@> [080704 01:10]:
> > Cflags: -I${includedir} -I${includedir}/vlc/plugins \
> >     -D_FILE_OFFSET_BITS=64 \
> Надо квотить, quote_sed_regexp() можно взять из /bin/shell-regexp
Их бы неплохо ещё и склеивать...

-- 
Regards,
Sir Raorn.

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 189 bytes --]

Re: [devel] fixup-pkgconfig падает на правильной конструкции в .pc

[Dmitry V. Levin]

[-- Attachment #1: Type: text/plain, Size: 349 bytes --]

On Fri, Jul 04, 2008 at 02:09:49AM +0400, Alexey I. Froloff wrote:
> * Dmitry V. Levin <ldv@> [080704 01:10]:
> > > Cflags: -I${includedir} -I${includedir}/vlc/plugins \
> > >     -D_FILE_OFFSET_BITS=64 \
> > Надо квотить, quote_sed_regexp() можно взять из /bin/shell-regexp
> Их бы неплохо ещё и склеивать...

А что это даст?


-- 
ldv

[-- Attachment #2: Type: application/pgp-signature, Size: 197 bytes --]

Re: [devel] fixup-pkgconfig падает на правильной конструкции в .pc

[Alexey I. Froloff]

[-- Attachment #1: Type: text/plain, Size: 394 bytes --]

* Dmitry V. Levin <ldv@> [080704 02:20]:
> > > > Cflags: -I${includedir} -I${includedir}/vlc/plugins \
> > > >     -D_FILE_OFFSET_BITS=64 \
> > > Надо квотить, quote_sed_regexp() можно взять из /bin/shell-regexp
> > Их бы неплохо ещё и склеивать...
> А что это даст?
fixup-pkgconfig будет работать со всем содержимым поля, а не
только с его первой строкой.

-- 
Regards,
Sir Raorn.

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 189 bytes --]

Re: [devel] fixup-pkgconfig падает на правильной конструкции в .pc

[Pavlov Konstantin]

[-- Attachment #1: Type: text/plain, Size: 744 bytes --]

On Fri, Jul 04, 2008 at 01:06:01AM +0400, Dmitry V. Levin wrote:
> > С точки зрения pkg-config конструкция в Cflags вполне валидна.
> > 
> > Запатчить fixup-pkgconfig у меня не хватит душевных сил :(
> 
> Надо квотить, quote_sed_regexp() можно взять из /bin/shell-regexp

Втупую применил и сработало: 91e17e342 у меня в git.alt.

Кажется, также должно исправить #11437.

-- 
<Lost[work]> hiddenman: русскоязычное сообщество
<dottedmag> Lost[work]: а ты по-английски не шпрехаешь?
<hiddenman> Lost[work]: а де оно? :)
<gns> hiddenman: да вот же :)
<Lost[work]> dottedmag: на русском приятнее. Где еще можно увидеть халявный
             суппорт с чувством юмора...
<gns> Lost[work]: и с ба-а-альшими запасами хорошей травы %)

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 189 bytes --]

Re: [devel] fixup-pkgconfig падает на правильной конструкции в .pc

[Dmitry V. Levin]

[-- Attachment #1: Type: text/plain, Size: 594 bytes --]

On Mon, Jul 07, 2008 at 06:35:49PM +0400, Pavlov Konstantin wrote:
> On Fri, Jul 04, 2008 at 01:06:01AM +0400, Dmitry V. Levin wrote:
> > > С точки зрения pkg-config конструкция в Cflags вполне валидна.
> > > 
> > > Запатчить fixup-pkgconfig у меня не хватит душевных сил :(
> > 
> > Надо квотить, quote_sed_regexp() можно взять из /bin/shell-regexp
> 
> Втупую применил и сработало: 91e17e342 у меня в git.alt.

Первый hunk нормальный, а два других неправильные. :)
Квотить надо только $new_val.

> Кажется, также должно исправить #11437.

Очень даже может быть.


-- 
ldv

[-- Attachment #2: Type: application/pgp-signature, Size: 197 bytes --]