[devel] unmatched pattern in gear

[devel] unmatched pattern in gear

[Anton Farygin]

Привет.

столкнулся тут с такой закорючкой:
$ gear -t 9294c0be4a8b887d08dd7d38c8c1259e46eafcf7 aaa.tar
gear: .gear-rules line 2: Unmatched pattern "*.patch" specified

Действительно.. был патч, потом кончился.

Может быть не стоит ругаться на эту тему ?

Достаточно warning вывести.. на мой взгляд.

Rgds,
Rider

Re: [devel] unmatched pattern in gear

[Dmitry V. Levin]

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

On Sat, May 13, 2006 at 01:59:59AM +0400, Anton Farygin wrote:
> Привет.
> 
> столкнулся тут с такой закорючкой:
> $ gear -t 9294c0be4a8b887d08dd7d38c8c1259e46eafcf7 aaa.tar
> gear: .gear-rules line 2: Unmatched pattern "*.patch" specified
> 
> Действительно.. был патч, потом кончился.
> 
> Может быть не стоит ругаться на эту тему ?
> 
> Достаточно warning вывести.. на мой взгляд.

Warning'и никто не читает, особенно если gear запускает робот.
Я думаю, надо добавить либо пачку директив копирования, которые будут
игнорировать Unmatched, либо модификатор этих директив, для этих же целей.
У меня тоже есть пакеты, в которых патчи иногда кончаются, так что мне
тоже интересно.


-- 
ldv

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

Re: [devel] unmatched pattern in gear

[Anton Farygin]

Dmitry V. Levin wrote:
> On Sat, May 13, 2006 at 01:59:59AM +0400, Anton Farygin wrote:
>> Привет.
>>
>> столкнулся тут с такой закорючкой:
>> $ gear -t 9294c0be4a8b887d08dd7d38c8c1259e46eafcf7 aaa.tar
>> gear: .gear-rules line 2: Unmatched pattern "*.patch" specified
>>
>> Действительно.. был патч, потом кончился.
>>
>> Может быть не стоит ругаться на эту тему ?
>>
>> Достаточно warning вывести.. на мой взгляд.
> 
> Warning'и никто не читает, особенно если gear запускает робот.
> Я думаю, надо добавить либо пачку директив копирования, которые будут
> игнорировать Unmatched, либо модификатор этих директив, для этих же целей.
> У меня тоже есть пакеты, в которых патчи иногда кончаются, так что мне
> тоже интересно.

да, модификатор тоже будет вполне приемлемо:

например copy(unmatched): *.patch
кстати, пока было не нужно, но на мой взгляд может быть интересно - 
директива exclude: <pattern>

пример, когда это будет нужно:

