* Re: [devel] [#203516] EPERM errut.git=1.1.0-alt1 @ 2018-04-04 16:10 ` Grigory Ustinov 2018-04-05 5:11 ` Anton Farygin 0 siblings, 1 reply; 6+ messages in thread From: Grigory Ustinov @ 2018-04-04 16:10 UTC (permalink / raw) To: devel Переделал с патчем. 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 ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [devel] [#203516] EPERM errut.git=1.1.0-alt1 2018-04-04 16:10 ` [devel] [#203516] EPERM errut.git=1.1.0-alt1 Grigory Ustinov @ 2018-04-05 5:11 ` Anton Farygin 2018-04-05 10:49 ` Grigory Ustinov 0 siblings, 1 reply; 6+ messages in thread From: Anton Farygin @ 2018-04-05 5:11 UTC (permalink / raw) To: ALT Linux Team development discussions, Grigory Ustinov Можно я тоже немного покритикую ? Сам иногда тоже всякую фигню пишу в коммит-мессейдж. Но сообщение: "редактирую спек" не несёт совсем никакой информативности: 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 ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [devel] [#203516] EPERM errut.git=1.1.0-alt1 2018-04-05 5:11 ` Anton Farygin @ 2018-04-05 10:49 ` Grigory Ustinov 2018-04-05 12:28 ` Sergey Afonin 2018-04-07 9:23 ` Michael Shigorin 0 siblings, 2 replies; 6+ messages in thread From: Grigory Ustinov @ 2018-04-05 10:49 UTC (permalink / raw) To: devel Можно. 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 ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [devel] [#203516] EPERM errut.git=1.1.0-alt1 2018-04-05 10:49 ` Grigory Ustinov @ 2018-04-05 12:28 ` Sergey Afonin 2018-04-05 13:42 ` Anton Farygin 2018-04-07 9:23 ` Michael Shigorin 1 sibling, 1 reply; 6+ messages in thread From: Sergey Afonin @ 2018-04-05 12:28 UTC (permalink / raw) To: ALT Linux Team development discussions On Thursday 05 April 2018, Grigory Ustinov wrote: > Тут именно редактирование. В частности, апстрим переехал на cmake и > надо было реализовать сборку другим способом. Не знаю, как у вас, а > в нашем офисе принята идеалогия, что лучше максимально дифференцировать > изменения на коммиты. Лучше чуть больше коммитов, чем чуть меньше. Это, скорее, хорошо, чем плохо, да. Но комментарий а-ля "changed building to cmake" звучал бы лучше. -- С уважением, Сергей Афонин. ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [devel] [#203516] EPERM errut.git=1.1.0-alt1 2018-04-05 12:28 ` Sergey Afonin @ 2018-04-05 13:42 ` Anton Farygin 0 siblings, 0 replies; 6+ messages in thread From: Anton Farygin @ 2018-04-05 13:42 UTC (permalink / raw) To: ALT Linux Team development discussions, Sergey Afonin 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 по сути сразу означает что спек и правила адаптированы под новую версию пакета, в которой поменялась система сборки. Не делать же коммиты на каждую строчку. ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [devel] [#203516] EPERM errut.git=1.1.0-alt1 2018-04-05 10:49 ` Grigory Ustinov 2018-04-05 12:28 ` Sergey Afonin @ 2018-04-07 9:23 ` Michael Shigorin 1 sibling, 0 replies; 6+ messages in thread From: Michael Shigorin @ 2018-04-07 9:23 UTC (permalink / raw) To: devel 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 ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2018-04-07 9:23 UTC | newest] Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2018-04-04 16:10 ` [devel] [#203516] EPERM errut.git=1.1.0-alt1 Grigory Ustinov 2018-04-05 5:11 ` Anton Farygin 2018-04-05 10:49 ` Grigory Ustinov 2018-04-05 12:28 ` Sergey Afonin 2018-04-05 13:42 ` Anton Farygin 2018-04-07 9:23 ` Michael Shigorin
ALT Linux Team development discussions This inbox may be cloned and mirrored by anyone: git clone --mirror http://lore.altlinux.org/devel/0 devel/git/0.git # If you have public-inbox 1.1+ installed, you may # initialize and index your mirror using the following commands: public-inbox-init -V2 devel devel/ http://lore.altlinux.org/devel \ devel@altlinux.org devel@altlinux.ru devel@lists.altlinux.org devel@lists.altlinux.ru devel@linux.iplabs.ru mandrake-russian@linuxteam.iplabs.ru sisyphus@linuxteam.iplabs.ru public-inbox-index devel Example config snippet for mirrors. Newsgroup available over NNTP: nntp://lore.altlinux.org/org.altlinux.lists.devel AGPL code for this site: git clone https://public-inbox.org/public-inbox.git