Ubuntu
sqliteodbc package

On arm, at least, builds with libsqlite0 instead of libsqlite3

Bug #438450 reported by David Sugar
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
sqliteodbc (Debian)
Fix Released
Unknown
sqliteodbc (Ubuntu)
Fix Released
Wishlist
Unassigned

Bug Description

This can build an odbc driver for the original sqlite library, or the newer sqlite3. The control file uses libsqllite-dev, which pulls in libqllite0-dev, and the older odbc driver does not build on gcc 4.4. I believe this should pull in libsqlite3-dev instead. There may be a simple fix for the libsqlite0 based code, so I will see if I can fix that, but I am also tempted to change the control dependency. In any case, this is a place-holder bug for me until I research and resolve these two issues satisfactory.

Related branches

lp:ubuntu/lucid/sqliteodbc
Revision history for this message
David Sugar (dyfet-deactivatedaccount) wrote :

Correction, builds both sqlite0 and sqlite3. But older sqliteodbc (0, not 3) code makes assumption about converting char * to a va_list which is really invalid for gcc 4.4. In any case, I have a patch and a fix I am testing and will upload shortly.

Changed in sqliteodbc (Ubuntu):
status: New → In Progress
assignee: nobody → David Sugar (dyfet)
Revision history for this message
David Sugar (dyfet-deactivatedaccount) wrote :

Debdiff attached...

Revision history for this message
David Sugar (dyfet-deactivatedaccount) wrote :

This should be closable with the attached diff...

Fabrice Coutadeur (fabricesp)
Changed in sqliteodbc (Ubuntu):
assignee: David Sugar (dyfet) → nobody
status: In Progress → Confirmed
importance: Undecided → Wishlist
Revision history for this message
Fabrice Coutadeur (fabricesp) wrote :

Hi,

As explained by irc, please do not introduce a patch system if Debian does not have one, especially if the Debian maintainer has done changes directly in the source code, as its' the case here.

Also, please submit your patch to Debian to see if they would adopt it.

Thanks,

Fabrice

Revision history for this message
David Sugar (dyfet-deactivatedaccount) wrote :

I have re-done the debdiff patch without adding a patch system to the deb, and will also cross-submit a patch to debian...

Changed in sqliteodbc (Ubuntu):
assignee: nobody → David Sugar (dyfet)
Revision history for this message
David Sugar (dyfet-deactivatedaccount) wrote :

This is now in revu at http://revu.ubuntuwire.com/details.py?upid=6949

Revision history for this message
Scott Kitterman (kitterman) wrote :

Uploaded. Thank you for your contribution to Ubuntu. I did have to change the maintainer. Please keep this in mind for the next time.

Changed in sqliteodbc (Ubuntu):
status: Confirmed → Fix Committed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package sqliteodbc - 0.80-1ubuntu1

---------------
sqliteodbc (0.80-1ubuntu1) karmic; urgency=low

  * GCC 4.4 fix, architecture specific issue using a char * as fake va_list
    (LP: #438450).

 -- David Sugar <email address hidden> Mon, 28 Sep 2009 19:50:15 -0400

Changed in sqliteodbc (Ubuntu):
status: Fix Committed → Fix Released
Bug Watch Updater (bug-watch-updater)
Changed in sqliteodbc (Debian):
status: Unknown → New
Bug Watch Updater (bug-watch-updater)
Changed in sqliteodbc (Debian):
status: New → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.