bin/summary: Use ostree_repo_regenerate_metadata

Call `ostree_repo_regenerate_metadata` with the provided GPG and sign
keys to generate and sign the summary in one call. This changes the
handling when GPGME support is disabled but GPG keys are supplied.
Instead of silently ignoring the option, it will now error. IMO that's
better as callers would otherwise not know that the summary is not GPG
signed.

repo: Add API to regenerate and sign repo metadata

Currently this is just a wrapper around regenerating and signing the
summary in one call, but later it will be used to also generate the
`ostree-metadata` commit if the repo has a collection ID.

Strip trailing whitespace on all C files

My editor started following the configuration in .editorconfig and is
applying this rule to many files I'm editing. Let's just get this over
with and strip everything. This was done like so:

  git ls-files | grep '\.[ch]$' | xargs sed -ri 's/\s+$//'

Merge pull request #2795 from cgwalters/fix-docsrs-build

bindings: Use default for uninitialized fields in checkout opts

bindings: Consistently check for `dox` feature

This fixes the build on docs.rs:

https://docs.rs/crate/ostree-ext/0.10.2/builds/699039
```
[INFO] running `Command { std: "docker" "start" "-a" "7d90466f040d987cbcc393bc89f5f0688f052f49e2339c354dd731fdaf0f9667", kill_on_drop: false }`
[INFO] [stderr]     Checking ostree v0.17.0
[INFO] [stderr] error[E0063]: missing field `process_passthrough_whiteouts` in initializer of `repo_checkout_at_options::RepoCheckoutAtOptions`
[INFO] [stderr]   --> /opt/rustwide/cargo-home/registry/src/github.com-1ecc6299db9ec823/ostree-0.17.0/rust-bindings/src/repo_checkout_at_options/mod.rs:61:9
[INFO] [stderr]    |
[INFO] [stderr] 61 |         RepoCheckoutAtOptions {
[INFO] [stderr]    |         ^^^^^^^^^^^^^^^^^^^^^ missing `process_passthrough_whiteouts`
```

Merge pull request #2807 from dbnicholson/hyper-staticfile-yanked

tests/inst: Update hyper-staticfile to 0.9.4

tests/inst: Update hyper-staticfile to 0.9.4

Due to some vulnerabilities[1][2], all versions prior to 0.9.4 have been
yanked from crates.io. There have been some API changes since 0.6.0, but
none of them affect the simple `hyper_staticfile::Static` usage here.

1. https://rustsec.org/advisories/RUSTSEC-2022-0069.html
2. https://rustsec.org/advisories/RUSTSEC-2022-0072.html

Merge pull request #2793 from ericcurtin/aboot-bootloader-support

bootloader: Add an aboot (Android) bootloader backend

Merge pull request #2799 from jeamland/replace-radix64

Replace the radix64 crate with base64

Replace the radix64 crate with base64

The radix64 crate was last updated over 3 years ago. On the other
hand the base64 crate appears to be far more actively maintained,
supports all the needed features and has a few orders of magnitude
more users.

Merge pull request #2798 from akiernan/support-musl-erange

tests: Handle musl's ERANGE mapping

tests: Handle musl's ERANGE mapping

musl uses "Result not representable" for ERANGE, support this in
addition to glibc's "Numerical result out of range".

Signed-off-by: Alex Kiernan <alex.kiernan@gmail.com>

Merge pull request #2797 from akiernan/fix-musl-allperms

lib/repo-checkout: Add ALLPERMS for musl

lib/repo-checkout: Add ALLPERMS for musl

ALLPERMS is glibc specific, add a definition for musl.

Signed-off-by: Alex Kiernan <alex.kiernan@gmail.com>

bootloader: Add an aboot (Android) bootloader backend

aboot is special in that it packages kernel, initrd, cmdline, dtb and
signature one combined image (similar to upcoming unified kernel
images). This is then loaded as an image into an aboot partition.

This image is signed by the OS vendor and covers everything in the
image. So locally on the deployed system it should not be possible to
boot an unsigned image (unless signature checking is turned off).

