Skip to content

Slightly modernize CI - #795

Merged
cgwalters merged 3 commits into
containers:mainfrom
smcv:ci
Sep 16, 2026
Merged

cgwalters merged 3 commits into
containers:mainfrom
smcv:ci

Conversation

@smcv

@smcv smcv commented Sep 15, 2026

Copy link
Copy Markdown
Collaborator
  • CI: Use meson setup $builddir instead of deprecated meson $builddir

  • CI: Convert CFLAGS into Meson built-in options where possible

Signed-off-by: Simon McVittie <smcv@collabora.com>
Signed-off-by: Simon McVittie <smcv@collabora.com>
@xxyzz

xxyzz commented Sep 16, 2026

Copy link
Copy Markdown
Contributor

Would you mind also adding the meson setup command in the README.md file's "Installation" section?

Signed-off-by: Simon McVittie <smcv@collabora.com>
@smcv

smcv commented Sep 16, 2026

Copy link
Copy Markdown
Collaborator Author

Would you mind also adding the meson setup command in the README.md file's "Installation" section?

Sure, done.

(I continue to be somewhat sceptical about having these generic installation instructions in our readme, because it isn't really bubblewrap's job to teach new developers how to compile a project, and something as subtle and security-sensitive as bubblewrap isn't really a great place to start learning Unix programming. But while we have them, they might as well be as correct as possible...)

@cgwalters
cgwalters merged commit fb9acea into containers:main Sep 16, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants