Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Various CI and build tweaks #751

Merged
merged 8 commits into from
Apr 21, 2024
Merged

Various CI and build tweaks #751

merged 8 commits into from
Apr 21, 2024

Conversation

TheOneric
Copy link
Member

  • automatically use all available cores
  • add muon CI
    • muon already fixed e.g. the false-positive warnings we noticed earlier and currently is also tweaking formatting rules. The latter means our buildfiles no longer match muon style, but let’s wait for 0.3.0 before deciding whether to care at all or reformat and make this an actual fail
  • ensure meson build works from tarballs
  • get rid of unneeded CONFIG_ macros in both builds
  • share library and release version across buildsystems via a toplevel RELEASEVERSION file (exact name and placement can be changed)

astiob
astiob approved these changes Apr 20, 2024
View reviewed changes
Copy link
Member

@astiob astiob left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

idk meson, lgtm

.github/workflows/meson.yml Outdated Show resolved Hide resolved
@TheOneric
build/meson: read library version from automake
5de7be3
@TheOneric
build: share release version between autoconf and meson
bfebfa6
@TheOneric
ci/gha: reduce meson CI runners
50adc6d 
One release build with wrap fallbacks is sufficient
@TheOneric
ci/gha: add muon test
469a947
@TheOneric
ci/gha: test meson installation from tarball
ae6f88b
@TheOneric
ci/gha: use system deps in one Windows meson build
ffe070b 
And install meson from system except for Chocolatey
which doesn’t like installing meson without a reboot.

Split out Chocolatey is necessary, because using bash as a shell
for some reason also changes the used linker leading to config-time
failures for MSVC.
@TheOneric TheOneric merged commit ffe070b into libass:master Apr 21, 2024
15 checks passed
@TheOneric TheOneric deleted the tweak-build+ci branch April 21, 2024 16:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@astiob astiob astiob approved these changes

Assignees
No one assigned
Labels
build
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@TheOneric @astiob