From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <legion@altlinux.ru>
X-Spam-Checker-Version: SpamAssassin 3.4.1 (2015-04-28) on
 sa.local.altlinux.org
X-Spam-Level: 
X-Spam-Status: No, score=-1.9 required=5.0 tests=BAYES_00 autolearn=ham
 autolearn_force=no version=3.4.1
Date: Mon, 21 Jan 2019 16:53:01 +0100
From: Alexey Gladkov <legion@altlinux.ru>
To: ALT Linux Team development discussions <devel@lists.altlinux.org>
Message-ID: <20190121155301.GC11968@Legion-PC.fortress>
References: <20190121120526.B3E9284402EC@gitery.altlinux.org>
 <20190121141648.GA14417@altlinux.org>
 <20190121153926.GC9715@imap.altlinux.org>
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
Content-Transfer-Encoding: 8bit
In-Reply-To: <20190121153926.GC9715@imap.altlinux.org>
Subject: Re: [devel] [SCM] packages/make-initrd: heads/sisyphus
X-BeenThere: devel@lists.altlinux.org
X-Mailman-Version: 2.1.12
Precedence: list
Reply-To: ALT Linux Team development discussions <devel@lists.altlinux.org>
List-Id: ALT Linux Team development discussions <devel.lists.altlinux.org>
List-Unsubscribe: <https://lists.altlinux.org/mailman/options/devel>,
 <mailto:devel-request@lists.altlinux.org?subject=unsubscribe>
List-Archive: <http://lists.altlinux.org/pipermail/devel>
List-Post: <mailto:devel@lists.altlinux.org>
List-Help: <mailto:devel-request@lists.altlinux.org?subject=help>
List-Subscribe: <https://lists.altlinux.org/mailman/listinfo/devel>,
 <mailto:devel-request@lists.altlinux.org?subject=subscribe>
X-List-Received-Date: Mon, 21 Jan 2019 15:53:06 -0000
Archived-At: <http://lore.altlinux.org/devel/20190121155301.GC11968@Legion-PC.fortress/>
List-Archive: <http://lore.altlinux.org/devel/>
List-Post: <mailto:devel@altlinux.org>

On Mon, Jan 21, 2019 at 06:39:26PM +0300, Michael Shigorin wrote:
> On Mon, Jan 21, 2019 at 05:16:48PM +0300, Dmitry V. Levin wrote:
> > >     - Add changes in "add-modules" to fix error with sha256 at i586 architecture
> > Когда вы пишете %changelog, вы пишете не commit message,
> > а текст, который увидят люди, которые не увидят весь commit
> > целиком.
> 
> Напомню ещё один момент про описания коммитов:
> документировать стоит _причину_, _намерение_ или
> ещё что-либо неочевидное из реализации, которая
> и так задокументирована кодом.
> 
> > Поэтому старайтесь, пожалуйста, писать %changelog так,
> > чтобы его можно было понять, не заглядывая в код.
> 
> ...в данном разе, возможно, хватило бы чего-то вроде
> 
> add-modules: fix sha256 on i586

Если мы говорим о форме, то да. Но это сообщение не отражает того, что
делается в коде. Никакого фикса нет. Есть простое игнорирование проблемы,
что не отражено в сообщении.

-- 
Rgrds, legion