We call a shell script aboot-deploy when it is required to write a new
image to the aboot partition (a file typically starting with aboot and
ending in .img extension). This shell script may also read some
configurations from a .cfg file.

Signed-off-by: Eric Curtin <ecurtin@redhat.com>

Merge pull request #2766 from cgwalters/avoid-endless-object-queueing

Closes https://github.com/ostreedev/ostree/issues/2732

Merge pull request #2789 from lucab/ups/autoconf-gpg-libs

configure: use pkg-config with newer gpgme and gpg-error

lib/repo: fix an 'unused-function' warning

This silences a warning when building the project with `--with-gpme=no`.

configure: use pkg-config with newer gpgme and gpg-error

This tweaks autoconf logic in order to use pkg-config for gpgme
and gpg-error when available.
Recent versions of gpgme directly provide threaded support, and
gpg-error started shipping a .pc file. Thus on recent distributions
it is possible to directly use pkg-config for both. On older
environments, the legacy logic is kept in place.

Merge pull request #2788 from lucab/ups/lib-assertions

lib: assert more invariants

Merge pull request #2787 from cgwalters/release-rust-bindings

rust: Bump to 0.17

lib: assert more invariants

This converts a few more safety checks into either plain GErrors
or hard assertions.

rust: Bump to 0.17

We switched gio and cap-std versions, so we need to bump
our own semver.

Merge pull request #2786 from cgwalters/release

Release 2022.7

configure: post-release version bump

Release 2022.7

Merge pull request #2784 from cgwalters/ci-build-integration-tests

ci: Add a flow that combines C build and Rust integration tests

Merge pull request #2781 from cgwalters/bump-bindings

ci: Add a flow that combines C build and Rust integration tests

Today it's just the Jenkins flow which builds our Rust
integration tests.  Let's add a flow to the GH actions which
does this, and also runs inside a (privileged) container.

Prep for doing more testing of the ostree-container path here.

Merge pull request #2782 from cgwalters/testinst-updates

Merge pull request #2783 from cgwalters/test-sudo

tests: Require OSTREE_TEST_SUDO to be set to use `sudo`

tests: Require OSTREE_TEST_SUDO to be set to use `sudo`

IMO, tools invoking `sudo` automatically is a huge anti-pattern.
Require this to be opt-in.

In practice also...these tests really want to be *integration*
and not unit tests because involving `sudo` alongside uninstalled
binaries just creates a giant mess.

Closes: https://github.com/ostreedev/ostree/issues/2761

tests/inst: Update sh-inline

tests/inst: Update cap-std-ext to 1.0

tests/inst: Drop nix dependency

It has various semver bumps and we were literally just using
it to duplicate what's already in `libc`.

tests/inst: Update to latest ostree-ext

rust: Regenerate with latest gir and C sources

Merge pull request #2768 from cgwalters/update-gio

rust: Update to gio 0.16

Merge pull request #2779 from cgwalters/sysroot-mountns

sysroot: Add an API to initialize with mountns

sysroot: Add an API to initialize with mountns

This lowers down into the C library some logic we
have in the binary/app logic, in prep for having more Rust-native
CLI code in https://github.com/ostreedev/ostree-rs-ext/pull/412

Basically we want to *ensure* a mount namespace by invoking
`unshare()` if necessary, instead of requiring our callers
to do this dance.

This also helps fix e.g.
Closes: https://github.com/ostreedev/ostree/issues/2769

Merge pull request #2776 from cgwalters/fix-potential-double-unwind

repo: Avoid potential double unwind when writing panic value

Allow missing docs on two `glib::wrapper` functions

This is being triggered in gtk-rs generated code.

repo: Avoid potential double unwind when writing panic value

Thanks to @shinmao for the report!

Closes: https://github.com/ostreedev/ostree/issues/2775

ostree: manually patch generated files

This manually adds a missing `ToGlibPtr` import, which seems to be result
of some bugs in `gir` code-generation.

rust: Update to gio 0.16

