From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <darktemplar@altlinux.org>
X-Spam-Checker-Version: SpamAssassin 3.4.1 (2015-04-28) on
 sa.local.altlinux.org
X-Spam-Level: 
X-Spam-Status: No, score=-2.9 required=5.0 tests=ALL_TRUSTED,BAYES_00
 autolearn=unavailable autolearn_force=no version=3.4.1
To: devel@lists.altlinux.org
References: <20191018094206.D4E578440208@gitery.altlinux.org>
 <20191023120056.GA14668@altlinux.org>
From: Aleksei Nikiforov <darktemplar@altlinux.org>
Message-ID: <018e56d2-8713-c85b-e0fa-789d6ba89d5a@altlinux.org>
Date: Thu, 24 Oct 2019 16:59:16 +0300
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101
 Thunderbird/60.8.0
MIME-Version: 1.0
In-Reply-To: <20191023120056.GA14668@altlinux.org>
Content-Type: text/plain; charset=koi8-r; format=flowed
Content-Language: ru
Content-Transfer-Encoding: 8bit
Subject: Re: [devel] [SCM] packages/apt: tags/0.5.15lorg2-alt76
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: Thu, 24 Oct 2019 13:59:22 -0000
Archived-At: <http://lore.altlinux.org/devel/018e56d2-8713-c85b-e0fa-789d6ba89d5a@altlinux.org/>
List-Archive: <http://lore.altlinux.org/devel/>
List-Post: <mailto:devel@altlinux.org>

23.10.2019 15:00, Dmitry V. Levin пишет:
>> commit 921b58bdef65edea9908d38bed16cb1e0fe5d0e3
>> Author: Aleksei Nikiforov <darktemplar@altlinux>
>> Date:   Fri Oct 11 16:47:43 2019 +0300
>>
>>      Check subsecond modification time for cached files
> 
> Поскольку без этого (или аналогичного этому) изменения тесты
> не проходят, оно обязательно должно быть сделано до включения тестов.
> Это основополагающее правило, следование которому позволяет
> бисектить регрессии.
> 
> По-хорошему, в commit message должно быть написано, что эта фича была
> реализована в первую очередь для того, чтобы включить тестирование.
> 

Хорошо, перемещу коммит. Но в первую очередь это багфикс проблемы, 
найденной тестами.

>> commit 31b29bd8c12dbf2a5fcee3724e9b170a73deec63
>> Author: Aleksei Nikiforov <darktemplar@altlinux>
>> Date:   Fri Oct 4 11:04:45 2019 +0300
>>
>>      Add initial integration tests
>>      
>>      Based on tests framework from Debian.
> 
> Для того, чтобы понимать, где наши баги, а где чужие, нужно как можно
> более тщательно различать чужие заимствования и наши модификации.
> 
> В данном случае это совсем просто сделать, достаточно сперва
> импортировать каталог test из Debian apt в неизменном виде,
> после чего вносить наши изменения.
> 
Хорошо, сделаю.