ALT Linux Team development discussions
 help / color / mirror / Atom feed
* [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