* [devel] /usr/lib/rpm/pkgconfig.prov -- слишком строгая проверка на версию
@ 2010-06-30 12:42 Konstantin Pavlov
2010-06-30 12:50 ` Konstantin Pavlov
2010-06-30 12:52 ` Dmitry V. Levin
0 siblings, 2 replies; 7+ messages in thread
From: Konstantin Pavlov @ 2010-06-30 12:42 UTC (permalink / raw)
To: devel
[-- 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 --]
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [devel] /usr/lib/rpm/pkgconfig.prov -- слишком строгая проверка на версию
2010-06-30 12:42 [devel] /usr/lib/rpm/pkgconfig.prov -- слишком строгая проверка на версию Konstantin Pavlov
@ 2010-06-30 12:50 ` Konstantin Pavlov
2010-06-30 12:55 ` Dmitry V. Levin
2010-07-05 17:24 ` Alexey Tourbin
2010-06-30 12:52 ` Dmitry V. Levin
1 sibling, 2 replies; 7+ messages in thread
From: Konstantin Pavlov @ 2010-06-30 12:50 UTC (permalink / raw)
To: devel
[-- 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 --]
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [devel] /usr/lib/rpm/pkgconfig.prov -- слишком строгая проверка на версию
2010-06-30 12:42 [devel] /usr/lib/rpm/pkgconfig.prov -- слишком строгая проверка на версию Konstantin Pavlov
2010-06-30 12:50 ` Konstantin Pavlov
@ 2010-06-30 12:52 ` Dmitry V. Levin
1 sibling, 0 replies; 7+ messages in thread
From: Dmitry V. Levin @ 2010-06-30 12:52 UTC (permalink / raw)
To: ALT Devel discussion list
[-- 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 --]
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [devel] /usr/lib/rpm/pkgconfig.prov -- слишком строгая проверка на версию
2010-06-30 12:50 ` Konstantin Pavlov
@ 2010-06-30 12:55 ` Dmitry V. Levin
2010-06-30 13:00 ` Konstantin Pavlov
2010-07-05 17:24 ` Alexey Tourbin
1 sibling, 1 reply; 7+ messages in thread
From: Dmitry V. Levin @ 2010-06-30 12:55 UTC (permalink / raw)
To: ALT Devel discussion list
[-- 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 --]
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [devel] /usr/lib/rpm/pkgconfig.prov -- слишком строгая проверка на версию
2010-06-30 12:55 ` Dmitry V. Levin
@ 2010-06-30 13:00 ` Konstantin Pavlov
0 siblings, 0 replies; 7+ messages in thread
From: Konstantin Pavlov @ 2010-06-30 13:00 UTC (permalink / raw)
To: devel
[-- 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 --]
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [devel] /usr/lib/rpm/pkgconfig.prov -- слишком строгая проверка на версию
2010-06-30 12:50 ` Konstantin Pavlov
2010-06-30 12:55 ` Dmitry V. Levin
@ 2010-07-05 17:24 ` Alexey Tourbin
2010-07-05 18:02 ` Dmitry V. Levin
1 sibling, 1 reply; 7+ messages in thread
From: Alexey Tourbin @ 2010-07-05 17:24 UTC (permalink / raw)
To: devel
[-- 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 --]
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [devel] /usr/lib/rpm/pkgconfig.prov -- слишком строгая проверка на версию
2010-07-05 17:24 ` Alexey Tourbin
@ 2010-07-05 18:02 ` Dmitry V. Levin
0 siblings, 0 replies; 7+ messages in thread
From: Dmitry V. Levin @ 2010-07-05 18:02 UTC (permalink / raw)
To: ALT Devel discussion list
[-- 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 --]
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2010-07-05 18:02 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2010-06-30 12:42 [devel] /usr/lib/rpm/pkgconfig.prov -- слишком строгая проверка на версию Konstantin Pavlov
2010-06-30 12:50 ` Konstantin Pavlov
2010-06-30 12:55 ` Dmitry V. Levin
2010-06-30 13:00 ` Konstantin Pavlov
2010-07-05 17:24 ` Alexey Tourbin
2010-07-05 18:02 ` Dmitry V. Levin
2010-06-30 12:52 ` Dmitry V. Levin
ALT Linux Team development discussions
This inbox may be cloned and mirrored by anyone:
git clone --mirror http://lore.altlinux.org/devel/0 devel/git/0.git
# If you have public-inbox 1.1+ installed, you may
# initialize and index your mirror using the following commands:
public-inbox-init -V2 devel devel/ http://lore.altlinux.org/devel \
devel@altlinux.org devel@altlinux.ru devel@lists.altlinux.org devel@lists.altlinux.ru devel@linux.iplabs.ru mandrake-russian@linuxteam.iplabs.ru sisyphus@linuxteam.iplabs.ru
public-inbox-index devel
Example config snippet for mirrors.
Newsgroup available over NNTP:
nntp://lore.altlinux.org/org.altlinux.lists.devel
AGPL code for this site: git clone https://public-inbox.org/public-inbox.git