Bug 26396 - repocop-cronbuild-git: Возможность явно задать сборочный тег.
: repocop-cronbuild-git: Возможность явно задать сборочный тег.
Status: CLOSED FIXED
: Sisyphus
(All bugs in Sisyphus/gear-cronbuild)
: unstable
: all Linux
: P3 normal
Assigned To:
:
: http://git.altlinux.org/people/solo/p...
: patch
: 26393
:
  Show dependency tree
 
Reported: 2011-10-01 21:08 by
Modified: 2011-10-02 00:53 (History)


Attachments


Note

You need to log in before you can comment on or make changes to this bug.


Description From 2011-10-01 21:08:44
Для сборки 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 From 2011-10-01 22:47:34 -------
файл .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 From 2011-10-01 23:08:09 -------
(В ответ на комментарий №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 From 2011-10-01 23:56:03 -------
(В ответ на комментарий №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 From 2011-10-02 00:53:40 -------
я записал немножко компактнее,
http://git.altlinux.org/people/viy/packages/?p=cronbuild-rear.git;a=commit;h=f6d5b10616c82d7a77a2dc99ec30a685b0deea27
и выложил на песочницу.