all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* [bug#47640] [PATCH v2] Add sqlitebrowser.
@ 2021-04-07 11:09 Allan Adair
  2021-04-07 15:56 ` Nicolò Balzarotti
  2022-02-21 13:53 ` bug#47640: " Guillaume Le Vaillant
  0 siblings, 2 replies; 4+ messages in thread
From: Allan Adair @ 2021-04-07 11:09 UTC (permalink / raw)
  To: 47640; +Cc: Allan Adair

---
 gnu/packages/databases.scm | 28 ++++++++++++++++++++++++++++
 1 file changed, 28 insertions(+)

diff --git a/gnu/packages/databases.scm b/gnu/packages/databases.scm
index ce7afa82d0..ca80099254 100644
--- a/gnu/packages/databases.scm
+++ b/gnu/packages/databases.scm
@@ -48,6 +48,7 @@
 ;;; Copyright © 2020 Vincent Legoll <vincent.legoll@gmail.com>
 ;;; Copyright © 2021 Sharlatan Hellseher <sharlatanus@gmail.com>
 ;;; Copyright © 2021 Greg Hogan <code@greghogan.com>
+;;; Copyright © 2021 Allan Adair <allan@adair.no>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -3829,3 +3830,30 @@ PostreSQL, SQLite, ODBC and MySQL.")
      "FreeTDS is an implementation of the Tabular DataStream protocol, used for
 connecting to MS SQL and Sybase servers over TCP/IP.")
     (license license:lgpl2.0+)))
+
+(define-public sqlitebrowser
+  (package
+    (name "sqlitebrowser")
+    (version "3.12.1")
+    (source
+     (origin
+       (method git-fetch)
+       (uri (git-reference
+             (url "https://github.com/sqlitebrowser/sqlitebrowser")
+             (commit (string-append "v" version))))
+       (file-name (git-file-name name version))
+       (sha256
+        (base32
+         "0ray6cscx2qil1dfi7hmpijmq3kba49wn430ih1q4fkz9psjvrz1"))))
+    (build-system cmake-build-system)
+    (inputs
+     `(("qtbase" ,qtbase)
+       ("qttools" ,qttools)
+       ("sqlite" ,sqlite)))
+    (arguments
+     `(#:configure-flags '("-DENABLE_TESTING=ON")))
+    (home-page "https://sqlitebrowser.org/")
+    (synopsis "DB Browser for SQLite (DB4S) is a high quality, visual, open source tool to create, design, and edit database files compatible with SQLite.")
+    (description
+     "DB4S is for users and developers who want to create, search, and edit databases. DB4S uses a familiar spreadsheet-like interface, and complicated SQL commands do not have to be learned.")
+    (license (list license:gpl3+ license:mpl2.0))))
-- 
2.31.1





^ permalink raw reply related	[flat|nested] 4+ messages in thread

* [bug#47640] [PATCH v2] Add sqlitebrowser.
  2021-04-07 11:09 [bug#47640] [PATCH v2] Add sqlitebrowser Allan Adair
@ 2021-04-07 15:56 ` Nicolò Balzarotti
  2021-04-09 11:07   ` Allan Adair
  2022-02-21 13:53 ` bug#47640: " Guillaume Le Vaillant
  1 sibling, 1 reply; 4+ messages in thread
From: Nicolò Balzarotti @ 2021-04-07 15:56 UTC (permalink / raw)
  To: Allan Adair, 47640; +Cc: Allan Adair

Hi, thanks for the contribution!

A few comments below

Allan Adair <allan@adair.no> writes:

> +    (description
> +     "DB4S is for users and developers who want to create, search, and edit databases. DB4S uses a familiar spreadsheet-like interface, and complicated SQL commands do not have to be learned.")
I think the linter here will complain about the single space after the
period, and for the really long line.

Did you run ./pre-inst-env guix lint sqlitebrowser ?  This should tell
you if something is wrong

> +    (license (list license:gpl3+ license:mpl2.0))))
I see that there are bundled dependencies (under /libs), could you try
unbundle them (remove the lib folder with a source snippet) and build
using guix dependencies?  There are only four of them, two of which
(nlohmann-json-cpp and qscintilla) are already in guix.  Else it might
be needed to fix the licence.

There are the cmake flags FORCE_INTERNAL_QSCINTILLA,
FORCE_INTERNAL_QCUSTOMPLOT, FORCE_INTERNAL_QHEXEDIT which suggest this
should be possible.

Thanks, Nicolò




^ permalink raw reply	[flat|nested] 4+ messages in thread

* [bug#47640] [PATCH v2] Add sqlitebrowser.
  2021-04-07 15:56 ` Nicolò Balzarotti
@ 2021-04-09 11:07   ` Allan Adair
  0 siblings, 0 replies; 4+ messages in thread
From: Allan Adair @ 2021-04-09 11:07 UTC (permalink / raw)
  To: Nicolò Balzarotti, 47640

Hi Nicolò.

On 4/7/21 5:56 PM, Nicolò Balzarotti wrote:

> Did you run ./pre-inst-env guix lint sqlitebrowser ?  This should tell
> you if something is wrong 

Thanks! I really appreciate this guidance! It was an oversight on my part.

>> +    (license (list license:gpl3+ license:mpl2.0))))
> I see that there are bundled dependencies (under /libs), could you try
> unbundle them (remove the lib folder with a source snippet) and build
> using guix dependencies?  There are only four of them, two of which
> (nlohmann-json-cpp and qscintilla) are already in guix.  Else it might
> be needed to fix the licence.

I have been trying over the past couple of days with some difficulty. I 
will continue with some more effort next week, but since I am not an 
expert on the codebase (or cmake!) I may beg for fixing the licenses 
instead.





^ permalink raw reply	[flat|nested] 4+ messages in thread

* bug#47640: Add sqlitebrowser
  2021-04-07 11:09 [bug#47640] [PATCH v2] Add sqlitebrowser Allan Adair
  2021-04-07 15:56 ` Nicolò Balzarotti
@ 2022-02-21 13:53 ` Guillaume Le Vaillant
  1 sibling, 0 replies; 4+ messages in thread
From: Guillaume Le Vaillant @ 2022-02-21 13:53 UTC (permalink / raw)
  To: 34612-done, 47640-done

[-- Attachment #1: Type: text/plain, Size: 78 bytes --]

sqlitebrowser is in Guix (9183b8142b43082fee55a2730399243dea47ad0a).
Closing.

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 247 bytes --]

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2022-02-21 14:08 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-04-07 11:09 [bug#47640] [PATCH v2] Add sqlitebrowser Allan Adair
2021-04-07 15:56 ` Nicolò Balzarotti
2021-04-09 11:07   ` Allan Adair
2022-02-21 13:53 ` bug#47640: " Guillaume Le Vaillant

Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/guix.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.