unofficial mirror of guix-patches@gnu.org 
 help / color / mirror / Atom feed
* [bug#40288] [PATCH 1/1] gnu: Add lxqt-organizer.
@ 2020-03-28 16:41 Hamzeh Nasajpour
  2020-03-30  8:46 ` Efraim Flashner
  0 siblings, 1 reply; 3+ messages in thread
From: Hamzeh Nasajpour @ 2020-03-28 16:41 UTC (permalink / raw)
  To: 40288

* gnu/packages/lxqt.scm (lxqt-organizer): New variable.
---
 gnu/packages/lxqt.scm | 34 ++++++++++++++++++++++++++++++++++
 1 file changed, 34 insertions(+)

diff --git a/gnu/packages/lxqt.scm b/gnu/packages/lxqt.scm
index a440491633..7ef6ca51ce 100644
--- a/gnu/packages/lxqt.scm
+++ b/gnu/packages/lxqt.scm
@@ -1269,6 +1269,40 @@ QTermWidget.")
 easily publishing them on internet image hosting services.")
     (license license:gpl2+)))
 
+(define-public lxqt-organizer
+  (package
+    (name "lxqt-organizer")
+    (version "v0.003")
+    (source
+      (origin
+        (method git-fetch)
+        (uri (git-reference
+               (url (string-append "https://github.com/lxqt/organizer.git"))
+               (commit "5aa556aa0f0409e00fac5b63d9118ff228d2e840")
+               (recursive? #t)))
+        (file-name (git-file-name name version))
+        (sha256
+          (base32 "1czs9gca7aqsfcnxhi7nzq97pxbmhfgbfh00j7nl8cig0yxb6apv"))))
+    (build-system cmake-build-system)
+    (inputs
+      `(("kwindowsystem" ,kwindowsystem)
+        ("liblxqt" ,liblxqt)
+        ("libqtxdg" ,libqtxdg)
+        ("qtbase" ,qtbase)
+        ("qtsvg" ,qtsvg)
+        ;;("qtlinguist" ,qttools)
+        ("qtx11extras" ,qtx11extras)))
+    (native-inputs
+      `(("pkg-config" ,pkg-config)
+        ("lxqt-build-tools" ,lxqt-build-tools)
+        ("qttools" ,qttools)))
+    (arguments
+      '(#:tests? #f))
+    (home-page "https://lxqt.org/")
+    (synopsis "LXQt Organizer is a lightweight organizer for personal information management.")
+    (description
+     "This package provides a Qt-based application for managing meetings, events and other personal information.")
+    (license license:lgpl2.1+)))
 
 (define-public lxqt-archiver
   (package
-- 
2.25.1

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

* [bug#40288] [PATCH 1/1] gnu: Add lxqt-organizer.
  2020-03-28 16:41 [bug#40288] [PATCH 1/1] gnu: Add lxqt-organizer Hamzeh Nasajpour
@ 2020-03-30  8:46 ` Efraim Flashner
  2020-11-21  9:19   ` Christopher Baines
  0 siblings, 1 reply; 3+ messages in thread
From: Efraim Flashner @ 2020-03-30  8:46 UTC (permalink / raw)
  To: Hamzeh Nasajpour; +Cc: 40288

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

I didn't realize we had so much of lxqt packaged.

Some comments inline:

On Sat, Mar 28, 2020 at 09:11:06PM +0430, Hamzeh Nasajpour wrote:
> * gnu/packages/lxqt.scm (lxqt-organizer): New variable.
> ---
>  gnu/packages/lxqt.scm | 34 ++++++++++++++++++++++++++++++++++
>  1 file changed, 34 insertions(+)
> 
> diff --git a/gnu/packages/lxqt.scm b/gnu/packages/lxqt.scm
> index a440491633..7ef6ca51ce 100644
> --- a/gnu/packages/lxqt.scm
> +++ b/gnu/packages/lxqt.scm
> @@ -1269,6 +1269,40 @@ QTermWidget.")
>  easily publishing them on internet image hosting services.")
>      (license license:gpl2+)))
>  
> +(define-public lxqt-organizer
> +  (package
> +    (name "lxqt-organizer")
> +    (version "v0.003")

The author seems to be using this time to do some rapid development and
has already (at the time I checked) already bumped the version to
v0.6.0. Can you update to the most recent version?

> +    (source
> +      (origin
> +        (method git-fetch)
> +        (uri (git-reference
> +               (url (string-append "https://github.com/lxqt/organizer.git"))
> +               (commit "5aa556aa0f0409e00fac5b63d9118ff228d2e840")
> +               (recursive? #t)))

I was going to mention using the version string for the commit but it
seems they haven't pushed any tags to the repo. Can you add a comment
saying they haven't tagged any releases in git? Also, it doesn't look
like there are any submodules so you don't need 'recursive? #t'

> +        (file-name (git-file-name name version))
> +        (sha256
> +          (base32 "1czs9gca7aqsfcnxhi7nzq97pxbmhfgbfh00j7nl8cig0yxb6apv"))))
> +    (build-system cmake-build-system)
> +    (inputs
> +      `(("kwindowsystem" ,kwindowsystem)
> +        ("liblxqt" ,liblxqt)
> +        ("libqtxdg" ,libqtxdg)
> +        ("qtbase" ,qtbase)
> +        ("qtsvg" ,qtsvg)
> +        ;;("qtlinguist" ,qttools)
> +        ("qtx11extras" ,qtx11extras)))

