| 1 |
On 02 Jun 2015 20:47, Michael Palimaka wrote: |
| 2 |
> On 02/06/15 17:04, Michał Górny wrote: |
| 3 |
> > Dnia 2015-06-02, o godz. 03:58:35 |
| 4 |
> > "Michael Sterrett (mr_bones_)" <mr_bones_@g.o> napisał(a): |
| 5 |
> >> -DEPEND="readline? ( sys-libs/readline ) |
| 6 |
> >> +DEPEND="readline? ( sys-libs/readline:0 ) |
| 7 |
> > |
| 8 |
> > This should be actually := (or :0=) for both deps since gnugo links to |
| 9 |
> > them. This also applies to your remaining 'warning silencing' commits. |
| 10 |
> |
| 11 |
> Why? Blindly adding the subslot dep is a bad idea. |
| 12 |
|
| 13 |
in this particular case, the subslot usage is what we want since we're |
| 14 |
compiling+linking against it. using readline:0 vs readline is still an |
| 15 |
improvement though. |
| 16 |
|
| 17 |
we also want a subslot on ncurses since we compile+link against it. |
| 18 |
|
| 19 |
i think it's pretty uncommon to use readline in a package and not want a |
| 20 |
subslot. your package would have to be doing something uncommon like |
| 21 |
dlopening it since the only thing readline provides is a library ... |
| 22 |
-mike |
Archives