Bug 26396

Summary:	repocop-cronbuild-git: Возможность явно задать сборочный тег.
Product:	Sisyphus	Reporter:	solo <solo>
Component:	gear-cronbuild	Assignee:	viy <viy>
Status:	CLOSED FIXED	QA Contact:	qa-sisyphus
Severity:	normal
Priority:	P3	CC:	viy
Version:	unstable	Keywords:	patch
Hardware:	all
OS:	Linux
URL:	http://git.altlinux.org/people/solo/packages/?p=cronbuild-rear.git;a=commitdiff;h=b774254891872ca431de5012f6a97718889b3ae8;hp=2289b46aa67386788544961de7be5418054b4f24
Bug Depends on:	26393
Bug Blocks:

Description solo 2011-10-01 21:08:44 MSK

Для сборки moodle через cronbuild мне нужна возможность задавать сборочный тег явным образом. Предлагаемый вариант (см. бранч master в http://git.altlinux.org/people/solo/packages/cronbuild-rear.git):

  Если в файле .gear/cronbuild-options определены переменные cronbuild_tagname и cronbuild_tagmsg -- вызывать girar-nmu-helper-git-push-build с параметрами -m "$cronbuild_tagmsg" и -t "$cronbuild_tagname" (см. diff http://git.altlinux.org/people/solo/packages/?p=cronbuild-rear.git;a=commitdiff;h=b774254891872ca431de5012f6a97718889b3ae8).

PS: Я не разобрался на какой пакет вешать баги серверной части cronbuild`а.

Comment 1 viy 2011-10-01 22:47:34 MSK

файл .gear/cronbuild-options задумывался как read-only файл для робота,
в котором хранятся настройки администратора.

Как вам такой вариант: вместо 
<<< Если в файле .gear/cronbuild-options определены переменные cronbuild_tagname
и cronbuild_tagmsg 
---
>>> Если в каталоге .gear/ найдены исполняемые скрипты cronbuild-tagname
и cronbuild-tagmsg,
то -m будет равно `cronbuild-tagmsg`, а -t будет равно `cronbuild-tagname`.

Comment 2 solo 2011-10-01 23:08:09 MSK

(В ответ на комментарий №1)
> файл .gear/cronbuild-options задумывался как read-only файл для робота,
> в котором хранятся настройки администратора.

  А он и будет read-only, если использовать  конструкции вида:

cronbuild_tagname=`<скрипт формирования tagname>`

> 
> Как вам такой вариант: вместо 
> <<< Если в файле .gear/cronbuild-options определены переменные
> cronbuild_tagname
> и cronbuild_tagmsg 
> ---
> >>> Если в каталоге .gear/ найдены исполняемые скрипты cronbuild-tagname
> и cronbuild-tagmsg,
> то -m будет равно `cronbuild-tagmsg`, а -t будет равно `cronbuild-tagname`.

  Ваш вариант красивее. Сейчас доработаю патч.

Comment 3 solo 2011-10-01 23:56:03 MSK

(В ответ на комментарий №2)
> (В ответ на комментарий №1)
> > файл .gear/cronbuild-options задумывался как read-only файл для робота,
> > в котором хранятся настройки администратора.
> 
>   А он и будет read-only, если использовать  конструкции вида:
> 
> cronbuild_tagname=`<скрипт формирования tagname>`
> 
> > 
> > Как вам такой вариант: вместо 
> > <<< Если в файле .gear/cronbuild-options определены переменные
> > cronbuild_tagname
> > и cronbuild_tagmsg 
> > ---
> > >>> Если в каталоге .gear/ найдены исполняемые скрипты cronbuild-tagname
> > и cronbuild-tagmsg,
> > то -m будет равно `cronbuild-tagmsg`, а -t будет равно `cronbuild-tagname`.
> 
>   Ваш вариант красивее. Сейчас доработаю патч.

  См. http://git.altlinux.org/people/solo/packages/?p=cronbuild-rear.git;a=shortlog;h=refs/heads/master (diff с предыдущим вариантом: http://git.altlinux.org/people/solo/packages/?p=cronbuild-rear.git;a=commitdiff;h=4feea63e357012983c752619a5ef826912cb0d39).

Comment 4 viy 2011-10-02 00:53:40 MSK

я записал немножко компактнее,
http://git.altlinux.org/people/viy/packages/?p=cronbuild-rear.git;a=commit;h=f6d5b10616c82d7a77a2dc99ec30a685b0deea27
и выложил на песочницу.