guix gc --references /path/to/item doesn't list all the inputs, so many
of them can actually be moved to native-inputs.

> +    (native-inputs
> +      `(("pkg-config" ,pkg-config)
> +        ("lxqt-build-tools" ,lxqt-build-tools)
> +        ("qttools" ,qttools)))
> +    (arguments
> +      '(#:tests? #f))

I saw that there were no tests in the repo, please add a note.

> +    (home-page "https://lxqt.org/")

Is there a project-specific home-page? If not I'd recommend just using
the github repo as the home-page.

> +    (synopsis "LXQt Organizer is a lightweight organizer for personal information management.")
Perhaps: "LXQt desktop organizer" or "LXQt organizer for personal
information management" Actually the second sounds clunky. "LXQt
personal information management organizer"

> +    (description
> +     "This package provides a Qt-based application for managing meetings, events and other personal information.")
> +    (license license:lgpl2.1+)))
>  
>  (define-public lxqt-archiver
>    (package
> -- 
> 2.25.1
> 

In addition, in the desktop file (in %out/share/application/) there's a
second exec line which starts /usr/local/lxqt-organizer. Can you add a
phase to patch that?

Also, 'guix lint' complains about the length of two of the lines and
about the period in the synopsis.


-- 
Efraim Flashner   <efraim@flashner.co.il>   אפרים פלשנר
GPG key = A28B F40C 3E55 1372 662D  14F7 41AA E7DC CA3D 8351
Confidentiality cannot be guaranteed on emails sent or received unencrypted

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

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

* [bug#40288] [PATCH 1/1] gnu: Add lxqt-organizer.
  2020-03-30  8:46 ` Efraim Flashner
@ 2020-11-21  9:19   ` Christopher Baines
  0 siblings, 0 replies; 3+ messages in thread
From: Christopher Baines @ 2020-11-21  9:19 UTC (permalink / raw)
  To: Hamzeh Nasajpour; +Cc: 40288


It's been a few months, is this patch still relevant?

Thanks,

Chris




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

end of thread, other threads:[~2020-11-21  9:20 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-03-28 16:41 [bug#40288] [PATCH 1/1] gnu: Add lxqt-organizer Hamzeh Nasajpour
2020-03-30  8:46 ` Efraim Flashner
2020-11-21  9:19   ` Christopher Baines

unofficial mirror of guix-patches@gnu.org 

This inbox may be cloned and mirrored by anyone:

	git clone --mirror https://yhetil.org/guix-patches/1 guix-patches/git/1.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 guix-patches guix-patches/ https://yhetil.org/guix-patches \
		guix-patches@gnu.org
	public-inbox-index guix-patches

Example config snippet for mirrors.
Newsgroup available over NNTP:
	nntp://news.yhetil.org/yhetil.gnu.guix.patches


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git