unofficial mirror of guix-devel@gnu.org 
 help / color / mirror / code / Atom feed
* [PATCH] Add nautilus
@ 2016-02-01 16:51 rennes
  2016-02-02  0:28 ` Leo Famulari
  2016-02-07  1:46 ` 宋文武
  0 siblings, 2 replies; 6+ messages in thread
From: rennes @ 2016-02-01 16:51 UTC (permalink / raw)
  To: guix-devel

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

Hi,

i attached patch for review.

Regards

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: 0001-Add-nautilus.patch --]
[-- Type: text/x-diff; name=0001-Add-nautilus.patch, Size: 2752 bytes --]

From 18bddcf21cac4245997a76d4ea5cfc856d1b6157 Mon Sep 17 00:00:00 2001
From: Rene Saavedra <rennes@openmailbox.org>
Date: Mon, 1 Feb 2016 17:45:22 +0100
Subject: [PATCH] Add nautilus

---
 gnu/packages/nautilus.scm | 63 +++++++++++++++++++++++++++++++++++++++++++++++
 1 file changed, 63 insertions(+)
 create mode 100644 gnu/packages/nautilus.scm

diff --git a/gnu/packages/nautilus.scm b/gnu/packages/nautilus.scm
new file mode 100644
index 0000000..f982f0b
--- /dev/null
+++ b/gnu/packages/nautilus.scm
@@ -0,0 +1,63 @@
+2016-02-01  Rene Saavedra  <rennes@openmailbox.org>
+
+	*  gnu/packages/nautilus.scm: New file.
+	*  gnu-system (GNU_SYSTEM_MODULES): Add it.
+
+(define-module (gnu packages nautilus)
+  #:use-module ((guix licenses) #:prefix license:)
+  #:use-module (guix packages)
+  #:use-module (guix download)
+  #:use-module (guix build-system gnu)
+  #:use-module (guix utils)
+  #:use-module (gnu packages freedesktop)
+  #:use-module (gnu packages glib)
+  #:use-module (gnu packages gnome)
+  #:use-module (gnu packages gtk)
+  #:use-module (gnu packages photo)
+  #:use-module (gnu packages pkg-config)
+  #:use-module (gnu packages xml)
+  #:use-module (gnu packages xorg))
+(define-public nautilus
+  (package
+    (name "nautilus")
+    (version "3.18.2")
+    (source
+     (origin
+      (method url-fetch)
+      (uri (string-append "mirror://gnome/sources/" name "/"
+                          (version-major+minor version) "/"
+                           name "-" version ".tar.xz"))
+      (sha256
+       (base32
+        "0jj23n8vmmyc4gp5xhiz7slsxwksydp26blxi5m154yaw9lgdp38"))))
+    (build-system gnu-build-system)
+    (arguments
+     `(#:configure-flags
+       (list "--disable-tracker" "--disable-packagekit")
+       #:tests? #f))
+    (native-inputs
+     `(("glib" ,glib "bin")
+       ("gobject-introspection" ,gobject-introspection)
+       ("intltool" ,intltool)
+       ("pkg-config" ,pkg-config)))
+    (inputs
+     `(("adwaita-icon-theme" ,adwaita-icon-theme)
+       ("exempi" ,exempi)
+       ("glib" ,glib)
+       ("gnome-desktop" ,gnome-desktop)
+       ("gsettings-desktop-schemas" ,gsettings-desktop-schemas)
+       ("gtk+" ,gtk+)
+       ("gvfs" ,gvfs)
+       ("libexif" ,libexif)
+       ("libnotify" ,libnotify)
+       ("libx11" ,libx11)
+       ("libxml2" ,libxml2)
+       ("pango" ,pango)))
+    (home-page "https://wiki.gnome.org/Apps/Nautilus")
+    (synopsis "GNOME file manager")
+    (description
+     "Nautilus is a file manager designed for the GNOME desktop.
+Nautilus design provides the user a simple way and elegant to
+managing files and applications, through simple menus and
+notifications.")
+    (license license:gpl2+)))  ; libnautilus-extenstions under lgpl
-- 
2.6.3


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

* Re: [PATCH] Add nautilus
  2016-02-01 16:51 [PATCH] Add nautilus rennes
@ 2016-02-02  0:28 ` Leo Famulari
  2016-02-02 16:48   ` Andreas Enge
  2016-02-07  1:46 ` 宋文武
  1 sibling, 1 reply; 6+ messages in thread
From: Leo Famulari @ 2016-02-02  0:28 UTC (permalink / raw)
  To: rennes; +Cc: guix-devel

On Mon, Feb 01, 2016 at 05:51:33PM +0100, rennes@openmailbox.org wrote:
> Hi,
> 
> i attached patch for review.
> 
> Regards

> From 18bddcf21cac4245997a76d4ea5cfc856d1b6157 Mon Sep 17 00:00:00 2001
> From: Rene Saavedra <rennes@openmailbox.org>
> Date: Mon, 1 Feb 2016 17:45:22 +0100
> Subject: [PATCH] Add nautilus
> 
> ---
>  gnu/packages/nautilus.scm | 63 +++++++++++++++++++++++++++++++++++++++++++++++
>  1 file changed, 63 insertions(+)
>  create mode 100644 gnu/packages/nautilus.scm

Thanks! I will review some general points. It would be good to get a
GNOME / GTK expert to review this as well — perhaps 宋文武 has some
feedback?

> 
> diff --git a/gnu/packages/nautilus.scm b/gnu/packages/nautilus.scm
> new file mode 100644
> index 0000000..f982f0b
> --- /dev/null
> +++ b/gnu/packages/nautilus.scm
> @@ -0,0 +1,63 @@
> +2016-02-01  Rene Saavedra  <rennes@openmailbox.org>
> +
> +	*  gnu/packages/nautilus.scm: New file.
> +	*  gnu-system (GNU_SYSTEM_MODULES): Add it.

The above lines should go in the commit message, rather than in the
patch itself. You can then generate the patch with `git format-patch`.

> +
> +(define-module (gnu packages nautilus)
> +  #:use-module ((guix licenses) #:prefix license:)
> +  #:use-module (guix packages)
> +  #:use-module (guix download)
> +  #:use-module (guix build-system gnu)
> +  #:use-module (guix utils)
> +  #:use-module (gnu packages freedesktop)
> +  #:use-module (gnu packages glib)
> +  #:use-module (gnu packages gnome)
> +  #:use-module (gnu packages gtk)
> +  #:use-module (gnu packages photo)
> +  #:use-module (gnu packages pkg-config)
> +  #:use-module (gnu packages xml)
> +  #:use-module (gnu packages xorg))

Please insert a blank line here.

> +(define-public nautilus
> +  (package
> +    (name "nautilus")
> +    (version "3.18.2")
> +    (source
> +     (origin
> +      (method url-fetch)
> +      (uri (string-append "mirror://gnome/sources/" name "/"
> +                          (version-major+minor version) "/"
> +                           name "-" version ".tar.xz"))
> +      (sha256
> +       (base32
> +        "0jj23n8vmmyc4gp5xhiz7slsxwksydp26blxi5m154yaw9lgdp38"))))
> +    (build-system gnu-build-system)

This is really a question for a GNOME user, but I wonder if it would be
appropriate to use the glib-or-gtk-build-system?

> +    (arguments
> +     `(#:configure-flags
> +       (list "--disable-tracker" "--disable-packagekit")

It's a good idea to explain package-specific configure-flags, in my
opinion.

> +       #:tests? #f))

Please add a note explaining why tests are disabled. If there are no
tests, just say something like "no test suite".

> +    (native-inputs
> +     `(("glib" ,glib "bin")

Does glib need to be in both native-inputs and inputs?

> +       ("gobject-introspection" ,gobject-introspection)
> +       ("intltool" ,intltool)
> +       ("pkg-config" ,pkg-config)))
> +    (inputs
> +     `(("adwaita-icon-theme" ,adwaita-icon-theme)

I think our policy is to have users install the icons into their
profile, and let other packages find them through environment variables,
as explained here:
http://lists.gnu.org/archive/html/guix-devel/2016-01/msg00764.html
Is that appropriate in this case?

> +       ("exempi" ,exempi)
> +       ("glib" ,glib)
> +       ("gnome-desktop" ,gnome-desktop)
> +       ("gsettings-desktop-schemas" ,gsettings-desktop-schemas)
> +       ("gtk+" ,gtk+)
> +       ("gvfs" ,gvfs)
> +       ("libexif" ,libexif)
> +       ("libnotify" ,libnotify)
> +       ("libx11" ,libx11)
> +       ("libxml2" ,libxml2)
> +       ("pango" ,pango)))
> +    (home-page "https://wiki.gnome.org/Apps/Nautilus")
> +    (synopsis "GNOME file manager")
> +    (description
> +     "Nautilus is a file manager designed for the GNOME desktop.
> +Nautilus design provides the user a simple way and elegant to
> +managing files and applications, through simple menus and
> +notifications.")

I haven't used Nautilus before. Can someone else weigh in on the
description?

> +    (license license:gpl2+)))  ; libnautilus-extenstions under lgpl

You can provide list of licenses. There is an example in the package
definition for 'ruby-power-assert'. But please keep the comment about
what parts are covered under what license.

> -- 
> 2.6.3
> 

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

* Re: [PATCH] Add nautilus
  2016-02-02  0:28 ` Leo Famulari
@ 2016-02-02 16:48   ` Andreas Enge
  0 siblings, 0 replies; 6+ messages in thread
From: Andreas Enge @ 2016-02-02 16:48 UTC (permalink / raw)
  To: Leo Famulari; +Cc: guix-devel, rennes

Should this package not rather go to gnome.scm?

Andreas

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

* Re: [PATCH] Add nautilus
  2016-02-01 16:51 [PATCH] Add nautilus rennes
  2016-02-02  0:28 ` Leo Famulari
@ 2016-02-07  1:46 ` 宋文武
  2016-02-08  5:46   ` 宋文武
  1 sibling, 1 reply; 6+ messages in thread
From: 宋文武 @ 2016-02-07  1:46 UTC (permalink / raw)
  To: rennes; +Cc: guix-devel

rennes@openmailbox.org writes:

> Hi,
>
> i attached patch for review.
>
> Regards
>
> From 18bddcf21cac4245997a76d4ea5cfc856d1b6157 Mon Sep 17 00:00:00 2001
> From: Rene Saavedra <rennes@openmailbox.org>
> Date: Mon, 1 Feb 2016 17:45:22 +0100
> Subject: [PATCH] Add nautilus
>
> ---
>  gnu/packages/nautilus.scm | 63 +++++++++++++++++++++++++++++++++++++++++++++++
>  1 file changed, 63 insertions(+)
>  create mode 100644 gnu/packages/nautilus.scm
>
> diff --git a/gnu/packages/nautilus.scm b/gnu/packages/nautilus.scm
> new file mode 100644
> index 0000000..f982f0b
> --- /dev/null
> +++ b/gnu/packages/nautilus.scm
> @@ -0,0 +1,63 @@
> +2016-02-01  Rene Saavedra  <rennes@openmailbox.org>
> +
> +	*  gnu/packages/nautilus.scm: New file.
> +	*  gnu-system (GNU_SYSTEM_MODULES): Add it.
Should add to gnome.scm
> +
> +(define-module (gnu packages nautilus)
> +  #:use-module ((guix licenses) #:prefix license:)
> +  #:use-module (guix packages)
> +  #:use-module (guix download)
> +  #:use-module (guix build-system gnu)
> +  #:use-module (guix utils)
> +  #:use-module (gnu packages freedesktop)
> +  #:use-module (gnu packages glib)
> +  #:use-module (gnu packages gnome)
> +  #:use-module (gnu packages gtk)
> +  #:use-module (gnu packages photo)
> +  #:use-module (gnu packages pkg-config)
> +  #:use-module (gnu packages xml)
> +  #:use-module (gnu packages xorg))
> +(define-public nautilus
> +  (package
> +    (name "nautilus")
> +    (version "3.18.2")
I see the latest stable version is 3.18.5, but that require a
newer GTK+ last time I tried.
OK here with a comment about it.
> +    (source
> +     (origin
> +      (method url-fetch)
> +      (uri (string-append "mirror://gnome/sources/" name "/"
> +                          (version-major+minor version) "/"
> +                           name "-" version ".tar.xz"))
> +      (sha256
> +       (base32
> +        "0jj23n8vmmyc4gp5xhiz7slsxwksydp26blxi5m154yaw9lgdp38"))))
> +    (build-system gnu-build-system)
I think glib-or-gtk-build-system is needed, GNOME applications usually
required gsettings schemas to be run, glib-or-gtk-build-system handles
that by wrap the binaries with XDG_DATA_DIRS to find the schemas.

You can run nautilus with 'XDG_DATA_DIRS' unset to test.
> +    (arguments
> +     `(#:configure-flags
> +       (list "--disable-tracker" "--disable-packagekit")
Please add some comment, eg: XXX: not packaged.
> +       #:tests? #f))
Add comment why tests is disable, eg: XXX: failed with ....
> +    (native-inputs
> +     `(("glib" ,glib "bin")
> +       ("gobject-introspection" ,gobject-introspection)
> +       ("intltool" ,intltool)
> +       ("pkg-config" ,pkg-config)))
> +    (inputs
> +     `(("adwaita-icon-theme" ,adwaita-icon-theme)
> +       ("exempi" ,exempi)
> +       ("glib" ,glib)
> +       ("gnome-desktop" ,gnome-desktop)
> +       ("gsettings-desktop-schemas" ,gsettings-desktop-schemas)
> +       ("gtk+" ,gtk+)
> +       ("gvfs" ,gvfs)
> +       ("libexif" ,libexif)
> +       ("libnotify" ,libnotify)
> +       ("libx11" ,libx11)
> +       ("libxml2" ,libxml2)
> +       ("pango" ,pango)))
> +    (home-page "https://wiki.gnome.org/Apps/Nautilus")
> +    (synopsis "GNOME file manager")
> +    (description
> +     "Nautilus is a file manager designed for the GNOME desktop.
> +Nautilus design provides the user a simple way and elegant to
> +managing files and applications, through simple menus and
> +notifications.")
> +    (license license:gpl2+)))  ; libnautilus-extenstions under lgpl

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

* Re: [PATCH] Add nautilus
  2016-02-07  1:46 ` 宋文武
@ 2016-02-08  5:46   ` 宋文武
  2016-02-08 23:53     ` d.4.n.1
  0 siblings, 1 reply; 6+ messages in thread
From: 宋文武 @ 2016-02-08  5:46 UTC (permalink / raw)
  To: rennes; +Cc: guix-devel

iyzsong@member.fsf.org (宋文武) writes:

> rennes@openmailbox.org writes:
>
>> Hi,
>>
>> i attached patch for review.
>>
OK, I'm little hurry and pushed with following adjustments:

- Move to gnome.scm.
- Add comments.
- Use glib-or-gtk-build-system.
- Remove the '--disable-packagekit' flag, it only use dbus.
- Move 'gtk+' to propagated-inputs.
- Remove some unnecessarily inputs propagated by gnome-desktop.
- Remove 'gvfs', I can't get it work now (eg: trash not working).
  I think it's like dconf which have a gio module using dbus to
  talk with a daemon, maybe the daemon should be propagated too.
  But since the gvfs package is big, I'll try split it latter.
- Add copyright headers for us.

Thanks for the patch!

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

* Re: [PATCH] Add nautilus
  2016-02-08  5:46   ` 宋文武
@ 2016-02-08 23:53     ` d.4.n.1
  0 siblings, 0 replies; 6+ messages in thread
From: d.4.n.1 @ 2016-02-08 23:53 UTC (permalink / raw)
  To: iyzsong, rennes; +Cc: guix-devel

[-- Attachment #1: Type: text/html, Size: 1423 bytes --]

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

end of thread, other threads:[~2016-02-08 23:55 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-02-01 16:51 [PATCH] Add nautilus rennes
2016-02-02  0:28 ` Leo Famulari
2016-02-02 16:48   ` Andreas Enge
2016-02-07  1:46 ` 宋文武
2016-02-08  5:46   ` 宋文武
2016-02-08 23:53     ` d.4.n.1

Code repositories for project(s) associated with this public inbox

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

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).