Ticket #34 (closed defect: fixed)

Opened 3 years ago

Last modified 3 years ago

xulrunner is bundling libsqlite

Reported by: hicham Owned by: spot
Priority: major Milestone:
Component: Bundled Library Exception Version:
Keywords: Cc: gecko-maint@…, gholms
Blocked By: Blocking:

Description

xulrunner in Fedora 13 and up bundles sqlite due to mozilla forcing sqlite to be built with SQLITE_SECURE_DELETE. gecko-maint have chosen to use the bundled one as a temporary solution, but lasting for several releases doesn't make it temporary.

We have three options :

Change History

comment:1 Changed 3 years ago by gholms

  • Cc gholms added

comment:2 Changed 3 years ago by toshio

Please answer the questions given here https://fedoraproject.org/wiki/Packaging:No_Bundled_Libraries#Standard_questions

and then set meeting ticket. Until then, xulrunner does not have permission to bundle and must stop provided that it can/a patch exists/etc.

comment:3 Changed 3 years ago by toshio

s/meeting ticket/meeting keyword/

comment:4 Changed 3 years ago by spot

FWIW, sqlite has had this enabled in rawhide since September, so there is no reason that xulrunner cannot use the system copy.

  • Tue Sep 2 2010 Tom "spot" Callaway <tcallawa@…> - 3.7.0.1-2
  • enable SQLITE_SECURE_DELETE, SQLITE_ENABLE_UNLOCK_NOTIFY for firefox 4

I've also had it enabled in my f13 and f14 builds of xulrunner2, and there have been no issues linked to it.

comment:5 Changed 3 years ago by hicham

here is the situation :

  • In F-13 and F-14 xulrunner cannot use the system copy of sqlite since the required minimum version isn't satisfied ( >= 3.7.1 )
  • In rawhide gecko-maint said ( per a conversation on IRC ) that they don't care since the final sqlite version that will be used in released xulrunner2 isn't known yet

comment:6 Changed 3 years ago by spot

  • Owner set to spot
  • Status changed from new to assigned

Xulrunner (gecko-maint) must enable system sqlite in rawhide, and if bundling becomes necessary, to request permission with FPC in the future.

Voting (+1: 6, 0:0, -1: 1)

In addition, if an appropriate version of sqlite becomes available as an update for release branches, they must also use it in any future xulrunner updates to those release branches.

Voting (+1: 6, 0: 1, -1: 0)

comment:7 Changed 3 years ago by spot

  • Status changed from assigned to closed
  • Resolution set to fixed

xulrunner-2.0-0.14b8.fc15 uses system sqlite.

Note: See TracTickets for help on using tickets.