From: Andrey Savchenko <bircoph@altlinux.org> To: "devel@ where you _can_ ask" <devel-newbies@lists.altlinux.org> Subject: Re: [newbies] Дайджест списка рассылки devel-newbies; том 43, выпуск 2 Date: Sat, 20 Feb 2021 02:21:40 +0300 Message-ID: <20210220022140.b6b5a97c484e72c3480cb9f5@altlinux.org> (raw) In-Reply-To: <2a663b86-986b-8966-76b6-8e3e9ace9c49@basealt.ru> [-- Attachment #1: Type: text/plain, Size: 2514 bytes --] Добрый вечер! On Tue, 16 Feb 2021 21:47:47 +0300 Николай Бурыкин wrote: > > % > > Свойство патчей "отваливаться в случае изменений" - это важное преимущество, > > а вовсе не недостаток, как полагают многие. > > -- ldv in devel@ > > % > Только что осознал, что кроме дайджестов на адрес в домене basealt.ru > других писем из списка рассылки почему-то не получаю, поэтому пропустил > переписку на тему патчей. > Добавил еще один адрес, для отказоустойчивости. Думаю, следует посмотреть настройки почты: https://lists.altlinux.org/mailman/options/devel-newbies Там внизу страницы можно залогиниться и в настройках выставить Set Digest Mode -> Off. > Сейчас прочитал, и у меня возник вопрос. > А если применять подход с патчами, то как будет правильнее: > Формировать по принципу один патч на одно исправление, > или формировать патч из исправлений, решающих конкретную проблему? > Я подумал, что в моём случае я решаю проблему предупреждений при сборке, > и собрал их в отдельный патч warnings. Но вот теперь задумался на тему > правильности. Это дело вкуса. Однотипные исправления можно в один патч сливать. В то же время патчи лучше делать более-менее атомарными. Так что если там сильно разные предупреждения, то я бы сделал отдельные патчи на каждую группу. Но это не принципиальный момент, так что слишком заморачиваться не следует: лучше исправить проблему и идти дальше, чем застрять на том, как лучше оформить решение. Best regards, Andrew Savchenko [-- Attachment #2: Type: application/pgp-signature, Size: 833 bytes --]
parent reply other threads:[~2021-02-19 23:21 UTC|newest] Thread overview: expand[flat|nested] mbox.gz Atom feed [parent not found: <2a663b86-986b-8966-76b6-8e3e9ace9c49@basealt.ru>]
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=20210220022140.b6b5a97c484e72c3480cb9f5@altlinux.org \ --to=bircoph@altlinux.org \ --cc=devel-newbies@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
devel@ where you _can_ ask This inbox may be cloned and mirrored by anyone: git clone --mirror http://lore.altlinux.org/devel-newbies/0 devel-newbies/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-newbies devel-newbies/ http://lore.altlinux.org/devel-newbies \ devel-newbies@lists.altlinux.org devel-newbies@lists.altlinux.ru devel-newbies@lists.altlinux.com public-inbox-index devel-newbies Example config snippet for mirrors. Newsgroup available over NNTP: nntp://lore.altlinux.org/org.altlinux.lists.devel-newbies AGPL code for this site: git clone https://public-inbox.org/public-inbox.git