1 |
Le lundi 14 février 2011 à 11:42 +0100, Pacho Ramos a écrit : |
2 |
> El lun, 14-02-2011 a las 11:40 +0100, Pacho Ramos escribió: |
3 |
> > El lun, 14-02-2011 a las 11:37 +0100, Gilles Dartiguelongue escribió: |
4 |
> > > Le lundi 14 février 2011 à 11:34 +0100, Pacho Ramos a écrit : |
5 |
> > > > El lun, 14-02-2011 a las 11:24 +0100, Gilles Dartiguelongue escribió: |
6 |
> > > > > Le lundi 14 février 2011 à 11:15 +0100, Pacho Ramos a écrit : |
7 |
> > > > > > El dom, 13-02-2011 a las 18:03 +0100, Pacho Ramos escribió: |
8 |
> > > > > > > Hello |
9 |
> > > > > > > |
10 |
> > > > > > > Please see attached news item for reviewing as part of the fix for |
11 |
> > > > > > > http://bugs.gentoo.org/show_bug.cgi?id=346491 |
12 |
> > > > > > > |
13 |
> > > > > > > Thanks |
14 |
> > > > > > > |
15 |
> > > > > > |
16 |
> > > > > > This is an updated news item for trying to cover Ciaran and Matthew |
17 |
> > > > > > suggestions: |
18 |
> > > > > > 1. It doesn't ask people to use USE="*" |
19 |
> > > > > > 2. Before sending this, I would add all cameras to base/make.defaults as |
20 |
> > > > > > already done for similar cases (like alsa, lcd devices and others) |
21 |
> > > > > > |
22 |
> > > > > > Do you agree with this? |
23 |
> > > > > > |
24 |
> > > > > > Thanks :-) |
25 |
> > > > > |
26 |
> > > > > no, adding all cameras is most likely a waste of time for maintainers, |
27 |
> > > > > it is prone to errors when cameras get added/removed which is already |
28 |
> > > > > complicated enough to maintain. |
29 |
> > > > > |
30 |
> > > > |
31 |
> > > > What kind of errors will people see? As I have just tested, if I run: |
32 |
> > > > CAMERAS="bbhsgdd" emerge -pv media-libs/libgphoto2 |
33 |
> > > > |
34 |
> > > > I get no error (that would be equivalent to a camera that got removed on |
35 |
> > > > a bump). |
36 |
> > > |
37 |
> > > I was talking about human error while adding stuff to appropriate files. |
38 |
> > > It happens already often enough with just the IUSE_EXPAND feature. |
39 |
> > > |
40 |
> > |
41 |
> > Then, as ebuild uses EAPI2, why don't enable all of them in ebuild |
42 |
> > itself by default? |
43 |
> > for camera in ${IUSE_CAMERAS}; do |
44 |
> > - IUSE="${IUSE} cameras_${camera}" |
45 |
> > + IUSE="${IUSE} +cameras_${camera}" |
46 |
> > done |
47 |
> > |
48 |
> > Looks to work as expected here and wouldn't be prone to human errors |
49 |
> |
50 |
> Bleh, forget it, it is a headache to disable cameras then :-S |
51 |
|
52 |
and it didn't work when I tried it as I said in the bug report, I guess |
53 |
it depends on the version of portage and I haven't checked other PMs. |
54 |
|
55 |
-- |
56 |
Gilles Dartiguelongue <eva@g.o> |
57 |
Gentoo |