guix-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH] Add upower


From: Mark H Weaver
Subject: Re: [PATCH] Add upower
Date: Sun, 05 Apr 2015 11:07:43 -0400
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux)

Hi Andy!

Here are some more comments to add to David's:

Andy Wingo <address@hidden> writes:

> From 4725d267b7c18bc4ba4294f90336871311386bb2 Mon Sep 17 00:00:00 2001
> From: Andy Wingo <address@hidden>
> Date: Sat, 4 Apr 2015 21:48:01 +0200
> Subject: [PATCH 2/3] Add upower package.

By our conventions, we write this first line as: "gnu: Add upower."

> * gnu/packages/gnome.scm (upower): New variable.
> * gnu/packages/patches/upower-builddir.patch: New patch.

The patch also needs to be added to 'dist_patch_DATA' in gnu-system.am.

David Thompson <address@hidden> wrote:
> Could you add a comment about the upstream status of this patch, perhaps
> with a URL pointing to the relevant bug tracker, if applicable?

Agreed, but I think this should go in the top of the patch file rather
than in the package definition itself.

      Thanks!
        Mark



reply via email to

[Prev in Thread] Current Thread [Next in Thread]