[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