Bug 26396 - repocop-cronbuild-git: Возможность явно задать сборочный тег.
Summary: repocop-cronbuild-git: Возможность явно задать сборочный тег.
Status: CLOSED FIXED
Alias: None
Product: Sisyphus
Classification: Development
Component: gear-cronbuild (show other bugs)
Version: unstable
Hardware: all Linux
: P3 normal
Assignee: viy
QA Contact: qa-sisyphus
URL: http://git.altlinux.org/people/solo/p...
Keywords: patch
Depends on: 26393
Blocks:
  Show dependency tree
 
Reported: 2011-10-01 21:08 MSK by solo
Modified: 2011-10-02 00:53 MSK (History)
1 user (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
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
и выложил на песочницу.