* [devel] Отдельный патч vs однострочник на sed/perl
@ 2009-03-04 11:30 Slava Semushin
2009-03-04 12:18 ` Led
2009-03-04 14:38 ` Konstantin Baev
0 siblings, 2 replies; 4+ messages in thread
From: Slava Semushin @ 2009-03-04 11:30 UTC (permalink / raw)
To: ALT Linux Team development discussions
4 марта 2009 г. 15:25 пользователь Damir Shayhutdinov <damir@altlinux> написал:
>> Можно патч сделать, а можно из спека sed-ом поправить, например. И
>> грань между тем когда и как лучше _не_ всегда очевидна.
>>
>> Всё приходит с опытом..
> Грань простая - если хочется отправить в апстрим - тогда патч. Если
> хочется "быстро и просто" - sed-ом.
Не совсем IMHO. Если в файле/ах исправляется одна строка, то вполне
однострочник подойдёт.
Когда desktop-file-validate находил в desktop-файлах несоответствия,
то я автору (xterm-а AFAIR) такой вот однострочник и отправлял. Потому
что извлекать из письма аттач, просматривать его и прикладывать
patch'ем в этом случае было дольше, чем просто выполнить тривиальную
команду для заменты строчки.
--
+ Slava Semushin | slava.semushin @ gmail.com
+ ALT Linux Team | php-coder @ altlinux.ru
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [devel] Отдельный патч vs однострочник на sed/perl
2009-03-04 11:30 [devel] Отдельный патч vs однострочник на sed/perl Slava Semushin
@ 2009-03-04 12:18 ` Led
2009-03-04 12:21 ` Andrey Rahmatullin
2009-03-04 14:38 ` Konstantin Baev
1 sibling, 1 reply; 4+ messages in thread
From: Led @ 2009-03-04 12:18 UTC (permalink / raw)
To: ALT Linux Team development discussions
On Wednesday 04 March 2009 13:30:55 Slava Semushin wrote:
> 4 марта 2009 г. 15:25 пользователь Damir Shayhutdinov <damir@altlinux>
написал:
> >> Можно патч сделать, а можно из спека sed-ом поправить, например. И
> >> грань между тем когда и как лучше _не_ всегда очевидна.
> >>
> >> Всё приходит с опытом..
> >
> > Грань простая - если хочется отправить в апстрим - тогда патч. Если
> > хочется "быстро и просто" - sed-ом.
>
> Не совсем IMHO. Если в файле/ах исправляется одна строка, то вполне
> однострочник подойдёт.
...А потом забыть про него и получать с какого-то времени кривой пакет. Потому
как "sed бла-бла-бла" практически всегда отработает тихо, а патч просто
отвалится в %setup, если он уже не подходит или неактуален.
--
Led
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [devel] Отдельный патч vs однострочник на sed/perl
2009-03-04 12:18 ` Led
@ 2009-03-04 12:21 ` Andrey Rahmatullin
0 siblings, 0 replies; 4+ messages in thread
From: Andrey Rahmatullin @ 2009-03-04 12:21 UTC (permalink / raw)
To: devel
On Wed, Mar 04, 2009 at 02:18:25PM +0200, Led wrote:
> ...А потом забыть про него и получать с какого-то времени кривой пакет. Потому
> как "sed бла-бла-бла" практически всегда отработает тихо, а патч просто
> отвалится в %setup, если он уже не подходит или неактуален.
Свойство патчей "отваливаться в случае изменений" - это важное
преимущество,
а вовсе не недостаток, как полагают многие.
-- ldv in devel@
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [devel] Отдельный патч vs однострочник на sed/perl
2009-03-04 11:30 [devel] Отдельный патч vs однострочник на sed/perl Slava Semushin
2009-03-04 12:18 ` Led
@ 2009-03-04 14:38 ` Konstantin Baev
1 sibling, 0 replies; 4+ messages in thread
From: Konstantin Baev @ 2009-03-04 14:38 UTC (permalink / raw)
To: ALT Linux Team development discussions
>>> Можно патч сделать, а можно из спека sed-ом поправить, например. И
>>> грань между тем когда и как лучше _не_ всегда очевидна.
>>>
... а можно не делать ни того, ни другого, а просто грубо править
исходники в
гит - так и быстрее и надежнее. В гите все изменения видны. Апстриму можно
давать ссылку на diff в веб-интерфейсе.
Конечно, в отдельном бранче должны лежать чистые исходники. В этот бранч
получаем обновления, смотрим изменения и мерджим их в рабочий бранч.
Можно, конечно, генерить патч(и) при помощи хитрых gear-rulez, но этому ещё
научиться надо.
--
Konstantin Baev
kipruss@altlinux.org
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2009-03-04 14:38 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2009-03-04 11:30 [devel] Отдельный патч vs однострочник на sed/perl Slava Semushin
2009-03-04 12:18 ` Led
2009-03-04 12:21 ` Andrey Rahmatullin
2009-03-04 14:38 ` Konstantin Baev
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