This was quite seamless; the only thing I had to tweak was adding
`+ Send` bounds in the `Box<dyn Error>` manual checksum_async API.

Merge pull request #2778 from pwithnall/small-leaks

Fix a couple of small leaks

ot-builtin-pull: Fix a minor leak with progress reporting

`ostree_async_progress_get_status()` returns an allocated string.

Signed-off-by: Philip Withnall <pwithnall@endlessos.org>

lib/pull: Fix a small leak when a variable is reused

Signed-off-by: Philip Withnall <pwithnall@endlessos.org>

Merge pull request #2770 from lucab/ups/git-describe-tags

ci: adjust git tags filtering

Merge pull request #2772 from lucab/ups/bump-msrv

cargo: bump minimum Rust version to 1.63

cargo: bump minimum Rust version to 1.63

This bumps MSRV to 1.63, in order to prepare for the next version
of gtk-rs stack.

Ref: https://gtk-rs.org/blog/2022/10/18/new-release.html

ci: adjust git tags filtering

Another attempt at fixing logic for version detection in COPR.

Merge pull request #2774 from cgwalters/ci-fedora-fix

ci: Fix case where cosa != buildroot

Merge pull request #2777 from Razaloc/wip/rafaelgarrui/deletepresent

Idempotent delete operation for  OstreeKernelArgs

ci: Fix case where cosa != buildroot

We can't override host binaries in this case.  We'll need
to watch carefully and verify we're still testing things.

Idempotent delete operation for  OstreeKernelArgs

ostree_kernel_args_delete_if_present checks if an argument is present
in OstreeKernelArgs and delete it.

Signed-off-by: Rafael Garcia Ruiz <rafael.garcia@collabora.com>

Merge pull request #2773 from pwithnall/typo-fix

Improve a help string for ostree-commit

ot-builtin-commit: Don’t mention a C function name in a help string

Instead mention the formal name of the format accepted by
`g_variant_parse()`.

Signed-off-by: Philip Withnall <pwithnall@endlessos.org>

ot-builtin-commit: Fix a typo in a command line help string

Signed-off-by: Philip Withnall <pwithnall@endlessos.org>

Merge pull request #2771 from Razaloc/wip/rafaelgarrui/contains

ostree_kernel_args_contains for OstreeKernelArgs

ostree_kernel_args_contains for OstreeKernelArgs

Check if an argument is present in OstreeKernelArgs. This is a way to
make easier idempotent append and delete operations.
ostree_kernel_args_append_if_missing uses it to avoid inserting a
duplicate key.

Closes #2329

Signed-off-by: Rafael Garcia Ruiz <rafael.garcia@collabora.com>

fetcher: Avoid too large queues for metadata processing

We added backoff/queueing for fetching via HTTP, but we have
another queue in the metadata scanning which can also grow
up to the number of outstanding objects, which can be large.

Capping the scanning operation when we have hit our operation
limit will avoid potentially large amounts of allocations in the
case of e.g. a slow network.

Closes: https://github.com/ostreedev/ostree/issues/2732

Merge pull request #2765 from cgwalters/drop-openat

rust: Drop `openat` dependency

Merge pull request #2751 from smcv/memdup2

Replace calls to g_memdup() with g_memdup2()

rust: Drop `openat` dependency

We can use cap-std in our tests.

Merge pull request #2764 from cgwalters/bump-cap-std

rust: Update to cap-std 1.0

Replace calls to g_memdup() with g_memdup2()

g_memdup() is subject to an integer overflow on 64-bit machines if the
object being copied is larger than UINT_MAX bytes. I suspect none of
these objects can actually be that large in practice, but it's easier
to replace all the calls than it is to assess whether we need to
replace them.

A backport in libglnx is used on systems where GLib is older than 2.68.x.

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

rust: Update to cap-std 1.0

This was one source of our semver bumps; let's switch to 1.0.

Merge pull request #2763 from cgwalters/testiso-more-memory

ci: Bump memory for ISO testing flow

ci: Bump memory for ISO testing flow

It seems likely that we're allocating more RAM here.  What we really
need to do is for cosa to express the necessary RAM
requirements declaratively, then we compute pod requirements
from that.

