Repository navigation
Add packaging cpack and mingw - #2
Open
timandtheocean wants to merge 3 commits into
Open
timandtheocean wants to merge 3 commits into
timandtheocean wants to merge 3 commits into
Conversation
… component Hi Martin, Long time no speak. Hope things are going well with you and your family. I started to prepare to build a preset for the Korg Z1 and added package generation directives (cpack). On my machine I created Deb, tgz, rpm, (windows .zip and .exe ) packages and strived to make it as portable as possible. On Debian the build and deployment was tested and on Windows the .zip and .exe where tested. Maybe after some test you can add the packages to the git repo release section or on your site. Best regards, Tim (Prague) ## Summary - Fix `cpack` failure (installed header from non-existent `src/sysex.hpp`), remove duplicate install rules from CMakeLists.txt - Use distro RtMidi via pkg-config when no CMake config exists (Debian/Ubuntu `librtmidi-dev`) - Package only project files (CPack component); FetchContent deps no longer leak RtMidi/ArduinoJson/doctest headers into the package - DEB/RPM declare RtMidi as dependency - Add MinGW-w64 toolchain to cross-compile Windows ZIP/NSIS packages on Linux - Silence GCC 14 false-positive -Wstringop-overflow in tests - README.md: includes the packaging instructions ## Tests - Built of all packages: Operating System: Debian GNU/Linux 13 (trixie) Linux 6.16.3+deb13-amd64 Architecture: x86-64 - Deployed Debian: DEB next to librtmidi-dev, contains only electraone files - Windows cross build: electraone.exe depends only on KERNEL32/msvcrt/WINMM - Tested and deployed the .zip and .exe installer on Win10. - During tests there was 1 Warning during the test, "writing 1 byte into a region of size 0", with a follow-up note about a 13-byte buffer I added -Wno-stringop-overflow for the test target with GCC only (GCC 14)
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Hi Martin,
Long time no speak.
Hope things are going well with you and your family.
I started to prepare to build a preset for the Korg Z1 and added package generation directives (cpack).
On my machine I created Deb, tgz, rpm, (windows .zip and .exe ) packages and strived to make it as portable as possible.
On Debian the build and deployment was tested and on Windows the .zip and .exe where tested.
Maybe after some test you can add the packages to the git repo release section or on your site.
Best regards,
Tim (Prague)
Summary
cpackfailure (installed header from non-existentsrc/sysex.hpp), remove duplicate install rules from CMakeLists.txtlibrtmidi-dev)Tests