GNU bug report logs -
#60294
[PATCH] gnu: Add libxapp.
Previous Next
Reported by: "Wamm K. D" <jaft.r <at> outlook.com>
Date: Sat, 24 Dec 2022 07:41:01 UTC
Severity: normal
Tags: patch
Done: Liliana Marie Prikler <liliana.prikler <at> gmail.com>
Bug is archived. No further changes may be made.
To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 60294 in the body.
You can then email your comments to 60294 AT debbugs.gnu.org in the normal way.
Toggle the display of automated, internal messages from the tracker.
Report forwarded
to
guix-patches <at> gnu.org
:
bug#60294
; Package
guix-patches
.
(Sat, 24 Dec 2022 07:41:02 GMT)
Full text and
rfc822 format available.
Acknowledgement sent
to
"Wamm K. D" <jaft.r <at> outlook.com>
:
New bug report received and forwarded. Copy sent to
guix-patches <at> gnu.org
.
(Sat, 24 Dec 2022 07:41:02 GMT)
Full text and
rfc822 format available.
Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):
* gnu/packages/cinnamon.scm (libxapp): New variable.
---
I wasn't sure where to put this as, while definitely used by Cinnamon,
I think this is used by Mate, as well (and possibly XFCE?). I don't
know if a mint.scm file might make any sense as it's quite literally
under the Linux Mint repo. account? If it would be better to place
this elsewhere, just let me know and I'll adjust.
gnu/packages/cinnamon.scm | 72 +++++++++++++++++++++++++++++++++++++++
1 file changed, 72 insertions(+)
diff --git a/gnu/packages/cinnamon.scm b/gnu/packages/cinnamon.scm
index fe33e797e4..c1f258c18d 100644
--- a/gnu/packages/cinnamon.scm
+++ b/gnu/packages/cinnamon.scm
@@ -26,6 +26,7 @@ (define-module (gnu packages cinnamon)
#:use-module (guix utils)
#:use-module (guix build-system gnu)
#:use-module (gnu packages)
+ #:use-module (gnu packages admin)
#:use-module (gnu packages autotools)
#:use-module (gnu packages freedesktop)
#:use-module (gnu packages gettext)
@@ -37,6 +38,77 @@ (define-module (gnu packages cinnamon)
#:use-module (gnu packages python)
#:use-module (gnu packages xorg))
+(define-public libxapp
+ (package
+ (name "libxapp")
+ (version "2.4.2")
+ (source (origin
+ (method git-fetch)
+ (uri (git-reference
+ (url "https://github.com/linuxmint/xapp/")
+ (commit version)))
+ (sha256
+ (base32
+ "0cy9g0zqcbx9zscc9qavqmghfyfb8244cg299llv1ha8n6mpxl3s"))))
+ (build-system meson-build-system)
+ (arguments
+ (list
+ #:phases
+ #~(modify-phases %standard-phases
+ (add-before 'configure 'set-gtk-module-path
+ (lambda* (#:key inputs outputs #:allow-other-keys)
+ (substitute* "libxapp/meson.build"
+ (("gtk3_dep\\.get_pkgconfig_variable[(]'libdir'[)]")
+ (string-append "'" (assoc-ref outputs "out") "/lib'")))
+
+ (substitute* "pygobject/meson.build"
+ (("get_option[(]'py-overrides-dir'[)]")
+ (string-append "'"
+ (assoc-ref outputs "out")
+ "/lib/python"
+ #$(version-major+minor (package-version python))
+ "/site-packages/gi/overrides'")))
+
+ (substitute* "scripts/pastebin"
+ (("'nc'") (string-append "'"
+ (assoc-ref inputs "netcat")
+ "/bin/nc'")))
+
+ (substitute* "scripts/upload-system-info"
+ (("'inxi'") (string-append "'"
+ (assoc-ref inputs "inxi-minimal")
+ "/bin/inxi'"))
+ (("'/usr/bin/pastebin'") (string-append "'"
+ (assoc-ref outputs "out")
+ "/bin/pastebin'"))
+ (("'xdg-open'") (string-append "'"
+ (assoc-ref inputs "xdg-utils")
+ "/bin/xdg-open'"))))))))
+ (inputs
+ (list dbus
+ glib ; for gio
+ gtk+
+ inxi-minimal ; used by upload-system-info
+ libdbusmenu
+ libgnomekbd
+ netcat ; used by pastebin
+ xdg-utils ; used by upload-system-info
+ ))
+ (native-inputs
+ (list gettext-minimal
+ `(,glib "bin") ; for glib-mkenums
+ gobject-introspection
+ pkg-config
+ python
+ python-pygobject
+ vala))
+ (home-page "https://github.com/linuxmint/xapp")
+ (synopsis "Cross-desktop libraries and common resources for X-apps")
+ (description
+ "The components which are common to multiple GTK desktop environments
+(Cinnamon, MATE and Xfce) and required to implement cross-DE solutions.")
+ (license license:lgpl3)))
+
(define-public cinnamon-desktop
(package
(name "cinnamon-desktop")
--
2.38.1
Information forwarded
to
guix-patches <at> gnu.org
:
bug#60294
; Package
guix-patches
.
(Sat, 24 Dec 2022 10:37:01 GMT)
Full text and
rfc822 format available.
Message #8 received at 60294 <at> debbugs.gnu.org (full text, mbox):
Am Samstag, dem 24.12.2022 um 01:30 -0600 schrieb Wamm K. D:
> * gnu/packages/cinnamon.scm (libxapp): New variable.
> ---
> I wasn't sure where to put this as, while definitely used by
> Cinnamon,
> I think this is used by Mate, as well (and possibly XFCE?). I don't
> know if a mint.scm file might make any sense as it's quite literally
> under the Linux Mint repo. account? If it would be better to place
> this elsewhere, just let me know and I'll adjust.
I think it's in the right location.
>
> gnu/packages/cinnamon.scm | 72
> +++++++++++++++++++++++++++++++++++++++
> 1 file changed, 72 insertions(+)
>
> diff --git a/gnu/packages/cinnamon.scm b/gnu/packages/cinnamon.scm
> index fe33e797e4..c1f258c18d 100644
> --- a/gnu/packages/cinnamon.scm
> +++ b/gnu/packages/cinnamon.scm
> @@ -26,6 +26,7 @@ (define-module (gnu packages cinnamon)
> #:use-module (guix utils)
> #:use-module (guix build-system gnu)
> #:use-module (gnu packages)
> + #:use-module (gnu packages admin)
> #:use-module (gnu packages autotools)
> #:use-module (gnu packages freedesktop)
> #:use-module (gnu packages gettext)
> @@ -37,6 +38,77 @@ (define-module (gnu packages cinnamon)
> #:use-module (gnu packages python)
> #:use-module (gnu packages xorg))
>
> +(define-public libxapp
> + (package
> + (name "libxapp")
> + (version "2.4.2")
> + (source (origin
> + (method git-fetch)
> + (uri (git-reference
> + (url "https://github.com/linuxmint/xapp/")
> + (commit version)))
> + (sha256
> + (base32
> +
> "0cy9g0zqcbx9zscc9qavqmghfyfb8244cg299llv1ha8n6mpxl3s"))))
> + (build-system meson-build-system)
> + (arguments
> + (list
> + #:phases
> + #~(modify-phases %standard-phases
> + (add-before 'configure 'set-gtk-module-path
> + (lambda* (#:key inputs outputs #:allow-other-keys)
> + (substitute* "libxapp/meson.build"
> + (("gtk3_dep\\.get_pkgconfig_variable[(]'libdir'[)]")
> + (string-append "'" (assoc-ref outputs "out")
> "/lib'")))
> +
> + (substitute* "pygobject/meson.build"
> + (("get_option[(]'py-overrides-dir'[)]")
> + (string-append "'"
> + (assoc-ref outputs "out")
> + "/lib/python"
> + #$(version-major+minor (package-
> version python))
> + "/site-packages/gi/overrides'")))
I think python-build-system has a function to get this version in a
more reliable way.
> + (substitute* "scripts/pastebin"
> + (("'nc'") (string-append "'"
> + (assoc-ref inputs "netcat")
> + "/bin/nc'")))
Use search-input-file.
> + (substitute* "scripts/upload-system-info"
> + (("'inxi'") (string-append "'"
> + (assoc-ref inputs "inxi-
> minimal")
> + "/bin/inxi'"))
Use search-input-file.
> + (("'/usr/bin/pastebin'") (string-append "'"
> + (assoc-ref
> outputs "out")
> +
> "/bin/pastebin'"))
Oof, fine, but note that you can also write #$output
> + (("'xdg-open'") (string-append "'"
> + (assoc-ref inputs
> "xdg-utils")
> + "/bin/xdg-
> open'"))))))))
Use search-input-file.
> + (inputs
> + (list dbus
> + glib ; for gio
> + gtk+
> + inxi-minimal ; used by upload-system-info
> + libdbusmenu
> + libgnomekbd
> + netcat ; used by pastebin
> + xdg-utils ; used by upload-system-info
> + ))
> + (native-inputs
> + (list gettext-minimal
> + `(,glib "bin") ; for glib-mkenums
> + gobject-introspection
> + pkg-config
> + python
> + python-pygobject
> + vala))
> + (home-page "https://github.com/linuxmint/xapp")
> + (synopsis "Cross-desktop libraries and common resources for X-
> apps")
> + (description
> + "The components which are common to multiple GTK desktop
> environments
> +(Cinnamon, MATE and Xfce) and required to implement cross-DE
> solutions.")
The description should consist of full sentences.
Also, since X-Apps are quite specific to Linux Mint, you should
probably not take folks knowing what they are for granted.
> + (license license:lgpl3)))
No +?
Cheers
Information forwarded
to
guix-patches <at> gnu.org
:
bug#60294
; Package
guix-patches
.
(Sat, 24 Dec 2022 11:24:02 GMT)
Full text and
rfc822 format available.
Message #11 received at 60294 <at> debbugs.gnu.org (full text, mbox):
On Saturday, December 24, 2022 at 04:36:02 AM CST, Liliana Marie Prikler <liliana.prikler <at> gmail.com> wrote:
Am Samstag, dem 24.12.2022 um 01:30 -0600 schrieb Wamm K. D:
> > + (substitute* "pygobject/meson.build"
> > + (("get_option[(]'py-overrides-dir'[)]")
> > + (string-append "'"
> > + (assoc-ref outputs "out")
> > + "/lib/python"
> > + #$(version-major+minor (package-
> > version python))
> > + "/site-packages/gi/overrides'")))
> I think python-build-system has a function to get this version in a
> more reliable way.
Would you possibly know any more details on it? Just looking through the module, I don't see any functions regarding version number, I'm afraid, and it's not intuitive to me how I might use it in this context if I'm using a different build system for the package.
> > + (("'/usr/bin/pastebin'") (string-append "'"
> > + (assoc-ref
> > outputs "out")
> > +
> > "/bin/pastebin'"))
> Oof, fine, but note that you can also write #$output
Fair; I'd thought about it but the example I was working off of was using G-expressions and did it this way so I just went with it.
Something I've wondered: is there a point to having the lambda arguments be "#:key outputs", if you can just use "#$output"? I've seen a few setup their lambda that way only to use "#$output" and not ever use "outputs".
I may just be missing something regarding G-expressions, though.
> > + (synopsis "Cross-desktop libraries and common resources for X-
> > apps")
> > + (description
> > + "The components which are common to multiple GTK desktop
> > environments
> > +(Cinnamon, MATE and Xfce) and required to implement cross-DE
> > solutions.")
> The description should consist of full sentences.
> Also, since X-Apps are quite specific to Linux Mint, you should
> probably not take folks knowing what they are for granted.
True but is there any easy way to specify what they're related to without making it much longer than just a synopsis?
"Cross-desktop libraries and common resources" can apply to many things and is generic to the point of being unhelpful, I'd think.
I could always do "developed by Linux Mint" instead of "for X-apps" which might be more familiar but, again, it borders to being unhelpfully unspecific, again (even if less so), to me; there are probably a lot of "Cross-desktop libraries and common resources" developed by Linux Mint. This particular library was developed for use with the X-apps they're developing, specifically.
I'm just not sure what other description would work while still being a synopsis.
> > + (license license:lgpl3)))
> No +?
Mmm; I didn't think so but you raise a good point. Lemme double check myself; I'll adjust it, if otherwise.
Information forwarded
to
guix-patches <at> gnu.org
:
bug#60294
; Package
guix-patches
.
(Sat, 24 Dec 2022 14:14:02 GMT)
Full text and
rfc822 format available.
Message #14 received at 60294 <at> debbugs.gnu.org (full text, mbox):
Am Samstag, dem 24.12.2022 um 11:23 +0000 schrieb Jaft:
> On Saturday, December 24, 2022 at 04:36:02 AM CST, Liliana Marie
> Prikler <liliana.prikler <at> gmail.com> wrote:
>
> Am Samstag, dem 24.12.2022 um 01:30 -0600 schrieb Wamm K. D:
> > > + (substitute* "pygobject/meson.build"
> > > + (("get_option[(]'py-overrides-dir'[)]")
> > > + (string-append "'"
> > > + (assoc-ref outputs "out")
> > > + "/lib/python"
> > > + #$(version-major+minor (package-
> > > version python))
> > > + "/site-
> > > packages/gi/overrides'")))
> > I think python-build-system has a function to get this version in a
> > more reliable way.
>
> Would you possibly know any more details on it? Just looking through
> the module, I don't see any functions regarding version number, I'm
> afraid, and it's not intuitive to me how I might use it in this
> context if I'm using a different build system for the package.
You're looking for the (site-packages) function. As for how to use it,
you need to "mix in" the python build system as is done for instance in
python-gst.
> > > + (("'/usr/bin/pastebin'") (string-append "'"
> > > + (assoc-
> > > ref
> > > outputs "out")
> > > +
> > > "/bin/pastebin'"))
> > Oof, fine, but note that you can also write #$output
>
> Fair; I'd thought about it but the example I was working off of was
> using G-expressions and did it this way so I just went with it.
>
> Something I've wondered: is there a point to having the lambda
> arguments be "#:key outputs", if you can just use "#$output"? I've
> seen a few setup their lambda that way only to use "#$output" and not
> ever use "outputs".
>
> I may just be missing something regarding G-expressions, though.
The outputs key comes from a time in which #$output could not yet be
used in phases. There is currently no definitely preferred flavour.
> > > + (synopsis "Cross-desktop libraries and common resources for
> > > X-
> > > apps")
> > > + (description
> > > + "The components which are common to multiple GTK desktop
> > > environments
> > > +(Cinnamon, MATE and Xfce) and required to implement cross-DE
> > > solutions.")
> > The description should consist of full sentences.
> > Also, since X-Apps are quite specific to Linux Mint, you should
> > probably not take folks knowing what they are for granted.
>
> True but is there any easy way to specify what they're related to
> without making it much longer than just a synopsis?
>
> "Cross-desktop libraries and common resources" can apply to many
> things and is generic to the point of being unhelpful, I'd think.
>
> I could always do "developed by Linux Mint" instead of "for X-apps"
> which might be more familiar but, again, it borders to being
> unhelpfully unspecific, again (even if less so), to me; there are
> probably a lot of "Cross-desktop libraries and common resources"
> developed by Linux Mint. This particular library was developed for
> use with the X-apps they're developing, specifically.
>
> I'm just not sure what other description would work while still being
> a synopsis.
I think "Library for traditional GTK applications" would work fine,
looking at [1].
> >
Cheers
[1]
https://linuxmint-developer-guide.readthedocs.io/en/latest/xapps.html
Information forwarded
to
guix-patches <at> gnu.org
:
bug#60294
; Package
guix-patches
.
(Sun, 25 Dec 2022 00:58:01 GMT)
Full text and
rfc822 format available.
Message #17 received at 60294 <at> debbugs.gnu.org (full text, mbox):
* gnu/packages/cinnamon.scm (libxapp): New variable.
---
You were right about that copyright; /every/ file (which mentioned the
copyright) referenced regular LGPL other than the "copyright" file
under the "debian" directory, where it specified LGPL-3+.
gnu/packages/cinnamon.scm | 76 +++++++++++++++++++++++++++++++++++++++
1 file changed, 76 insertions(+)
diff --git a/gnu/packages/cinnamon.scm b/gnu/packages/cinnamon.scm
index fe33e797e4..1b5cd44f65 100644
--- a/gnu/packages/cinnamon.scm
+++ b/gnu/packages/cinnamon.scm
@@ -26,6 +26,7 @@ (define-module (gnu packages cinnamon)
#:use-module (guix utils)
#:use-module (guix build-system gnu)
#:use-module (gnu packages)
+ #:use-module (gnu packages admin)
#:use-module (gnu packages autotools)
#:use-module (gnu packages freedesktop)
#:use-module (gnu packages gettext)
@@ -37,6 +38,81 @@ (define-module (gnu packages cinnamon)
#:use-module (gnu packages python)
#:use-module (gnu packages xorg))
+(define-public libxapp
+ (package
+ (name "libxapp")
+ (version "2.4.2")
+ (source (origin
+ (method git-fetch)
+ (uri (git-reference
+ (url "https://github.com/linuxmint/xapp/")
+ (commit version)))
+ (sha256
+ (base32
+ "0cy9g0zqcbx9zscc9qavqmghfyfb8244cg299llv1ha8n6mpxl3s"))))
+ (build-system meson-build-system)
+ (arguments
+ (list
+ #:modules
+ `((guix build meson-build-system)
+ (guix build utils)
+ ((guix build python-build-system) #:prefix python:))
+ #:imported-modules
+ `(,@%meson-build-system-modules
+ (guix build python-build-system))
+ #:configure-flags
+ #~(list (string-append
+ "-Dpy-overrides-dir="
+ (python:site-packages %build-inputs %outputs) "/gi/overrides"))
+ #:phases
+ #~(modify-phases %standard-phases
+ (add-before 'configure 'set-gtk-module-path
+ (lambda* (#:key inputs outputs #:allow-other-keys)
+ (substitute* "libxapp/meson.build"
+ (("gtk3_dep\\.get_pkgconfig_variable[(]'libdir'[)]")
+ (string-append "'" (assoc-ref outputs "out") "/lib'")))
+
+ (substitute* "scripts/pastebin"
+ (("'nc'") (string-append "'"
+ (search-input-file inputs "/bin/nc")
+ "'")))
+
+ (substitute* "scripts/upload-system-info"
+ (("'inxi'") (string-append "'"
+ (search-input-file inputs "/bin/inxi")
+ "'"))
+ (("'/usr/bin/pastebin'") (string-append "'"
+ (assoc-ref outputs "out")
+ "/bin/pastebin'"))
+ (("'xdg-open'") (string-append "'"
+ (search-input-file inputs "/bin/xdg-open")
+ "'"))))))))
+ (inputs
+ (list dbus
+ glib ; for gio
+ gtk+
+ inxi-minimal ; used by upload-system-info
+ libdbusmenu
+ libgnomekbd
+ netcat ; used by pastebin
+ xdg-utils ; used by upload-system-info
+ ))
+ (native-inputs
+ (list gettext-minimal
+ `(,glib "bin") ; for glib-mkenums
+ gobject-introspection
+ pkg-config
+ python
+ python-pygobject
+ vala))
+ (home-page "https://github.com/linuxmint/xapp")
+ (synopsis "Library for traditional GTK applications")
+ (description
+ "The libxapp package contains the components which are common to multiple
+GTK desktop environments (Cinnamon, MATE and Xfce) and required to implement
+cross-DE solutions.")
+ (license license:lgpl3+)))
+
(define-public cinnamon-desktop
(package
(name "cinnamon-desktop")
--
2.38.1
Information forwarded
to
guix-patches <at> gnu.org
:
bug#60294
; Package
guix-patches
.
(Thu, 19 Jan 2023 16:07:01 GMT)
Full text and
rfc822 format available.
Message #20 received at 60294 <at> debbugs.gnu.org (full text, mbox):
Just wanted to double check if the changes I made fixed previous problems that were brought up with the first patch.
Information forwarded
to
guix-patches <at> gnu.org
:
bug#60294
; Package
guix-patches
.
(Thu, 19 Jan 2023 19:43:02 GMT)
Full text and
rfc822 format available.
Message #23 received at 60294 <at> debbugs.gnu.org (full text, mbox):
Hi,
Am Donnerstag, dem 19.01.2023 um 16:06 +0000 schrieb Jaft:
> Just wanted to double check if the changes I made fixed previous
> problems that were brought up with the first patch.
Sorry, this flew under the radar. Your v2 still had some minor errors,
but I have a local version that works and will be pushed with my next
batch on Sunday.
Cheers
Information forwarded
to
guix-patches <at> gnu.org
:
bug#60294
; Package
guix-patches
.
(Fri, 20 Jan 2023 05:51:02 GMT)
Full text and
rfc822 format available.
Message #26 received at 60294 <at> debbugs.gnu.org (full text, mbox):
Awesome; thanks so much, Liliana!
Reply sent
to
Liliana Marie Prikler <liliana.prikler <at> gmail.com>
:
You have taken responsibility.
(Sun, 22 Jan 2023 09:09:02 GMT)
Full text and
rfc822 format available.
Notification sent
to
"Wamm K. D" <jaft.r <at> outlook.com>
:
bug acknowledged by developer.
(Sun, 22 Jan 2023 09:09:02 GMT)
Full text and
rfc822 format available.
Message #31 received at 60294-done <at> debbugs.gnu.org (full text, mbox):
Am Freitag, dem 20.01.2023 um 05:50 +0000 schrieb Jaft:
> Awesome; thanks so much, Liliana!
Pushed now.
Cheers
bug archived.
Request was from
Debbugs Internal Request <help-debbugs <at> gnu.org>
to
internal_control <at> debbugs.gnu.org
.
(Sun, 19 Feb 2023 12:24:11 GMT)
Full text and
rfc822 format available.
This bug report was last modified 1 year and 67 days ago.
Previous Next
GNU bug tracking system
Copyright (C) 1999 Darren O. Benham,
1997,2003 nCipher Corporation Ltd,
1994-97 Ian Jackson.