Merge pull request #2762 from lucab/ups/copr-tags

copr: only use libostree tags

copr: only use libostree tags

This adds a tag filter to the logic which emits version labels for
COPR build, so that it avoids mistakenly picking up tags belonging
to the Rust bindings.

Merge pull request #2760 from cgwalters/offline-deploy-no-bwrap

deploy: Don't rebuild selinux policy on first deployment

Merge pull request #2754 from cgwalters/remove-readdir-rand

Remove readdir-rand

Merge pull request #2759 from oglok/http_500

Enabling retry for HTTP 500 internal server error

deploy: Don't rebuild selinux policy on first deployment

Basically, it should not be necessary - the policy should be
up-to-date.  We don't want to force on continual policy rebuilds.

Even trying to run bwrap when we're *not* in a booted
root can cause failures in nested containerization scenarios.

Closes: https://github.com/ostreedev/ostree/issues/2758

Add tests for cases under random 500s

Signed-off-by: Ricardo Noriega <rnoriega@redhat.com>

Enabling retry for HTTP 500 internal server error

Signed-off-by: Ricardo Noriega <rnoriega@redhat.com>
Use G_IO_ERROR_BUSY to represent 500 error

Signed-off-by: Ricardo Noriega <rnoriega@redhat.com>
Make G_IO_ERROR_BUSY independent from glib version

Signed-off-by: Ricardo Noriega <rnoriega@redhat.com>

Merge pull request #2756 from ostreedev/dependabot/submodules/libglnx-4e44fd9

build(deps): bump libglnx from `e701578` to `4e44fd9`

build(deps): bump libglnx from `e701578` to `4e44fd9`

Bumps libglnx from `e701578` to `4e44fd9`.

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

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

Remove readdir-rand

This was only there to reproduce a bug we hit long ago
with bootloader file ordering.  We're extremely unlikely
to reintroduce such a bug, and it's not worth carrying around
this code.

Merge pull request #2752 from smcv/copy-dirent

readdir-rand: Copy full size of struct dirent

readdir-rand: Copy full size of struct dirent

As noted in readdir(3), in the presence of long filenames it is
possible for a directory entry to be larger than `sizeof (struct dirent)`.
Copy the full length instead.

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

Merge pull request #2749 from lucab/ups/cargo-releases

cargo: prepare ostree 0.16.0 and ostree-sys 0.11.0

cargo: prepare ostree 0.16.0 and ostree-sys 0.11.0

This prepares for a new release of both crates, now using the
gtk-rs 0.15 stack.

Merge pull request #2748 from lucab/ups/ci-rust-bump-libostree-git

ci/rust: test from git using latest feature

ci/rust: test from git using latest feature

Merge pull request #2743 from lucab/ups/gtk-rs-0.15

rust-bindings: regenerate for gtk-rs 0.15

rust/ostree: regenerate for gtk-rs 0.15

rust/ostree-sys: regenerate for gtk-rs 0.15

rust: update pinned gir tool to 0.15

Merge pull request #2742 from lucab/ups/rust-regenerate-2022.6

rust: regenerate bindings after latest release (2022.6)

Merge pull request #2745 from lucab/ups/ci-bump-rust-linting

ci/rust: bump linting toolchain to latest stable (1.64)

Merge pull request #2746 from lucab/ups/lib-repo-refs-error

lib/repo-refs: properly return an error value

lib/repo-refs: properly return an error value

This tweaks a logic check in order to return a proper error value
when the related condition is not satisfied.

Merge pull request #2740 from lucab/ups/lib-sign-ed25519-assertions

lib/sign-ed25519: convert invariant checks to assertions

ci/rust: bump linting toolchain to latest stable (1.64)

This bumps the Rust toolchain for clippy/rustfmt to 1.64.

rust: update manual helpers

ostree: manually patch generated files

This manually adds a missing `ToGlibPtr` import, which seems to be result
of some bugs in `gir` code-generation.

rust: regenerate bindings after latest release (2022.6)