Merge pull request #3038 from cgwalters/rust-drop-composefs

rust: Drop composefs from crate

Merge pull request #3037 from cgwalters/add-labeler-flow

ci: Add an automatic labeler action

rust: Drop composefs from crate

This greatly reduces the size.  TODO: switch to using `include`.

Merge pull request #3035 from cgwalters/release-rust

rust/sys: Also bump semver for this

ci: Add an automatic labeler action

To automate a bit.

Merge pull request #3030 from ostreedev/dependabot/submodules/composefs-af86742

build(deps): bump composefs from `d085fbf` to `af86742`

rust/sys: Also bump semver for this

Since we bumped the glib major.

build(deps): bump composefs from `d085fbf` to `af86742`

Bumps [composefs](https://github.com/containers/composefs) from `d085fbf` to `af86742`.
- [Release notes](https://github.com/containers/composefs/releases)
- [Commits](https://github.com/containers/composefs/compare/d085fbf6c8364f3781de7988b13c0f6dbf9e975b...af867425799db92c92d35feb51d2c957f3e16ac7)

---
updated-dependencies:
- dependency-name: composefs
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>

Merge pull request #3028 from ostreedev/dependabot/submodules/composefs-d085fbf

build(deps): bump composefs from `597a766` to `d085fbf`

Merge pull request #3029 from cgwalters/bump-glib

rust: Port to glib 0.18

rust: Bump semver to 0.19

- New glib 0.18 dependency
- Drop cap-std APIs and features
- Other minor changes

rust: Port to glib 0.18

- Also rolls up a few other changes in the C API
- Drop `&mut` from `OstreeKernelArgs`; it now confuses type inference.
  More generally while using `&mut` here was well intentioned, it
  goes against the glib standard of mapping everything to `&` and
  accepting interior mutability.

build(deps): bump composefs from `597a766` to `d085fbf`

Bumps [composefs](https://github.com/containers/composefs) from `597a766` to `d085fbf`.
- [Release notes](https://github.com/containers/composefs/releases)
- [Commits](https://github.com/containers/composefs/compare/597a76609574cf9133d1e17ec0a7b99877a0b709...d085fbf6c8364f3781de7988b13c0f6dbf9e975b)

---
updated-dependencies:
- dependency-name: composefs
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>

Merge pull request #3027 from cgwalters/drop-cap-std-public

Drop cap-std from our public APIs

ci: Move lints into main build

And drop another hardcoded MSRV.

rust: Bump rust-version = 1.70

To pick up the new `AsFd` etc.

Drop cap-std from our public APIs

Since it bumped semver (when I didn't expect it to; xref
https://github.com/bytecodealliance/cap-std/commit/963eebf3ab52b04a2e8b9ba88ce6308bbed5cbd0#r121651362

It's not load-bearing enough here to matter versus just passing
an untyped file descriptor.

This mainly means that it will be the `glib` ecosystem which
forces transitive semver bumps for us, not both.

Merge pull request #3018 from ostreedev/dependabot/submodules/composefs-597a766

build(deps): bump composefs from `1aed878` to `597a766`

Merge pull request #3012 from cgwalters/default-early-prune

sysroot: Promote the "early prune" behavior to default

sysroot: Promote the "early prune" behavior to default

I think we have enough testing for this, let's default it to
on, and change the variable to allow opt-out in case it does
introduce a regression.

Merge pull request #2968 from cgwalters/drop-global-syncfs-by-default

deploy: Remove global `sync` by default

Merge pull request #3024 from cgwalters/clang-analyzer-fixes-5

Clang analyzer fixes 5

build(deps): bump composefs from `1aed878` to `597a766`

Bumps [composefs](https://github.com/containers/composefs) from `1aed878` to `597a766`.
- [Release notes](https://github.com/containers/composefs/releases)
- [Commits](https://github.com/containers/composefs/compare/1aed8781d6ce617234175d5d3b0458f91a6e38f3...597a76609574cf9133d1e17ec0a7b99877a0b709)

---
updated-dependencies:
- dependency-name: composefs
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>

Merge pull request #3023 from ostreedev/dependabot/submodules/libglnx-54ad67d

build(deps): bump libglnx from `c02eb59` to `54ad67d`

commit: Quiet clang-analyzer warning

Another conditional ownership.

keyfile-utils: Quiet a clang-analyzer warning

It complains that we could leak memory if the return value
pointer isn't set.  That's actually a nonsensical case, there's
no reason to call this and ignore the return value.

So change things to require it be set, and also change the
`g_return_val_if_fail` to be hard assertions per our new policy.

build(deps): bump libglnx from `c02eb59` to `54ad67d`

Bumps libglnx from `c02eb59` to `54ad67d`.

---
updated-dependencies:
- dependency-name: libglnx
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>

Merge pull request #3020 from cgwalters/less-return-if-fail-1

mutable-tree: Change some `g_return_if_fail` to `g_assert()`

Merge pull request #3019 from cgwalters/more-analyzer-fixes-4

mutable-tree: Quiet clang-analyzer warning

mutable-tree: Change some `g_return_if_fail` to `g_assert()`

We did this in some prior patches because it's better for
static analyzers; code is either reachable or not, and we don't
want the default desktop-style behavior of "try to stumble on
without crashing".  It's just likely to lead to a crash somewhere
else.

mutable-tree: Quiet clang-analyzer warning

The warning here was about a use-after-free which is serious, but
I don't think it was reachable in practice, because we'd only
enter the second conditional if `contents_checksum != self->contents_checksum`,
yet we made them the same thing here.

Nevertheless, refactor the code to do a direct return in this
case because that's basically what happened before, and doing
so is a lot clearer.

Merge pull request #3017 from cgwalters/more-analyzer-fixes-3

More analyzer fixes 3

repo: Quiet clang-analyzer warning

Another conditional ownership.

commit: Quiet clang-analyzer warning

Another conditional ownership.

commit: Quiet clang-analyzer warning

Another conditional ownership.

lib/commit: Quiet clang-analyzer warning

Another conditional ownership.

lib/delta: Remove dead code

Found by clang-analyzer.

Merge pull request #3016 from cgwalters/more-analyzer-fixes-2

More analyzer fixes 2

Merge pull request #2994 from cgwalters/refactor-composefs-warnings

Refactor composefs warnings

pull: Quiet clang-analyzer warning

Just used for side-effects.

commit: Quiet clang-analyzer warning

This is just another instance of the "conditionally owned"
pattern.

pull: Add assertions to quiet clang-analyzer

It's warning the value can be NULL, but I don't think that's true.
I believe we're just hitting analysis limit (the path length is 119).

lib/repo-finder: Squash memory leak

We want to free this value on each iteration; it's moved below
if the value is used.

Merge pull request #3013 from cgwalters/more-analyzer-fixes

2023.6 coverity minor fixes

Merge pull request #3006 from cgwalters/misc-c99-style-5

Misc c99 style 5

switchroot: Use g_new/g_free consistently

Coverity complains about this, even though they're the same thing.

sign/ed25519: Fix two memory leaks

Spotted by coverity.

commit: Drop dead code

Coverity points out that since we're now *always* initializing
metadata, the `create_empty_gvariant_dict()` is unreachable.

switchroot: Lower config parser to otcore, add unit tests

Part of the continuation of unit testing coverage.

composefs: Hard error except on ENOENT even in "optional" case

Since we enabled composefs at build time, the default (non-composefs)
case now always prints
`composefs: Optional support failed: No such file or directory`
But that's normal and expected.

Rework things here so that in the very special case where
we are in "maybe/optional" mode and we get ENOENT, then we
output a much more normal-looking message that doesn't include
the string "failed".

Now on the flip side - if I have explicitly enabled signature
checking, I think we *do* want to make that fatal even if
composefs is in "maybe" mode.

(This part is more debatable; perhaps we should just disallow
 the case of "maybe" + signatures at all; but I think this is
 an improvement in that direction)

prepare-root: Fold together composefs signature cases

Now that we don't support digest-but-not-signature verification
for composefs, the logic here was unnecessarily complicated.
With a prior prep patch that moved the composefs option
initialization up, we can just have everything related to signature
verification in a single conditonal.

prepare-root: Init composefs options earlier

Prep for a later patch.

prepare-root: Drop redundant print about signature/digest

We print if we're doing a signature+digest verification; its absence is
sufficient in the other case.  The goal here is to avoid polluting
the logs when signatures are not enabled.

Merge pull request #3005 from cgwalters/release

Release

configure: post-release version bump

Release 2023.6

Merge pull request #3010 from cgwalters/more-composefs-fixes

build-sys: Really fix composefs check

build-sys: Really fix composefs check

The ordering of the includes apparently matters...and I didn't
actually check that the previous change enables composefs on c9s.
But I did now.  For reals.

While we have the patient open, I switched to `AC_LANG_PROGRAM`
because I originally thought the bug had something to do with that.
As far as I understand, more cleanly separating the includes
from the injected body text is a useful thing in `AC_LANG_PROGRAM`.

Merge pull request #3009 from cgwalters/c9s-ci

C9s CI && composefs build detection fix

Merge pull request #3007 from cgwalters/optin-new-bootloader-naming

deploy: Add bootloader-naming-2 opt-in

build-sys: Look for both linux/mount.h and sys/mount.h

Since that's what composefs uses.

ci: Add c9s build

Since it's an active target of this project.

Merge pull request #3008 from ostreedev/maybe_ostree

prepare-root: If composefs is configured as "maybe" don't fail

prepare-root: If composefs is configured as "maybe" don't fail

If composefs is configured as "maybe", we should continue even if
composefs support is not built in.

deploy: Add bootloader-naming-2 opt-init

I've verified that this fixes compatibility with GRUB, which
parses the filename:
https://github.com/ostreedev/ostree/issues/2961

However, out of a large degree of conservatism I've made this
an opt-in behavior for now.

My plan is to test it out in the FCOS development streams first.

Merge pull request #3003 from cgwalters/prepare-root-oscore-and-units

Move prepare-root karg helpers into otcore, add unit tests

cmd/show-remote-url: Port to C99 style

Just keeping up momentum.

cmd/log: Port to C99 style

Just keeping up momentum.

cmd/export: Fold libarchive error handling

The helper function here doesn't quite carry its weight; this
brings us inline with having single line (no braces) error
return paths.

cmd/export: Port to C99 style

Just keeping up momentum.

Move prepare-root karg helpers into otcore, add unit tests

Add long overdue unit testing coverage for this, which
at least slightly closes out the android boot CI gap.

Actually, this *copies* the karg parsing code into otcore because
it now uses glib, which we're not yet using in the static
prepare-root.  It's pretty tempting to drop support for the
static prepare root entirely.  But for now we'll live with some
code duplication.

Merge pull request #2997 from cgwalters/test-cleanups-1

tests/destructive: Port more to xshell

Merge pull request #2999 from cgwalters/add-oscore-units

tests: Add otcore unit tests

Merge pull request #3001 from cgwalters/misc-c99-style-4

Misc c99 style 4

cmd/grub2-generate: Port to C99 style

Just keeping up momentum.

cmd/init: Port to C99 style

Just keeping up momentum.

Merge pull request #3000 from cgwalters/test-inst-update

tests/inst: Update to latest ostree-ext

Merge pull request #2995 from ostreedev/dependabot/submodules/composefs-1aed878

build(deps): bump composefs from `a6e827d` to `1aed878`

Merge pull request #2998 from cgwalters/disable-composefs-too-old

build-sys: Disable composefs on too-old Linux headers

tests/inst: Update to latest ostree-ext

(And glib, etc.)

Merge pull request #2975 from ostreedev/androidboot-single-slot-mode

prepare-root: On a non-A/B androidboot system, boot system slot a

tests: Add otcore unit tests

This just stubs out the basic infrastructure, to be expanded upon.

build-sys: Disable composefs on too-old Linux headers

This should fix the build with Google OSS-fuzz which currently
uses an old Ubuntu.

prepare-root: Changes made to find_proc_cmdline_key

Used strspn based on feedback from similar function.

prepare-root: On a non-A/B androidboot system, boot system slot a

Sometimes android bootloaders boot in a nonab way:

  https://source.android.com/docs/core/ota/nonab

In this case, "androidboot." kargs are present but not
"androidboot.slot_suffix" specifically.

In this case, rather than getting stuck in a partially booted
environment, boot system slot a.

Merge pull request #2996 from cgwalters/misc-c99-style-3

cli/set-origin: Port to C99 style

tests/destructive: Port more to xshell

Just keeping up momentum.

cli/set-origin: Port to C99 style

Just keeping up momentum.

build(deps): bump composefs from `a6e827d` to `1aed878`

Bumps [composefs](https://github.com/containers/composefs) from `a6e827d` to `1aed878`.
- [Release notes](https://github.com/containers/composefs/releases)
- [Commits](https://github.com/containers/composefs/compare/a6e827df2dbebb65d6d19a7211f2fb4a61d989ac...1aed8781d6ce617234175d5d3b0458f91a6e38f3)

---
updated-dependencies:
- dependency-name: composefs
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>

Merge pull request #2993 from cgwalters/misc-c99-style-2

Misc c99 style 2

checkout: Port to C99 style

Just keeping up momentum.

Merge pull request #2990 from cgwalters/init-is-stateroot

Two s/osname/stateroot/ patches

lzma: Port to C99 style

Just keeping up momentum.

Merge pull request #2991 from cgwalters/misc-c99-style

Misc c99 style

remote-add: Port to c99 style

- Drop `goto out`
- Use declare-and-initialize

admin: Port to c99 style

- Drop `goto out`
- Use declare-and-initialize

admin-deploy: Add `--stateroot` as alias for `--os`

To further help deprecate the confusing "osname" terminology.

Add `ostree admin stateroot-init` as alias for `os-init`

To further help deprecate the confusing "osname" terminology.

Merge pull request #2989 from cgwalters/lock-timeout-longer

repo: Bump lock timeout to 5 minutes

Merge pull request #2973 from ostreedev/dependabot/submodules/composefs-a6e827d

build(deps): bump composefs from `1704f82` to `a6e827d`