Re: [devel] [SCM] packages/boinc: heads/master

Re: [devel] [SCM] packages/boinc: heads/master

[Alexey I. Froloff]

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

On Wed, Jul 28, 2010 at 03:49:59PM +0400, Paul Wolneykien wrote:
>  diff: debian/upstream:. patches/README:. name=README.patch
>  diff: debian/upstream:. patches/alt-service-actions:. name=alt-service-actions.patch
>  diff: debian/upstream:. patches/fix-boinc-client-service:. name=fix-boinc-client-service.patch
> -diff: debian/upstream:. patches/install-examples:. name=examples.patch
> +diff: debian/upstream:. patches/install-examples:. name=install-examples.patch
>  diff: debian/upstream:. patches/install-icons:. name=install-icons.patch
>  diff: debian/upstream:. patches/install-init-script:. name=install-init-script.patch
>  diff: debian/upstream:. patches/install-schedules:. name=install-schedules.patch
Генерить патчи таким образом - не самая хорошая идея.  При сборке
они могут тупо не приложиться, если где-то есть пересечения.

diff: debian/upstream:. patches/README:. name=README.patch
diff: patches/README:. patches/alt-service-actions:. name=alt-service-actions.patch
diff: patches/alt-service-actions:. patches/fix-boinc-client-service:. name=fix-boinc-client-service.patch

И соответственно мержить надо из debian/upstream в
patches/README, из patches/README в patches/alt-service-actions и
т.д.

-- 
Regards,    --
Sir Raorn.   --- http://thousandsofhate.blogspot.com/

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

Re: [devel] [SCM] packages/boinc: heads/master

[Ildar Mulyukov]

On 28.07.2010 17:55:25, Alexey I. Froloff wrote:
> On Wed, Jul 28, 2010 at 03:49:59PM +0400, Paul Wolneykien wrote:
> >  diff: debian/upstream:. patches/README:. name=README.patch
> >  diff: debian/upstream:. patches/alt-service-actions:.  
> name=alt-service-actions.patch
> >  diff: debian/upstream:. patches/fix-boinc-client-service:.  
> name=fix-boinc-client-service.patch
> > -diff: debian/upstream:. patches/install-examples:.  
> name=examples.patch
> > +diff: debian/upstream:. patches/install-examples:.  
> name=install-examples.patch
> >  diff: debian/upstream:. patches/install-icons:.  
> name=install-icons.patch
> >  diff: debian/upstream:. patches/install-init-script:.  
> name=install-init-script.patch
> >  diff: debian/upstream:. patches/install-schedules:.  
> name=install-schedules.patch
> Генерить патчи таким образом - не самая хорошая идея.  При сборке
> они могут тупо не приложиться, если где-то есть пересечения.

Прошу пояснений.
Если сборка не пройдёт, то в этом ничего плохого не будет. Автор просто  
обновит ветки с патчами.
Можете прокомментировать свою мысль более подробно?
С уважением,
	Ильдар
-- 
Ildar  Mulyukov,  free SW designer/programmer/packager
=========================================
email: ildar@altlinux.ru
Jabber: ildar.mulyukov@gmail.com
ICQ: 4334029
ALT Linux Sisyphus http://www.sisyphus.ru
=========================================

Re: [devel] [SCM] packages/boinc: heads/master

[Alexey I. Froloff]

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

On Mon, Aug 02, 2010 at 02:59:42PM +0600, Ildar Mulyukov wrote:
> Если сборка не пройдёт, то в этом ничего плохого не будет.
> Автор просто  обновит ветки с патчами.
> Можете прокомментировать свою мысль более подробно?
Если все ветки с патчами растут из одного upstream и в двух
ветках находятся конфликтующие изменения, то сгенерённые таким
образом патчи не приложатся.  И никакое "обновление веток" тут не
поможет.

P.S. Вот только не надо опять начинать теоретизировать.  Наличие
проблемы можно определить по появлению конфликта при мерже
очередного патча в master.

P.P.S. А вот например в /people/voins/packages/WindowMaker.git
таких проблем не возникает.

-- 
Regards,    --
Sir Raorn.   --- http://thousandsofhate.blogspot.com/

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

Re: [devel] [SCM] packages/boinc: heads/master

[Ildar Mulyukov]

On 02.08.2010 15:37:25, Alexey I. Froloff wrote:
> On Mon, Aug 02, 2010 at 02:59:42PM +0600, Ildar Mulyukov wrote:
> > Если сборка не пройдёт, то в этом ничего плохого не будет.
> > Автор просто  обновит ветки с патчами.
> > Можете прокомментировать свою мысль более подробно?
> Если все ветки с патчами растут из одного upstream и в двух
> ветках находятся конфликтующие изменения, то сгенерённые таким
> образом патчи не приложатся.  И никакое "обновление веток" тут не
> поможет.

мысль понял.
Согласен, что так стоит вести только ортогональные патчи.
В случае, если патчи итерферируют, то имеет смысл diff-ить один из них  
от другого, а не от апстрима.

> P.S. Вот только не надо опять начинать теоретизировать.

:-) оценил.

> Наличие проблемы можно определить по появлению конфликта при мерже
> очередного патча в master.

Как я понимаю, Павел именно так и делает.

> P.P.S. А вот например в /people/voins/packages/WindowMaker.git таких  
> проблем не возникает.

С уважением,
-- 
Ildar  Mulyukov,  free SW designer/programmer/packager
=========================================
email: ildar@altlinux.ru
Jabber: ildar.mulyukov@gmail.com
ICQ: 4334029
ALT Linux Sisyphus http://www.sisyphus.ru
=========================================