copy: files/*
exclude: files/TODO

или:
copy: *
exclude: *.spec TODO Makefile

с .spec сегодня Зерг нарвался, у него было так для KDE:
copy: *
spec: kde_что-то-там.spec

Оно работало, но как-то странно, до первого изменения спека.

в тарболл попадала какая-то не та версия спека. После изменения rules 
стало всё нормально.

Перечислять паттернами все файлы конечно можно, но уж очень муторно, 
особенно во всяких развесистых пакетах, где помимо *.patch встречаются 
ещё png, xpm, .sh, .init, .cpp и т.д. Вплоть до неких файлов с 
документацией и непонятным расширением (или отсутствием такового).

Rgds,
Rider

Re: [devel] unmatched pattern in gear

[Dmitry V. Levin]

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

On Sat, May 13, 2006 at 02:23:31AM +0400, Anton Farygin wrote:
> да, модификатор тоже будет вполне приемлемо:
> 
> например copy(unmatched): *.patch

Не слишком длинно?

> кстати, пока было не нужно, но на мой взгляд может быть интересно - 
> директива exclude: <pattern>
> 
> пример, когда это будет нужно:
> 
> copy: files/*
> exclude: files/TODO
> 
> или:
> copy: *
> exclude: *.spec TODO Makefile

Неочевидно, что exclude относится к copy.

> с .spec сегодня Зерг нарвался, у него было так для KDE:
> copy: *
> spec: kde_что-то-там.spec
> 
> Оно работало, но как-то странно, до первого изменения спека.
> 
> в тарболл попадала какая-то не та версия спека. После изменения rules 
> стало всё нормально.

spec копируется последним, так что я не понимаю, как оно могло не
работать.  Нужен пример, может бага закралась.

> Перечислять паттернами все файлы конечно можно, но уж очень муторно, 
> особенно во всяких развесистых пакетах, где помимо *.patch встречаются 
> ещё png, xpm, .sh, .init, .cpp и т.д. Вплоть до неких файлов с 
> документацией и непонятным расширением (или отсутствием такового).

Да, бывает всякое.


-- 
ldv

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

Re: [devel] unmatched pattern in gear

[Anton Farygin]

Dmitry V. Levin wrote:
> On Sat, May 13, 2006 at 02:23:31AM +0400, Anton Farygin wrote:
>> да, модификатор тоже будет вполне приемлемо:
>>
>> например copy(unmatched): *.patch
> 
> Не слишком длинно?

Зато понятно ;)

Можно конечно сократить:
copy?:

> 
>> кстати, пока было не нужно, но на мой взгляд может быть интересно - 
>> директива exclude: <pattern>
>>
>> пример, когда это будет нужно:
>>
>> copy: files/*
>> exclude: files/TODO
>>
>> или:
>> copy: *
>> exclude: *.spec TODO Makefile
> 
> Неочевидно, что exclude относится к copy.

А помимо copy нет директив, которым нужен exclude.


> 
>> с .spec сегодня Зерг нарвался, у него было так для KDE:
>> copy: *
>> spec: kde_что-то-там.spec
>>
>> Оно работало, но как-то странно, до первого изменения спека.
>>
>> в тарболл попадала какая-то не та версия спека. После изменения rules 
>> стало всё нормально.
> 
> spec копируется последним, так что я не понимаю, как оно могло не
> работать.  Нужен пример, может бага закралась.

Попробуем в понедельник воспроизвести - состояние ведь сохранилось ;)

Rgds,
Rider

Re: [devel] unmatched pattern in gear

[Dmitry V. Levin]

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

On Sat, May 13, 2006 at 09:08:25AM +0400, Anton Farygin wrote:
> Dmitry V. Levin wrote:
> > On Sat, May 13, 2006 at 02:23:31AM +0400, Anton Farygin wrote:
> >> да, модификатор тоже будет вполне приемлемо:
> >>
> >> например copy(unmatched): *.patch
> > 
> > Не слишком длинно?
> 
> Зато понятно ;)
> 
> Можно конечно сократить:
> copy?:

Мне copy?: больше нравится.  Никто не возражает против этого варианта?

> >> кстати, пока было не нужно, но на мой взгляд может быть интересно - 
> >> директива exclude: <pattern>
> >>
> >> пример, когда это будет нужно:
> >>
> >> copy: files/*
> >> exclude: files/TODO
> >>
> >> или:
> >> copy: *
> >> exclude: *.spec TODO Makefile
> > 
> > Неочевидно, что exclude относится к copy.
> 
> А помимо copy нет директив, которым нужен exclude.

copy - это семейство, к которому относится также и bzip2/gzip.


-- 
ldv

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

Re: [devel] unmatched pattern in gear

[Anton Farygin]

Dmitry V. Levin wrote:
> On Sat, May 13, 2006 at 09:08:25AM +0400, Anton Farygin wrote:
>> Dmitry V. Levin wrote:
>>> On Sat, May 13, 2006 at 02:23:31AM +0400, Anton Farygin wrote:
>>>> да, модификатор тоже будет вполне приемлемо:
>>>>
>>>> например copy(unmatched): *.patch
>>> Не слишком длинно?
>> Зато понятно ;)
>>
>> Можно конечно сократить:
>> copy?:
> 
> Мне copy?: больше нравится.  Никто не возражает против этого варианта?

Я не возражаю.

> 
>>>> кстати, пока было не нужно, но на мой взгляд может быть интересно - 
>>>> директива exclude: <pattern>
>>>>
>>>> пример, когда это будет нужно:
>>>>
>>>> copy: files/*
>>>> exclude: files/TODO
>>>>
>>>> или:
>>>> copy: *
>>>> exclude: *.spec TODO Makefile
>>> Неочевидно, что exclude относится к copy.
>> А помимо copy нет директив, которым нужен exclude.
> 
> copy - это семейство, к которому относится также и bzip2/gzip.

IMHO т.к. ещё никто кроме меня не воспользовался exclude, то думаю что 
синтаксис можно будет устаканить по мере необходимости.

А пока что понимать exclude для всего, что нужно не включать в src.rpm.

Rgds,
Rider

Re: [devel] unmatched pattern in gear

[Dmitry V. Levin]

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

On Sun, May 14, 2006 at 09:59:26AM +0400, Anton Farygin wrote:
> Dmitry V. Levin wrote:
> > On Sat, May 13, 2006 at 09:08:25AM +0400, Anton Farygin wrote:
> >> Dmitry V. Levin wrote:
> >>> On Sat, May 13, 2006 at 02:23:31AM +0400, Anton Farygin wrote:
> >>>> да, модификатор тоже будет вполне приемлемо:
> >>>>
> >>>> например copy(unmatched): *.patch
> >>> Не слишком длинно?
> >> Зато понятно ;)
> >>
> >> Можно конечно сократить:
> >> copy?:
> > 
> > Мне copy?: больше нравится.  Никто не возражает против этого варианта?
> 
> Я не возражаю.

55a2b3b3

> >>>> кстати, пока было не нужно, но на мой взгляд может быть интересно - 
> >>>> директива exclude: <pattern>
> >>>>
> >>>> пример, когда это будет нужно:
> >>>>
> >>>> copy: files/*
> >>>> exclude: files/TODO
> >>>>
> >>>> или:
> >>>> copy: *
> >>>> exclude: *.spec TODO Makefile
> >>> Неочевидно, что exclude относится к copy.
> >> А помимо copy нет директив, которым нужен exclude.
> > 
> > copy - это семейство, к которому относится также и bzip2/gzip.
> 
> IMHO т.к. ещё никто кроме меня не воспользовался exclude, то думаю что 
> синтаксис можно будет устаканить по мере необходимости.
> 
> А пока что понимать exclude для всего, что нужно не включать в src.rpm.

OK, поскольку правила обрабатываются построчно, то проще сделать именно
так:
exclude: pattern...
будет просто удалять созданное предыдущими правилами (во временном sourcedir)
согласно указанному шаблону.

Никто не против?


-- 
ldv

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

Re: [devel] unmatched pattern in gear

[Anton Farygin]

Dmitry V. Levin wrote:
> On Sun, May 14, 2006 at 09:59:26AM +0400, Anton Farygin wrote:
>> Dmitry V. Levin wrote:
>>> On Sat, May 13, 2006 at 09:08:25AM +0400, Anton Farygin wrote:
>>>> Dmitry V. Levin wrote:
>>>>> On Sat, May 13, 2006 at 02:23:31AM +0400, Anton Farygin wrote:
>>>>>> да, модификатор тоже будет вполне приемлемо:
>>>>>>
>>>>>> например copy(unmatched): *.patch
>>>>> Не слишком длинно?
>>>> Зато понятно ;)
>>>>
>>>> Можно конечно сократить:
>>>> copy?:
>>> Мне copy?: больше нравится.  Никто не возражает против этого варианта?
>> Я не возражаю.
> 
> 55a2b3b3
> 
>>>>>> кстати, пока было не нужно, но на мой взгляд может быть интересно - 
>>>>>> директива exclude: <pattern>
>>>>>>
>>>>>> пример, когда это будет нужно:
>>>>>>
>>>>>> copy: files/*
>>>>>> exclude: files/TODO
>>>>>>
>>>>>> или:
>>>>>> copy: *
>>>>>> exclude: *.spec TODO Makefile
>>>>> Неочевидно, что exclude относится к copy.
>>>> А помимо copy нет директив, которым нужен exclude.
>>> copy - это семейство, к которому относится также и bzip2/gzip.
>> IMHO т.к. ещё никто кроме меня не воспользовался exclude, то думаю что 
>> синтаксис можно будет устаканить по мере необходимости.
>>
>> А пока что понимать exclude для всего, что нужно не включать в src.rpm.
> 
> OK, поскольку правила обрабатываются построчно, то проще сделать именно
> так:
> exclude: pattern...
> будет просто удалять созданное предыдущими правилами (во временном sourcedir)
> согласно указанному шаблону.
> 
> Никто не против?

Меня устраивает. Только надо где-то задокументировать, что порядок 
exclude в файле rules важен.

Rgds,
Rider

Re: [devel] unmatched pattern in gear

[Kirill A. Shutemov]

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

On 02:30 Sat 13 May, Dmitry V. Levin wrote:
> On Sat, May 13, 2006 at 02:23:31AM +0400, Anton Farygin wrote:
> > да, модификатор тоже будет вполне приемлемо:
> > 
> > например copy(unmatched): *.patch
> 
> Не слишком длинно?
> 
> > кстати, пока было не нужно, но на мой взгляд может быть интересно - 
> > директива exclude: <pattern>
> > 
> > пример, когда это будет нужно:
> > 
> > copy: files/*
> > exclude: files/TODO
> > 
> > или:
> > copy: *
> > exclude: *.spec TODO Makefile
> 
> Неочевидно, что exclude относится к copy.
Как вариант: всё-же добавить возможность использовать регэкспы, а там есть
негативные проверки.

-- 
Kirill A. Shutemov				Belarus, Minsk
E-mail:	k.shutemov (AT) sam-solutions.net
JID: kas (AT) altlinux.org
ICQ: 152302675

> Ладно, скажу всем по секрету - неработоспособность hotplug'а с ядром 2.4
> была мной достигнута _специально_ путем почти 30 минутного изменения
> кода.
Да тоже мне секрет.  Ты погоди, лето не за горами ;-)
		-- mike in sisyphus@

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