devel@ where you _can_ ask
 help / color / mirror / Atom feed
* [newbies] Git best practices with patches
@ 2010-04-01  6:41 Eugene Prokopiev
  0 siblings, 0 replies; only message in thread
From: Eugene Prokopiev @ 2010-04-01  6:41 UTC (permalink / raw)
  To: devel@ where you _can_ ask

> Нормальная работа при написании (и тестировании) патча.
>
> Как это делаю я (в случае если использую репо в стиле gear-srpm):
>
> 1. Прямо в дереве с исходниками хакаю код.
> 2. Когда мне нужно проверить запускаю rpmbb (да-да, без всяких коммитов)
> 3. Когда результат мне понравился делаю так:
> git diff > %name.<something>.patch
> git checkout -f (чтобы убрать свои изменения из дерева)
> в spec дописываю этот патч
> если ранее ни одного патча не было, то пишу в .gear/rules: "copy: *.patch"
> 4. profit!

Схема в первоначальном виде сильно медленнее хаканья внутри сборочного
чрута - т.е. реально нужно больше телодвижений. И тут уже выгоды от
git diff не больше выгод от gendiff. Поэтому - раз уж мучиться - я
заставляю себя выносить патчи в отдельные бранчи. Но я очень пассивный
майнтейнер, потому могу себе такое позволить.

PS: подумал, что здесь этой дискуссии самое место

-- 
С уважением,
Прокопьев Евгений

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2010-04-01  6:41 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2010-04-01  6:41 [newbies] Git best practices with patches Eugene Prokopiev

devel@ where you _can_ ask

This inbox may be cloned and mirrored by anyone:

	git clone --mirror http://lore.altlinux.org/devel-newbies/0 devel-newbies/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-newbies devel-newbies/ http://lore.altlinux.org/devel-newbies \
		devel-newbies@lists.altlinux.org devel-newbies@lists.altlinux.ru devel-newbies@lists.altlinux.com
	public-inbox-index devel-newbies

Example config snippet for mirrors.
Newsgroup available over NNTP:
	nntp://lore.altlinux.org/org.altlinux.lists.devel-newbies


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git