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

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

[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

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

[Anton Farygin]

Можно я тоже немного покритикую ?
Сам иногда тоже всякую фигню пишу в коммит-мессейдж. Но сообщение: 
"редактирую спек" не несёт совсем никакой информативности:

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

Re: [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

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

[Sergey Afonin]

On Thursday 05 April 2018, Grigory Ustinov wrote:

> Тут именно редактирование. В частности, апстрим переехал на cmake и
> надо было реализовать сборку другим способом. Не знаю, как у вас, а
> в нашем офисе принята идеалогия, что лучше максимально дифференцировать
> изменения на коммиты. Лучше чуть больше коммитов, чем чуть меньше.

Это, скорее, хорошо, чем плохо, да. Но комментарий а-ля "changed building
to cmake" звучал бы лучше.


-- 
С уважением, Сергей Афонин.

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

[Anton Farygin]

05.04.2018 15:28, Sergey Afonin пишет:
> On Thursday 05 April 2018, Grigory Ustinov wrote:
>
>> Тут именно редактирование. В частности, апстрим переехал на cmake и
>> надо было реализовать сборку другим способом. Не знаю, как у вас, а
>> в нашем офисе принята идеалогия, что лучше максимально дифференцировать
>> изменения на коммиты. Лучше чуть больше коммитов, чем чуть меньше.
> Это, скорее, хорошо, чем плохо, да. Но комментарий а-ля "changed building
> to cmake" звучал бы лучше.
тут скорее другая проблема - эти spec file edit не имеют никакого смысла 
в отрыве от новой версии.
Т.е. - build new version по сути сразу означает что спек и правила 
адаптированы под новую версию пакета, в которой поменялась система сборки.

Не делать же коммиты на каждую строчку.

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

[Michael Shigorin]

On Thu, Apr 05, 2018 at 01:49:56PM +0300, Grigory Ustinov wrote:
> Можно. Cleanup spec - это совершенно не то. Тут именно редактирование.

"Редактирование" -- это шибко обще, "ни о чём".

> В частности, апстрим переехал на cmake и надо было реализовать
> сборку другим способом.

А вот это уже полезная информация :)

- switched build to cmake

> Не знаю, как у вас, а в нашем офисе принята идеалогия, 

идеОлогия

> что лучше максимально дифференцировать изменения на коммиты.

Гм, для меня это новость -- сам воспринимаю commit или changeset
как группу логически связных изменений, переводящих репозиторий
исходных текстов из одного согласованного состояния в другое.

> Информативность коммита нужна, когда всё навалено в один коммит
> и сразу взглядом не получается определить, что же тут хотел
> сделать мейнтейнер. Я добиваюсь информативности разделением
> изменений по смыслу.

Гриш, по опыту commit message стоит писать так, как если бы ты
писал его для себя любимого через год -- когда напрочь забудешь,
что и зачем делал, а вывести из диффа это будет нетривиально.

Т.е. даже не _что_ сделано (это задокументировано в теле коммита),
а _зачем_ сделано.

> > Ну и я бы объединил это всё в два коммита - первый обновление 
> > исходников, второй - правка спека, правил сборки и патчей.

Порой объединяю правку спека и .gear/rules, но обычно разделяю.

-- 
 ---- WBR, Michael Shigorin / http://altlinux.org
  ------ http://opennet.ru / http://anna-news.info