diff mbox series

[bug#60108] gnu: musescore: Fix build.

Message ID 26ccc41a0b59e854f261e65cfee65803e78746f7.camel@gmail.com
State New
Headers
Series [bug#60108] gnu: musescore: Fix build. |

Commit Message

Liliana Marie Prikler Dec. 15, 2022, 11:31 p.m. UTC 
* gnu/packages/music.scm (musescore)[snippet]: Remove crashpad_handler and
associated install actions.
[arguments]<#:configure-flags>: Add “-DBUILD_DIAGNOSTICS=OFF”.
---
 gnu/packages/music.scm | 9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)

Comments

Nicolas Goaziou Dec. 16, 2022, 5:39 p.m. UTC | #1 
Hello,

Liliana Marie Prikler <liliana.prikler@gmail.com> writes:

> * gnu/packages/music.scm (musescore)[snippet]: Remove crashpad_handler and
> associated install actions.
> [arguments]<#:configure-flags>: Add “-DBUILD_DIAGNOSTICS=OFF”.

FWIW, qa.guix.gnu.org says it looks good to it.

Regards,
Vivien Kraus Dec. 18, 2022, 4:27 p.m. UTC | #2 
I tried this patch, but unfortunately MuseScore crashes on startup,
saying that: qrc:/qml/DevTools/Preferences/SettingsPage.qml:24:1:
module "QtQuick.Dialogs" is not installed

I checked the source, and QtQuick.Dialogs is indeed imported in a bunch
of QML files.

I don’t know much about qt, so I’m not sure how to proceed. I tried to
copy the package for qtquick-controls to qtquick-dialogs, but guix
can’t find any source for that.

Best regards,

Vivien
Vivien Kraus Dec. 18, 2022, 8:12 p.m. UTC | #3 
Le dimanche 18 décembre 2022 à 17:27 +0100, Vivien Kraus a écrit :
> I tried this patch, but unfortunately MuseScore crashes on startup,
> saying that: qrc:/qml/DevTools/Preferences/SettingsPage.qml:24:1:
> module "QtQuick.Dialogs" is not installed

Adding both qtquickcontrols-5 and qtquickcontrols2-5 fixes the issue :)
Nicolas Goaziou Dec. 19, 2022, 3:40 p.m. UTC | #4 
Hello,

Vivien Kraus via Guix-patches via <guix-patches@gnu.org> writes:

> Le dimanche 18 décembre 2022 à 17:27 +0100, Vivien Kraus a écrit :
>> I tried this patch, but unfortunately MuseScore crashes on startup,
>> saying that: qrc:/qml/DevTools/Preferences/SettingsPage.qml:24:1:
>> module "QtQuick.Dialogs" is not installed
>
> Adding both qtquickcontrols-5 and qtquickcontrols2-5 fixes the
> issue :)

Would you mind sending a patch including this, so that we can apply it
and close the issue?

Regards,
Vivien Kraus Dec. 19, 2022, 4:52 p.m. UTC | #5 
Le lundi 19 décembre 2022 à 16:40 +0100, Nicolas Goaziou a écrit :
> Would you mind sending a patch including this, so that we can apply
> it
> and close the issue?

I think I messed things up and the original "From:" header is now in
the commit message, sorry.

Can you fix it on your end? I’m not sure I know how to keep the patch
author information while still being able to send it.

Vivien
diff mbox series

Patch

diff --git a/gnu/packages/music.scm b/gnu/packages/music.scm
index 614eb57791..0d7fb21701 100644
--- a/gnu/packages/music.scm
+++ b/gnu/packages/music.scm
@@ -4787,14 +4787,19 @@  (define-public musescore
         (base32 "16rcwr6fzghv8100syzicabqg8jqvng3zzsi6h3ja4zkp9hcbkcr"))
        (modules '((guix build utils)))
        (snippet
-        ;; Remove unused libraries.
         '(begin
+           ;; Remove unused libraries...
            (for-each delete-file-recursively
-                     '("thirdparty/freetype"))))))
+                     '("thirdparty/freetype"))
+           ;; ... and precompiled binaries.
+           (delete-file-recursively "src/diagnostics/crashpad_handler")
+           (substitute* "src/diagnostics/CMakeLists.txt"
+             (("install") "#install"))))))
     (build-system qt-build-system)
     (arguments
      `(#:configure-flags
        `("-DDOWNLOAD_SOUNDFONT=OFF"
+         "-DBUILD_DIAGNOSTICS=OFF"
          "-DMUSESCORE_BUILD_CONFIG=release"
          "-DUSE_SYSTEM_FREETYPE=ON")
        ;; There are tests, but no simple target to run.  The command used to