[Buildroot] [PATCH 3/9] firefox: sqlite dependency needs new compile-time options
Stefan Fröberg
stefan.froberg at petroprogram.com
Tue Sep 11 23:35:58 UTC 2012
Hello Arnout
12.9.2012 0:51, Arnout Vandecappelle kirjoitti:
> On 09/05/12 16:28, Stefan Fröberg wrote:
>>
>> Signed-off-by: Stefan Fröberg<stefan.froberg at petroprogram.com>
>
> Acked-by: Arnout Vandecappelle (Essensium/Mind) <arnout at mind.be>
>
> Small comments below.
>
>> ---
>> package/sqlite/Config.in | 34 ++++++++++++++++++++++++++++++++++
>> package/sqlite/sqlite.mk | 12 ++++++++++++
>> 2 files changed, 46 insertions(+), 0 deletions(-)
>>
>> diff --git a/package/sqlite/Config.in b/package/sqlite/Config.in
>> index 25aaa3f..75bce70 100644
>> --- a/package/sqlite/Config.in
>> +++ b/package/sqlite/Config.in
>> @@ -22,3 +22,37 @@ config BR2_PACKAGE_SQLITE_STAT3
>> planner that can help SQLite to choose a better query plan under
>> certain situations.
>>
>> +config BR2_PACKAGE_SQLITE_ENABLE_FTS3
>> + bool "Enables version 3 of the full-text search engine"
>
> "Full-text search engine (fts3)"
>
Okay
>> + depends on BR2_PACKAGE_SQLITE
>
> Instead of a "depends on" every time, just enclose all the options in an
> if block.
>
No problem
>> + help
>> + When this option is defined in the amalgamation
>> + (see http://www.sqlite.org/amalgamation.html), version 3 of
>> + the full-text search engine is added to the build automatically.
>
> Is it relevant whether it's an amalgamation or a split build?
>
Well, no ...
> I'd just write:
>
> Adds the full-text search engine (version 3) to the build.
>
Ok
>> +
>> +config BR2_PACKAGE_SQLITE_ENABLE_UNLOCK_NOTIFY
>> + bool "Enables sqlite3_unlock_notify() interface"
>
> "sqlite3_unlock_notify() interface"
>
>> + depends on BR2_PACKAGE_SQLITE
>> + help
>> + This option enables the sqlite3_unlock_notify() interface and its
>> + accosiated functionality. See the documentation
>
> associated
>
>> + http://www.sqlite.org/unlock_notify.html for additional
>> information.
>> +
>> +config BR2_PACKAGE_SQLITE_SECURE_DELETE
>> + bool "Sets the secure_delete pragma on by default"
>> + depends on BR2_PACKAGE_SQLITE
>> + help
>> + This compile-time option changes the default settings of the
>> + secure_delete pragma. When this option is not used, secure_delete
>> + defaults to off. When this option is present, secure_delete
>> defaults
>> + to on.
>> +
>> + The secure_delete setting causes deleted content to be
>> overwritten
>> + with zeros. There is a small performance penalty for this since
>> + additional I/O must occur. On the other hand, secure_delete can
>> + prevent sensitive information from lingering in unused parts
>> + of the database file after it has allegedly been deleted.
>> + See the documentation on the
>> + http://www.sqlite.org/pragma.html#pragma_secure_delete
>> + for additional information.
>
> Is this needed for firefox? (Not that it matters, mind you.)
>
Yes. It is needed. Otherwise firefox starts barfing.
I cook a new patch later tomorrow for this based on your suggestions and
fixes.
Thanks!
Stefan
> [snip]
>
> Regards,
> Arnout
More information about the buildroot
mailing list