Skip to content

Conversation

@RomanPudashkin
Copy link
Contributor

As it requires xxd to be installed on Windows. And we don't want to make the build process even more difficult for new developers. @rpatters1 FYI

@RomanPudashkin RomanPudashkin requested a review from Eism February 9, 2026 08:58
@RomanPudashkin RomanPudashkin force-pushed the disable_mnx_in_local_builds branch from 3a68129 to 255d3df Compare February 9, 2026 09:02
@RomanPudashkin RomanPudashkin moved this to In Progress in MuseScore Studio 4.7 Feb 9, 2026
@RomanPudashkin RomanPudashkin added the dev Related to developer experience (compiling, code base, CI), rather than end user experience label Feb 9, 2026
@rpatters1
Copy link
Contributor

Is it possible to limit this just to the Win dev build? There is no reason not to build it on Linux or Mac devs, since those come with xxd installed. And the more computers running the build, the more likely we are to shake out bugs. (Heck, the Win dev build already shook out an issue.)

@rpatters1
Copy link
Contributor

rpatters1 commented Feb 9, 2026

Also, I will add here (just to document it) that to intall xxd on Windows requires

choco install xxd -y

I believe the standard Windows dev setup already suggests using choco.

@cbjeukendrup
Copy link
Member

Or can we do it based on the availability of xxd? I.e. look for xxd; if not found, print warning and skip mnx module

@RomanPudashkin RomanPudashkin force-pushed the disable_mnx_in_local_builds branch from 255d3df to 3a77786 Compare February 9, 2026 12:35
@rpatters1
Copy link
Contributor

rpatters1 commented Feb 9, 2026

Thank you so much. The minute #embed is available on our compilers, I will get rid of xxd. I don't love it.

@RomanPudashkin RomanPudashkin changed the title Enable MNX module only on CI Windows: enable MNX module only on CI Feb 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dev Related to developer experience (compiling, code base, CI), rather than end user experience

Projects

Status: In Progress

Development

Successfully merging this pull request may close these issues.

4 participants