| 1 |
On Sat, 10 Oct 2015 14:25:28 +0200 |
| 2 |
hasufell <hasufell@g.o> wrote: |
| 3 |
|
| 4 |
> On 10/10/2015 02:24 PM, Fabian Groffen wrote: |
| 5 |
> > On 10-10-2015 14:19:44 +0200, hasufell wrote: |
| 6 |
> >>> +RDEPEND=" |
| 7 |
> >>> + !libressl? ( dev-libs/openssl:0 ) |
| 8 |
> >>> + libressl? ( dev-libs/libressl ) |
| 9 |
> >>> + sys-libs/zlib |
| 10 |
> >>> + net-libs/http-parser |
| 11 |
> >> |
| 12 |
> >> Please order deps alphabetically (I know I added libressl without |
| 13 |
> >> reordering, but that was just to keep the diff as small as |
| 14 |
> >> possible). |
| 15 |
> > |
| 16 |
> > Is this a policy |
| 17 |
> |
| 18 |
> It is undocumented policy. |
| 19 |
|
| 20 |
|
| 21 |
It is useless style nitpick. Go maintain the whole tree if you want your |
| 22 |
style to be the only true one :) |
| 23 |
|
| 24 |
Sometimes it is indeed better to sort alphabetically, sometimes |
| 25 |
upstream's configure.ac/CMakeLists.txt/whatever ordering is better. |
| 26 |
Whatever minimizes chances of missing a dep change on version bump and |
| 27 |
accommodates best maintainer is policy... |
Archives