[Pkg-swan-devel] Bug#848890: Bug#848890: Revisiting "Ubuntu strongSwan changes"

Christian Ehrhardt christian.ehrhardt at canonical.com
Wed Dec 21 09:32:04 UTC 2016


Background to explain the reasoning behind the change 1/7:

Commits:
0733afe * d/rules: Reorganize to ease maintenance
be360c3 changelog: d/rules: Reorganize to ease maintenance
45adecc * d/control: Wrap and sort to ease maintenance
00bcdba changelog: d/control: Wrap and sort to ease maintenance

Background:
Diffs to the enabling / disabling options turned out to be very noisy.
That is due to multiple options being on one line, so a diff can easily be
misread.
Is it adding something, or removing - or actually chaning.

A common recommendation to avoid this issue is to have the entries
"one-per-line" (to avoid the diff noise) and sorted (to avoid duplicates by
adding e.g. at the bottom while being already at the top).

Ubuntu did have the diff of 0733afe this as a Delta to d/rules for quite a
while - and the change is purely cosmetical and should have no
functional impact, but helps maintenance.

The same applies to most files in debian/*. It is such a common need that
there is actually a Tool which does most of that for you "wrap and sort".
I realized while explaining we should add that right away as well,
so this 6872c7c is fixing up d/control by using that tool.
​
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.alioth.debian.org/pipermail/pkg-swan-devel/attachments/20161221/f8eeef9b/attachment.html>


More information about the Pkg-swan-devel mailing list