Bug 26396

Summary: repocop-cronbuild-git: Возможность явно задать сборочный тег.
Product: Sisyphus Reporter: solo <solo>
Component: gear-cronbuildAssignee: viy <viy>
Status: CLOSED FIXED QA Contact: qa-sisyphus
Severity: normal    
Priority: P3 CC: viy
Version: unstableKeywords: 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
и выложил на песочницу.