[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 10/24] gnu: networkmanager-qt: Propagate network-manager.
From: |
Alex Kost |
Subject: |
Re: [PATCH 10/24] gnu: networkmanager-qt: Propagate network-manager. |
Date: |
Tue, 16 Aug 2016 23:07:17 +0300 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux) |
David Craven (2016-08-16 21:39 +0300) wrote:
> * gnu/packages/kde-frameworks.scm (networkmanager-qt)
> [propagated-inputs]: Add NETWORK-MANAGER.
> [inputs]: Remove NETWORK-MANAGER.
I would write 'network-manager' instead of NETWORK-MANAGER.
Also there should be a reason for propagating (probably written as a
comment). Ideally it should be avoided.
> ---
> gnu/packages/kde-frameworks.scm | 5 +++--
> 1 file changed, 3 insertions(+), 2 deletions(-)
>
>
> diff --git a/gnu/packages/kde-frameworks.scm b/gnu/packages/kde-frameworks.scm
> index dbec5a3..61ed4aa 100644
> --- a/gnu/packages/kde-frameworks.scm
> +++ b/gnu/packages/kde-frameworks.scm
> @@ -934,9 +934,10 @@ messages.")
> `(("extra-cmake-modules" ,extra-cmake-modules)
> ("dbus" ,dbus)
> ("pkg-config" ,pkg-config)))
> + (propagated-inputs
> + `(("network-manager", network-manager)))
↑↑
BTW please fix the comma placement here:
↓↓
`(("network-manager" ,network-manager)))
> (inputs
> - `(("network-manager", network-manager)
> - ("qtbase" ,qtbase)))
> + `(("qtbase" ,qtbase)))
> (arguments
> `(#:phases
> (modify-phases %standard-phases
--
Alex
- Re: [PATCH 15/24] gnu: gstreamer: Symlink gstconfig.h into include directory., (continued)
[PATCH 09/24] gnu: freedesktop: Add wayland-protocols., David Craven, 2016/08/16
[PATCH 12/24] gnu: gnome: Add mobile-broadband-provider-info., David Craven, 2016/08/16
[PATCH 10/24] gnu: networkmanager-qt: Propagate network-manager., David Craven, 2016/08/16
- Re: [PATCH 10/24] gnu: networkmanager-qt: Propagate network-manager.,
Alex Kost <=
[PATCH 17/24] gnu: kwidgetsaddons: Fix test failure., David Craven, 2016/08/16
[PATCH 13/24] gnu: polkit-qt: Use modular qt., David Craven, 2016/08/16
[PATCH 16/24] gnu: gstreamer: Add qt-gstreamer., David Craven, 2016/08/16
[PATCH 19/24] gnu: fritzing: Use modular qt., David Craven, 2016/08/16