Bug 263907

Summary: databases/sqlite3: Include sqldiff in default build
Product: Ports & Packages Reporter: Tim Chase <freebsd>
Component: Individual Port(s)Assignee: freebsd-ports-bugs (Nobody) <ports-bugs>
Status: Closed Overcome By Events    
Severity: Affects Some People CC: fernape, pavelivolkov
Priority: --- Keywords: feature, needs-patch, needs-qa
Version: LatestFlags: koobs: maintainer-feedback? (pavelivolkov)
Hardware: Any   
OS: Any   
URL: https://www.sqlite.org/sqldiff.html
Attachments:
Description Flags
Patch to add sqldiff fernape: maintainer-approval? (pavelivolkov)

Description Tim Chase 2022-05-11 00:05:52 UTC 
According to the sqlite site,

  https://www.sqlite.org/sqldiff.html

`sqldiff` should be part of the sqlite.  However, I have sqlite on my FreeBSD box (13.0p8) yet it doesn't seem to include sqldiff, nor do I see sqldiff as its own package

    $ for p in sqlite{,3} ; do pkg info $p | head -3 ; done
    sqlite-2.8.17_5
    Name           : sqlite
    Version        : 2.8.17_5
    sqlite3-3.37.2,1
    Name           : sqlite3
    Version        : 3.37.2,1
    $ pkg search sqldiff || echo nope
    nope
    $ find /usr/local -name 'sqldiff*' 2>/dev/null # no results
    $ which sqldiff || echo nope
    nope

I'm not sure what's required to either include this as part of the sqlite package, or to make it its own stand-alone package, but I was encouraged to report it here:

  https://twitter.com/FreeBSDHelp/status/1524174819742212096

so here's a bug-tracker entry to do so.
Comment 1 Kubilay Kocak freebsd_committer freebsd_triage 2022-05-11 00:55:30 UTC 
Thank you for the report Tim!
Comment 2 Fernando Apesteguía freebsd_committer freebsd_triage 2022-05-20 10:56:03 UTC 
Created attachment 234054 [details]
Patch to add sqldiff

This patch will add sqldiff to the package. Some other tools (as sqlite3_analyzer) are added depending on the TCL option.

Some Linux distributions provide a sqlite3-tools package that provides all this tools separately. sqlite web page itself has a binary package for precisely this (https://sqlite.org/2022/sqlite-tools-linux-x86-3380500.zip)

I honestly would not bother to create just another package in this case. It makes sense to have sqldiff shipped with the normal package.
Comment 3 Pavel Volkov 2022-05-23 10:47:20 UTC 
(In reply to Fernando Apesteguía from comment #2)

Hello.
This patch has been taken into account when updating the port to the new version. (https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=264173)
Thank you.
Comment 4 Fernando Apesteguía freebsd_committer freebsd_triage 2022-05-23 15:01:02 UTC 
Done in bug #264173