I've attached an updated version 3 of the music.scm patch.

Den ons 8 jan. 2025 kl 13:01 skrev Liliana Marie Prikler <liliana.prikler@gmail.com>:
Am Mittwoch, dem 01.01.2025 um 19:37 +0100 schrieb Rovanion Luckey:
> -        (base32
> "1hx0l6d7avyfbh88hwn01h9q51mgd9zix91q2kgg1ax73pqxhfs2"))
> -       (modules '((guix build utils)))
> -       (snippet
> -        '(begin
> -           ;; Delete precompiled binaries.
> -           (delete-file-recursively
> "src/diagnostics/crashpad_handler")
> -           (substitute* "src/diagnostics/CMakeLists.txt"
> -             (("install") "#install"))))))
> +        (base32
> "0cjp1sp50pwmrgvpxjxg849s0vsvk2vcb66ym617nvlj761h0ngz"))
> +       (modules '((guix build utils)))))
This change is undocumented.  If you do drop the snippet (is it really
safe to?), then you can also drop the modules.
As far as I can tell, yes. This step is superseeded by the flag -DMUSE_MODULE_DIAGNOSTICS_CRASHPAD_CLIENT=OFF.

>      (build-system qt-build-system)
>      (arguments
> -     `(#:configure-flags
> -       `("-DDOWNLOAD_SOUNDFONT=OFF"
> -         "-DBUILD_DIAGNOSTICS=OFF"
> -         "-DMUSESCORE_BUILD_CONFIG=release"
> -         "-DUSE_SYSTEM_FREETYPE=ON")
> +     `(#:qtbase ,qtbase
Is this flag needed?  Anyway, new flags should be documented…
Yes, Musescore will crash on startup otherwise as it would be linked against the wrong qtbase. Added a comment.

It's of course a bit insane that the qt-build-system doesn't pick up the qt-base passed as arguments for the package, but that is what it is.

> +       #:configure-flags
> +       `("-DMUSE_APP_BUILD_MODE=release"
> +         ;; Disable the build and usage of the
> `/bin/crashpad_handler` utility -
> +         ;; it does automatic crash reporting and is distributed as
> a
> +         ;; pre-compiled binary in the source-tree of MuseScore:
> +         ;;  https://github.com/musescore/MuseScore/issues/15571
> +         ;; Renamed from MUE_BUILD_CRASHPAD_CLIENT,
> MUE_BUILD_DIAGNOSTICS_MODULE
> +         ;;
> https://github.com/musescore/MuseScore/commit/6f269e8b072cca36cb76eb016cb60c1c1c2b9906
> +         "-DMUSE_MODULE_DIAGNOSTICS_CRASHPAD_CLIENT=OFF"
> +         ;; Use Guix' versions of system libraries.
> +         "-DMUE_COMPILE_USE_SYSTEM_FREETYPE=ON"
> +         "-DMUE_COMPILE_USE_SYSTEM_HARFBUZZ=ON"
> +         "-DMUE_COMPILE_USE_SYSTEM_TINYXML=ON"
> +         "-DMUE_COMPILE_USE_SYSTEM_OPUSENC=ON" ; Ipmlies -
> DMUE_COMPILE_USE_SYSTEM_OPUS=ON
> +         "-DMUE_COMPILE_USE_SYSTEM_FLAC=ON"
> +         ;; Disable download of soundfont during build.
> +         "-DDOWNLOAD_SOUNDFONT=OFF"
> +         ;; Don't bundle Qt QML files, relevant really only for
> Darwin.
> +         ;; "-DMUE_COMPILE_INSTALL_QTQML_FILES=OFF"
> +         )
… as should changes here. 
I clearified the comment spanning multiple lines and hopefully understood the meaning of document correctly when I added some text to the commit message.

Is everything renamed from MUSE to MUE or
are these typos?
Yes, renamed.

Also, parentheses are social animals, they like to group together :)
>         ;; There are tests, but no simple target to run.  The command
> used to
>         ;; run them is:
>         ;;
> @@ -5404,30 +5414,31 @@ (define-public musescore
>         ;; So we simply skip them.
>         #:tests? #f))
>      (native-inputs
> -     (list git-minimal pkg-config qttools-5))
> +     (list git-minimal pkg-config qttools))
Write this out in the ChangeLog.
Sure.

>      (inputs
>       (list alsa-lib
>             freetype
>             `(,gtk+ "bin")               ;for gtk-update-icon-cache
> +           harfbuzz-meson
Whoopsie, this can be regular harfbuzz with the new 1/2.
Fixed.

>             jack-1
>             lame
>             libogg
> +           libopusenc
>             libsndfile
>             libvorbis
>             portaudio
>             portmidi
>             pulseaudio
>             python
> -           qtbase-5
> -           qtdeclarative-5
> -           qtgraphicaleffects
> -           qtnetworkauth-5
> -           qtquickcontrols-5
> -           qtquickcontrols2-5
> -           qtscript
> -           qtsvg-5
> -           qtx11extras
> -           qtxmlpatterns))
> +           qt5compat
> +           qtbase
> +           qtdeclarative
> +           qtnetworkauth
> +           qtscxml
> +           qtshadertools
> +           qtsvg
> +           qtwayland
> +           tinyxml2))
Write these out in the ChangeLog.
Done.

>      (propagated-inputs
>       (list `(,alsa-plugins "pulseaudio"))) ;for
> libasound_module_conf_pulse.so
>      (synopsis "Music composition and notation software")

Cheers

Thanks for the review, hope this one fits the bill.