Bug 218701

Summary: security/softhsm2: Fix build with SQLITE3 storage backend
Product: Ports & Packages Reporter: Sergey Akhmatov <sergey>
Component: Individual Port(s)Assignee: Danilo G. Baio <dbaio>
Status: Closed FIXED    
Severity: Affects Only Me CC: dbaio, jaap
Priority: --- Keywords: needs-qa
Version: LatestFlags: jaap: maintainer-feedback+
dbaio: merge-quarterly+
Hardware: Any   
OS: Any   
URL: https://reviews.freebsd.org/D11607
Bug Depends on: 220744    
Bug Blocks:    
Attachments:
Description Flags
Fix sqlite support koobs: maintainer-approval+

Description Sergey Akhmatov 2017-04-17 12:25:25 UTC
Created attachment 181839 [details]
Fix sqlite support

Dependency on sqlite3 in this port is actually useless: configure option --with-objectstore-backend-db is required to actually use sqlite3. Without it port isn't built with sqlite.

Proposed patch fixes build and makes SQLITE usage optional.
Comment 1 Jaap Akkerhuis 2017-04-18 21:23:13 UTC
Yes, it looks like these thins are a left over from a previous version.

I agree with the patch.
Comment 2 Danilo G. Baio freebsd_committer freebsd_triage 2017-07-15 14:25:38 UTC
Good catch
Comment 3 commit-hook freebsd_committer freebsd_triage 2017-07-17 15:19:07 UTC
A commit references this bug:

Author: dbaio
Date: Mon Jul 17 15:18:37 UTC 2017
New revision: 446078
URL: https://svnweb.freebsd.org/changeset/ports/446078

Log:
  security/softhsm2: Update to 2.3.0, Fix build with SQLITE3 storage backend

  Disable support for GOST if openssl-devel is used

  Add option SQLITE (optional) to fix build with SQLITE3 storage backend [1]

  While here:
    Add LICENSE_FILE
    Add USES= ssl (Q/A warning)

  PR:		220744
  PR:		218701 [1]
  Submitted by:	jaap@NLnetLabs.nl (maintainer)
  Submitted by:	sergey@akhmatov.ru [1]
  Approved by:	garga (mentor)
  MFH:		2017Q3
  Differential Revision:	https://reviews.freebsd.org/D11607

Changes:
  head/security/softhsm2/Makefile
  head/security/softhsm2/distinfo
  head/security/softhsm2/files/
  head/security/softhsm2/files/patch-src_lib_object__store_DB.h
  head/security/softhsm2/pkg-plist
Comment 4 commit-hook freebsd_committer freebsd_triage 2017-07-18 11:37:41 UTC
A commit references this bug:

Author: dbaio
Date: Tue Jul 18 11:37:28 UTC 2017
New revision: 446138
URL: https://svnweb.freebsd.org/changeset/ports/446138

Log:
  MFH: r446078 r446079

  security/softhsm2: Update to 2.3.0, Fix build with SQLITE3 storage backend

  Disable support for GOST if openssl-devel is used

  Add option SQLITE (optional) to fix build with SQLITE3 storage backend [1]

  While here:
    Add LICENSE_FILE
    Add USES= ssl (Q/A warning)

  PR:		220744
  PR:		218701 [1]
  Submitted by:	jaap@NLnetLabs.nl (maintainer)
  Submitted by:	sergey@akhmatov.ru [1]
  Approved by:	ports-secteam (junovitch), garga (mentor)
  Differential Revision:	https://reviews.freebsd.org/D11607

Changes:
_U  branches/2017Q3/
  branches/2017Q3/security/softhsm2/Makefile
  branches/2017Q3/security/softhsm2/distinfo
  branches/2017Q3/security/softhsm2/files/
  branches/2017Q3/security/softhsm2/pkg-plist
Comment 5 Danilo G. Baio freebsd_committer freebsd_triage 2017-07-18 11:40:57 UTC
Committed, thanks.