[devel] /usr/lib/rpm/pkgconfig.prov -- слишком строгая проверка на версию

[devel] /usr/lib/rpm/pkgconfig.prov -- слишком строгая проверка на версию

[Konstantin Pavlov]

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

Добрый день.

В /usr/lib/rpm/pkgconfig.prov используется слишком строгая проверка на
версию в .pc-файле:

case "$3" in
    *[!A-Za-z0-9_.+-]*)
		Fatal "$f: invalid pkg-config output: $line" ;;
    [0-9]*) ;;
        *) Fatal "$f: invalid pkg-config output: $line" ;;
esac

При этом, в man-странице не регламентируется формат строки Version, так же как
и в коде pkg-config нет подобных проверок.

Предлагаю убрать излишний ум из /usr/lib/rpm/pkgconfig.prov.

-- 
Don't over-comment.
            - The Elements of Programming Style (Kernighan & Plaugher)

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

Re: [devel] /usr/lib/rpm/pkgconfig.prov -- слишком строгая проверка на версию

[Konstantin Pavlov]

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

On Wed, Jun 30, 2010 at 04:42:21PM +0400, Konstantin Pavlov wrote:
> Добрый день.
> 
> В /usr/lib/rpm/pkgconfig.prov используется слишком строгая проверка на
> версию в .pc-файле:
> 
> case "$3" in
>     *[!A-Za-z0-9_.+-]*)
> 		Fatal "$f: invalid pkg-config output: $line" ;;
>     [0-9]*) ;;
>         *) Fatal "$f: invalid pkg-config output: $line" ;;
> esac
> 
> При этом, в man-странице не регламентируется формат строки Version, так же как
> и в коде pkg-config нет подобных проверок.
> 
> Предлагаю убрать излишний ум из /usr/lib/rpm/pkgconfig.prov.

К примеру,

thresh@snowwhite ~ $ /usr/lib/rpm/pkgconfig.prov ./librtmp.pc
pkgconfig.prov: ERROR: /home/thresh/librtmp.pc: invalid pkg-config output: librtmp = v2.2f

К тому же, вполне возможны версии типа "a", "b", "c" и т.п., разработчики
иногда попадаются и с Марса.

-- 
<frob> а слабо iconv-ом время из Unix-time в GMT конвертить? =)
<Pilot> frob: что, может???
<frob> если потренировать и пообещать банан, то думаю сможет

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

Re: [devel] /usr/lib/rpm/pkgconfig.prov -- слишком строгая проверка на версию

[Dmitry V. Levin]

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

Hi,

On Wed, Jun 30, 2010 at 04:42:21PM +0400, Konstantin Pavlov wrote:
> Добрый день.
> 
> В /usr/lib/rpm/pkgconfig.prov используется слишком строгая проверка на
> версию в .pc-файле:
> 
> case "$3" in
>     *[!A-Za-z0-9_.+-]*)
> 		Fatal "$f: invalid pkg-config output: $line" ;;
>     [0-9]*) ;;
>         *) Fatal "$f: invalid pkg-config output: $line" ;;
> esac
> 
> При этом, в man-странице не регламентируется формат строки Version, так же как
> и в коде pkg-config нет подобных проверок.
> 
> Предлагаю убрать излишний ум из /usr/lib/rpm/pkgconfig.prov.

Строгость проверок в pkgconfig.prov, скорее всего, связана с
необходимостью соблюсти формат зависимостей.

Есть ли пример .pc-файла, претендующего на валидность, который не проходит
проверку в pkgconfig.prov?


-- 
ldv

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

Re: [devel] /usr/lib/rpm/pkgconfig.prov -- слишком строгая проверка на версию

[Dmitry V. Levin]

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

On Wed, Jun 30, 2010 at 04:50:00PM +0400, Konstantin Pavlov wrote:
> On Wed, Jun 30, 2010 at 04:42:21PM +0400, Konstantin Pavlov wrote:
> > Добрый день.
> > 
> > В /usr/lib/rpm/pkgconfig.prov используется слишком строгая проверка на
> > версию в .pc-файле:
> > 
> > case "$3" in
> >     *[!A-Za-z0-9_.+-]*)
> > 		Fatal "$f: invalid pkg-config output: $line" ;;
> >     [0-9]*) ;;
> >         *) Fatal "$f: invalid pkg-config output: $line" ;;
> > esac
> > 
> > При этом, в man-странице не регламентируется формат строки Version, так же как
> > и в коде pkg-config нет подобных проверок.
> > 
> > Предлагаю убрать излишний ум из /usr/lib/rpm/pkgconfig.prov.
> 
> К примеру,
> 
> thresh@snowwhite ~ $ /usr/lib/rpm/pkgconfig.prov ./librtmp.pc
> pkgconfig.prov: ERROR: /home/thresh/librtmp.pc: invalid pkg-config output: librtmp = v2.2f
> 
> К тому же, вполне возможны версии типа "a", "b", "c" и т.п., разработчики
> иногда попадаются и с Марса.

Если разрешить, в дополнение к [0-9]*, ещё и [a-z]*, этого будет
достаточно?


-- 
ldv

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

Re: [devel] /usr/lib/rpm/pkgconfig.prov -- слишком строгая проверка на версию

[Konstantin Pavlov]

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

On Wed, Jun 30, 2010 at 04:55:38PM +0400, Dmitry V. Levin wrote:
> > К примеру,
> > 
> > thresh@snowwhite ~ $ /usr/lib/rpm/pkgconfig.prov ./librtmp.pc
> > pkgconfig.prov: ERROR: /home/thresh/librtmp.pc: invalid pkg-config output: librtmp = v2.2f
> > 
> > К тому же, вполне возможны версии типа "a", "b", "c" и т.п., разработчики
> > иногда попадаются и с Марса.
> 
> Если разрешить, в дополнение к [0-9]*, ещё и [a-z]*, этого будет
> достаточно?

Да, вполне.

-- 
> Я уже писал Вам, что мне не нужно присылать ничего, кроме файлов с
> расширением .patch и .diff
Даже если это новогодняя открытка? :-)
		-- mike in sisyphus@

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

Re: [devel] /usr/lib/rpm/pkgconfig.prov -- слишком строгая проверка на версию

[Alexey Tourbin]

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

On Wed, Jun 30, 2010 at 04:50:00PM +0400, Konstantin Pavlov wrote:
> thresh@snowwhite ~ $ /usr/lib/rpm/pkgconfig.prov ./librtmp.pc
> pkgconfig.prov: ERROR: /home/thresh/librtmp.pc: invalid pkg-config output: librtmp = v2.2f

In "v2.2f", "f" is probably okay, but "v" is bad idea - as you try
to compare v-version against non-v version, a funny thing will happen.

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

Re: [devel] /usr/lib/rpm/pkgconfig.prov -- слишком строгая проверка на версию

[Dmitry V. Levin]

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

On Mon, Jul 05, 2010 at 09:24:37PM +0400, Alexey Tourbin wrote:
> On Wed, Jun 30, 2010 at 04:50:00PM +0400, Konstantin Pavlov wrote:
> > thresh@snowwhite ~ $ /usr/lib/rpm/pkgconfig.prov ./librtmp.pc
> > pkgconfig.prov: ERROR: /home/thresh/librtmp.pc: invalid pkg-config output: librtmp = v2.2f
> 
> In "v2.2f", "f" is probably okay, but "v" is bad idea - as you try
> to compare v-version against non-v version, a funny thing will happen.

These pkg-config version "numbers" are not our choice, we just have to
support it.


-- 
ldv

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