Bug 263907 - databases/sqlite3: Include sqldiff in default build
Summary: databases/sqlite3: Include sqldiff in default build
Status: Closed Overcome By Events
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Some People
Assignee: freebsd-ports-bugs (Nobody)
URL: https://www.sqlite.org/sqldiff.html
Keywords: feature, needs-patch, needs-qa
Depends on:
Blocks:
 
Reported: 2022-05-11 00:05 UTC by Tim Chase
Modified: 2022-05-23 15:01 UTC (History)
2 users (show)

See Also:
koobs: maintainer-feedback? (pavelivolkov)


Attachments
Patch to add sqldiff (1.06 KB, patch)
2022-05-20 10:56 UTC, Fernando Apesteguía
fernape: maintainer-approval? (pavelivolkov)
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
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