[devel] [#203516] EPERM errut.git=1.1.0-alt1

[Grigory Ustinov]

Можно. Cleanup spec - это совершенно не то. Тут именно редактирование. В 
частности, апстрим переехал на cmake и надо было реализовать сборку 
другим способом. Не знаю, как у вас, а в нашем офисе принята идеалогия, 
что лучше максимально дифференцировать изменения на коммиты. Лучше чуть 
больше коммитов, чем чуть меньше. Информативность коммита нужна, когда 
всё навалено в один коммит и сразу взглядом не получается определить, 
что же тут хотел сделать мейнтейнер. Я добиваюсь информативности 
разделением изменений по смыслу.


On 05.04.2018 08:11, Anton Farygin wrote:
> Можно я тоже немного покритикую ?
> Сам иногда тоже всякую фигню пишу в коммит-мессейдж. Но сообщение: 
> "редактирую спек" не несёт совсем никакой информативности:
>
> http://git.altlinux.org/tasks/203516/gears/100/git?p=git;a=commitdiff;h=0500bc7f3f63a4d17a97ae1b5deee01b5af6fda5 
>
>
> Можно переделать на cleanup spec.
>
> Ну и я бы объединил это всё в два коммита - первый обновление 
> исходников, второй - правка спека, правил сборки и патчей.
>
> 04.04.2018 19:10, Grigory Ustinov пишет:
>> Переделал с патчем.
>>
>>
>> On 04.04.2018 19:09, Girar Builder awaiter robot wrote:
>>> http://git.altlinux.org/tasks/203516/logs/events.1.1.log
>>>
>>> 2018-Apr-04 16:06:32 :: task #203516 for sisyphus started by grenka:
>>> #100 build 1.1.0-alt1 from /people/grenka/packages/errut.git fetched 
>>> at 2018-Apr-04 16:06:32
>>> 2018-Apr-04 16:06:33 :: [i586] #100 errut.git 1.1.0-alt1: build start
>>> 2018-Apr-04 16:06:33 :: [x86_64] #100 errut.git 1.1.0-alt1: build start
>>> 2018-Apr-04 16:07:09 :: [i586] #100 errut.git 1.1.0-alt1: build OK
>>> 2018-Apr-04 16:07:27 :: [x86_64] #100 errut.git 1.1.0-alt1: build OK
>>> 2018-Apr-04 16:07:31 :: build check OK
>>> 2018-Apr-04 16:07:31 :: noarch check OK
>>> 2018-Apr-04 16:07:32 :: plan: src +1 -1 =18286, i586 +1 -1 =34177, 
>>> x86_64 +1 -1 =34185
>>> 2018-Apr-04 16:07:33 :: version check OK
>>> 2018-Apr-04 16:08:13 :: generated apt indices
>>> 2018-Apr-04 16:08:13 :: created next repo
>>> 2018-Apr-04 16:08:31 :: dependencies check OK
>>> 2018-Apr-04 16:08:33 :: ELF symbols check OK
>>> 2018-Apr-04 16:08:53 :: [i586] #100 errut: install check OK
>>> 2018-Apr-04 16:09:03 :: [x86_64] #100 errut: install check OK
>>> 2018-Apr-04 16:09:20 :: [x86_64-i586] generated apt indices
>>> 2018-Apr-04 16:09:20 :: [x86_64-i586] created next repo
>>> 2018-Apr-04 16:09:28 :: [x86_64-i586] dependencies check OK
>>> 2018-Apr-04 16:09:28 :: gears inheritance check OK
>>> 2018-Apr-04 16:09:28 :: srpm inheritance check OK
>>> girar-check-perms: access to errut DENIED for grenka: does not 
>>> belong to approved builders list: rt
>>> errut: Operation not permitted
>>> 2018-Apr-04 16:09:28 :: acl check FAILED
>>> 2018-Apr-04 16:09:28 :: task #203516 for sisyphus EPERM
>>
>> _______________________________________________
>> Devel mailing list
>> Devel на lists.altlinux.org
>> https://lists.altlinux.org/mailman/listinfo/devel
>
>
> _______________________________________________
> Devel mailing list
> Devel на lists.altlinux.org
> https://lists.altlinux.org/mailman/listinfo/devel