Project

General

Profile

Bug #6935

Fix SQLite3 minimum version to reflect actual interface requirements.

Added by Ben Morgan almost 6 years ago. Updated over 5 years ago.

Status:
Closed
Priority:
Low
Target version:
Start date:
09/05/2014
Due date:
% Done:

100%

Estimated time:
Spent time:
Duration:

Description

The top level CMakeLists.txt states a dependence on sqlite3 3.7.8:

https://cdcvs.fnal.gov/redmine/projects/fhicl-cpp/repository/revisions/master/entry/CMakeLists.txt#L42

whereas it's 3.8.5 in

https://cdcvs.fnal.gov/redmine/projects/fhicl-cpp/repository/revisions/master/entry/ups/product_deps#L9

I don't know which of these takes precedence, but certainly fhicl-cpp will fail to compile with sqlite3 < 3.7.15 due to usage of the sqlite3_errmsg interface.

History

#1 Updated by Ben Morgan almost 6 years ago

Whoops, that should be "usage of the sqlite3_errstr" interface....

#2 Updated by Christopher Green almost 6 years ago

  • Status changed from New to Accepted

The version as specified in the CMakeLists.txt file is a minimum version, used by CMake's find_package facility. It does not affect the version which is set up, but does place requirements on the version that is found, causing a failure at CMake time if the requirements are not met. As you correctly point out however, this minimum version should be updated to match the actual interface requirements. This will be fixed with the next version of fhicl-cpp that we release.

#3 Updated by Christopher Green almost 6 years ago

  • Status changed from Accepted to Resolved
  • Assignee set to Christopher Green
  • Target version set to 1.12.00
  • % Done changed from 0 to 100

SQLite3 minimum version updated as advised with c30cdde.

#4 Updated by Christopher Green almost 6 years ago

  • Subject changed from Sqlite3 version mismatch in CMakeLists.txt vs ups/product_deps to Fix SQLite3 minimum version to reflect actual interface requirements.

#5 Updated by Christopher Green over 5 years ago

  • Status changed from Resolved to Closed


Also available in: Atom PDF