Kdenlive Fixes #2

  • Done
  • quality assurance status badge
Details
2 participants
  • Ludovic Courtès
  • Prafulla Giri
Owner
unassigned
Submitted by
Prafulla Giri
Severity
normal
P
P
Prafulla Giri wrote on 9 Sep 2020 11:20
(address . guix-patches@gnu.org)
CAFw+=j0reVBEg=rPRcLdonzutRGCdRzgEVH-ymuSRDv2+1ymgw@mail.gmail.com
Esteemed maintainers,

This is a follow-up to https://issues.guix.gnu.org/43200and addresses the
second issue mentioned there: breeze icon theme not being found by kdenlive.

Attached are two patches: the first makes the breeze icon theme available
to kdenlive during runtime by setting XDG_DATA_DIRS to include the 'share'
directory of the breeze package. The second updates kdenlive's source url
to follow permanent redirection as reported by `guix lint`.

Thank you.
Attachment: file
From 01b6f35a272e569c9cd0beed2ed210f536e5e44f Mon Sep 17 00:00:00 2001
From: Prafulla Giri <pratheblackdiamond@gmail.com>
Date: Wed, 9 Sep 2020 14:02:24 +0545
Subject: [PATCH 1/2] gnu: kde: kdenlive: Make breeze theme available at
runtime.

* gnu/packages/kde.scm (kdenlive):
[inputs]: Add breeze.
[arguments]: In wrap-program phase, add XDG_DATA_DIRS to include
the 'share' sub-directory of the package breeze.
---
gnu/packages/kde.scm | 4 ++++
1 file changed, 4 insertions(+)

Toggle diff (28 lines)
diff --git a/gnu/packages/kde.scm b/gnu/packages/kde.scm
index a9ac23e049..b22c3f0d8f 100644
--- a/gnu/packages/kde.scm
+++ b/gnu/packages/kde.scm
@@ -177,6 +177,7 @@ This package contains GUI widgets for baloo.")
("qtquickcontrols" ,qtquickcontrols)
("qtquickcontrols2" ,qtquickcontrols2)
("kiconthemes" ,kiconthemes)
+ ("breeze" ,breeze)
("qtgraphicaleffects" ,qtgraphicaleffects)
("kplotting" ,kplotting)))
(arguments
@@ -192,10 +193,13 @@ This package contains GUI widgets for baloo.")
(qtdeclarative (assoc-ref inputs "qtdeclarative"))
(frei0r (assoc-ref inputs "frei0r-plugins"))
(ffmpeg (assoc-ref inputs "ffmpeg"))
+ (breeze (assoc-ref inputs "breeze"))
(qml "/lib/qt5/qml"))
(wrap-program (string-append out "/bin/kdenlive")
`("PATH" ":" prefix
,(list (string-append ffmpeg "/bin")))
+ `("XDG_DATA_DIRS" ":" prefix
+ ,(list (string-append breeze "/share")))
`("QT_PLUGIN_PATH" ":" prefix
,(map (lambda (label)
(string-append (assoc-ref inputs label)
--
2.28.0
From 84b32417f838f703aa37a69ff7b974341504c046 Mon Sep 17 00:00:00 2001
From: Prafulla Giri <pratheblackdiamond@gmail.com>
Date: Wed, 9 Sep 2020 14:55:22 +0545
Subject: [PATCH 2/2] gnu: kdenlive: Change source URL to follow permanent
redirection.

* gnu/packages/kde.scm (kdenlive)[source]: Change url to follow
permanent redirection.
---
gnu/packages/kde.scm | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

Toggle diff (15 lines)
diff --git a/gnu/packages/kde.scm b/gnu/packages/kde.scm
index b22c3f0d8f..75e7f39c4b 100644
--- a/gnu/packages/kde.scm
+++ b/gnu/packages/kde.scm
@@ -142,7 +142,7 @@ This package contains GUI widgets for baloo.")
(origin
(method git-fetch)
(uri (git-reference
- (url "https://anongit.kde.org/kdenlive.git")
+ (url "https://invent.kde.org/multimedia/kdenlive")
(commit (string-append "v" version))))
(file-name (string-append name "-" version "-checkout"))
(sha256
--
2.28.0
L
L
Ludovic Courtès wrote on 14 Sep 2020 23:03
(name . Prafulla Giri)(address . pratheblackdiamond@gmail.com)(address . 43288-done@debbugs.gnu.org)
87r1r49ikp.fsf@gnu.org
Hi,

Prafulla Giri <pratheblackdiamond@gmail.com> skribis:

Toggle quote (11 lines)
> From 01b6f35a272e569c9cd0beed2ed210f536e5e44f Mon Sep 17 00:00:00 2001
> From: Prafulla Giri <pratheblackdiamond@gmail.com>
> Date: Wed, 9 Sep 2020 14:02:24 +0545
> Subject: [PATCH 1/2] gnu: kde: kdenlive: Make breeze theme available at
> runtime.
>
> * gnu/packages/kde.scm (kdenlive):
> [inputs]: Add breeze.
> [arguments]: In wrap-program phase, add XDG_DATA_DIRS to include
> the 'share' sub-directory of the package breeze.

[...]

Toggle quote (9 lines)
> From 84b32417f838f703aa37a69ff7b974341504c046 Mon Sep 17 00:00:00 2001
> From: Prafulla Giri <pratheblackdiamond@gmail.com>
> Date: Wed, 9 Sep 2020 14:55:22 +0545
> Subject: [PATCH 2/2] gnu: kdenlive: Change source URL to follow permanent
> redirection.
>
> * gnu/packages/kde.scm (kdenlive)[source]: Change url to follow
> permanent redirection.

Applied, thanks!

Ludo’.
Closed
P
P
Prafulla Giri wrote on 15 Sep 2020 07:31
(name . Ludovic Courtès)(address . ludo@gnu.org)(address . 43288-done@debbugs.gnu.org)
CAFw+=j1h58WNorEqVE7_946AOZe=y3xBX4d15jSmo+iHcD_8pQ@mail.gmail.com
Always an honour, Mr. Courtès.

Thank you very much.

On Tue, Sep 15, 2020 at 2:49 AM Ludovic Courtès <ludo@gnu.org> wrote:

Toggle quote (30 lines)
> Hi,
>
> Prafulla Giri <pratheblackdiamond@gmail.com> skribis:
>
> > From 01b6f35a272e569c9cd0beed2ed210f536e5e44f Mon Sep 17 00:00:00 2001
> > From: Prafulla Giri <pratheblackdiamond@gmail.com>
> > Date: Wed, 9 Sep 2020 14:02:24 +0545
> > Subject: [PATCH 1/2] gnu: kde: kdenlive: Make breeze theme available at
> > runtime.
> >
> > * gnu/packages/kde.scm (kdenlive):
> > [inputs]: Add breeze.
> > [arguments]: In wrap-program phase, add XDG_DATA_DIRS to include
> > the 'share' sub-directory of the package breeze.
>
> [...]
>
> > From 84b32417f838f703aa37a69ff7b974341504c046 Mon Sep 17 00:00:00 2001
> > From: Prafulla Giri <pratheblackdiamond@gmail.com>
> > Date: Wed, 9 Sep 2020 14:55:22 +0545
> > Subject: [PATCH 2/2] gnu: kdenlive: Change source URL to follow permanent
> > redirection.
> >
> > * gnu/packages/kde.scm (kdenlive)[source]: Change url to follow
> > permanent redirection.
>
> Applied, thanks!
>
> Ludo’.
>
Attachment: file
Closed
?