1 |
On Tue, Oct 5, 2010 at 7:41 PM, Donnie Berkholz <dberkholz@g.o> wrote: |
2 |
> On 15:52 Tue 05 Oct , "Paweł Hajdan, Jr." wrote: |
3 |
>> The meaning is identical in all those cases, and I think the number of |
4 |
>> packages may have hit the threshold for a global flag. |
5 |
>> |
6 |
>> However, we already have a very similar global USE flag: sqlite, which |
7 |
>> makes this a bit more tricky. The difference is very subtle: |
8 |
>> |
9 |
>> IUSE="sqlite" means "the sqlite support is optional; -sqlite means no |
10 |
>> sqlite support" |
11 |
>> |
12 |
>> IUSE="system-sqlite" means "we can use the bundled sqlite or the |
13 |
>> system one; -system-sqlite means we use the bundled one; both settings |
14 |
>> result in sqlite support" |
15 |
>> |
16 |
>> If we'd make system-sqlite a global USE flag, I'd suggest a |
17 |
>> description like "Use the system-wide dev-db/sqlite instead of bundled |
18 |
>> copy of sqlite." |
19 |
> |
20 |
> I think making this a global flag implicitly endorses building bundled |
21 |
> code as an OK thing to do. Why is it an option at all, local or |
22 |
> otherwise? |
23 |
> |
24 |
|
25 |
The reason is that with SQLITE_SECURE_DELETE, sqlite zeroes out |
26 |
portions that have been deleted causing a very noticeable drop in |
27 |
performance. This may be acceptable for browsers, but is rarely |
28 |
desirable system-wide. See |
29 |
https://bugzilla.mozilla.org/show_bug.cgi?id=546162 for details. |
30 |
|
31 |
To fix this problem sqlite upstream made a specific change allowing a |
32 |
#pragma to be used to define where secure-delete is required, avoiding |
33 |
the need to use secure-delete *everywhere*.[1] However, Mozilla |
34 |
upstream considers the two-line-change[2] an unacceptable level of |
35 |
maintenance burden, and refused to implement it[3]. |
36 |
|
37 |
I presume Chromium upstream has a similar stance on the issue, hence |
38 |
making the use-flag necessary for us since we (at least mozilla team) |
39 |
avoid adding patches that are not upstream yet. |
40 |
|
41 |
1. https://bugzilla.mozilla.org/show_bug.cgi?id=546162#c8 |
42 |
2. https://bugzilla.mozilla.org/show_bug.cgi?id=546162#c11 |
43 |
3. https://bugzilla.mozilla.org/show_bug.cgi?id=546162#c15 |
44 |
-- |
45 |
~Nirbheek Chauhan |
46 |
|
47 |
Gentoo GNOME+Mozilla Team |