From: Michael Shigorin <mike@altlinux.org> To: devel@lists.altlinux.org Subject: Re: [devel] [#203516] EPERM errut.git=1.1.0-alt1 Date: Sat, 7 Apr 2018 12:23:00 +0300 Message-ID: <20180407092300.GA3938@imap.altlinux.org> (raw) In-Reply-To: <4ba203cf-8c71-49b3-7201-644ecc57dc67@altlinux.org> 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
prev parent reply other threads:[~2018-04-07 9:23 UTC|newest] Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top 2018-04-04 16:10 ` 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 [this message]
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=20180407092300.GA3938@imap.altlinux.org \ --to=mike@altlinux.org \ --cc=devel@lists.altlinux.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: link
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