ci: specify Linux Rust toolchain versions to build with

- test builds: minimum supported version (1.74)
- release builds: explicit most recent version for which a buster
  docker image exists (1.79)
- code checks: latest

Merge branch 'cargo-lock' into 'main'

Cargo.lock: update outdated deps

See merge request xen-project/xen-guest-agent!85

Cargo.lock: update outdated deps

Signed-off-by: Yann Dirson <yann.dirson@vates.tech>

Merge branch 'cargo-lock' into 'main'

Cargo.lock: update outdated deps

See merge request xen-project/xen-guest-agent!84

Cargo.lock: update outdated deps

Signed-off-by: Yann Dirson <yann.dirson@vates.tech>

Merge branch 'cargo-lock' into 'main'

Cargo.lock: update outdated deps

See merge request xen-project/xen-guest-agent!83

Cargo.lock: update clap to latest, now requiring Rust 1.74 to build

Not changing package.rust-version in Cargo.toml, since there is nothing in
this package source code that actually requires it, besides the (generated)
Cargo.lock.

Signed-off-by: Yann Dirson <yann.dirson@vates.tech>

Cargo.lock: update outdated deps (except for clap)

Clap 4.5 pulls MSRV from 1.71 up to 1.74, so we still keep it out for
now.

Signed-off-by: Yann Dirson <yann.dirson@vates.tech>

Merge branch 'rpm-enable-service' into 'main'

rpm: enable and start service on first install

See merge request xen-project/xen-guest-agent!78

Merge branch 'cargo-lock' into 'main'

Cargo.lock: update outdated deps (except for clap)

See merge request xen-project/xen-guest-agent!80

Cargo.lock: update outdated deps (except for clap)

Clap 4.5 pulls MSRV from 1.71 up to 1.74, so we still keep it out for
now.

Signed-off-by: Yann Dirson <yann.dirson@vates.tech>

freebsd-netlink: use a libc patch based on 0.2.153

For some reason Cargo, accepts on current main commit to override
libc-0.2.153 with a patch advertising 0.2.152 (not even sure why that was
possible).  After cargo-updating it would not any more, so fix the mismatch.

Signed-off-by: Yann Dirson <yann.dirson@vates.tech>

Merge branch 'os_info-3.8' into 'main'

Build against os_info 3.8.2, detects Alma and Rocky (#13)

See merge request xen-project/xen-guest-agent!81

Build against os_info 3.8.2, detects Alma and Rocky (#13)

Not bumping the version in Cargo.toml, though, since there is not no
requirement at the code level.

Signed-off-by: Yann Dirson <yann.dirson@vates.tech>

Merge branch 'fix-deb-build' into 'main'

deb: don't attempt to run dwz on release binaries

See merge request xen-project/xen-guest-agent!82

deb: don't attempt to run dwz on release binaries

Apparently older rust would produce a debug info section in release
binaries, but now they make dwz in Debian 10 barf.

Signed-off-by: Yann Dirson <yann.dirson@vates.tech>

Merge branch 'cargo-lock' into 'main'

Cargo.lock: update outdated deps (except for clap)

See merge request xen-project/xen-guest-agent!79

Cargo.lock: update outdated deps (except for clap)

This finally updates to libc 0.2.153, as we had to rebase our
freebsd-libc0.2 branch to keep it mergeable.

Clap 4.5 pulls MSRV from 1.71 up to 1.74, keep it out for now.

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

rpm: declare Obsoletes: xe-guest-utilities to auto-uninstall it

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

Merge branch 'cargo-lock' into 'main'

Dependency updates

See merge request xen-project/xen-guest-agent!76

rpm: enable and start service on first install

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

Cargo.lock: update outdated deps (except for libc)

Nothing useful for us in libc 0.2.153, and updating it would require
rebasing our freebsd-libc0.2 branch.

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

cargo.lock: bump rtnetlink to 0.14.1, includes FreeBSD support

For FreeBSD support this should indeed be a "rtnetlink (0.14.1)" dep in
Cargo.toml, but strictly speaking there is no dependency in the code itself.

Nevertheless, we can now stop using our PR branch for this crate, and only
the libc crate still requires a patch.

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

netlink: declare compatibility with netlink-packet-route 0.19

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

Merge branch 'ci-adjustments' into 'main'

ci improvements: catch missing Cargo.lock updates, simplify job graph

See merge request xen-project/xen-guest-agent!77

ci: separate build checks from quick check to avoid wasting CI cycles

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

ci: avoid pkg-source fetching artifacts from binary jobs

Having this fetching is awkward, rven if they are not supposed to be used.
Let's just not wait for those jobs.

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

ci: new check job for missing Cargo.lock updates.

This could have been a larger check to make sure build introduces no
diffs, but our reliance on a crate patch for FreeBSD makes this too
messy.  At least the common case of forgetting Cargo.lock will fail
the pipeline early.

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

Merge branch 'build-reqs' into 'main'

Document build requirements and dig compat issues with Rust 1.70

See merge request xen-project/xen-guest-agent!75

Use xenstore-rs 0.7, FreeBSD build now easier with pkgconfig

Cargo: bump MSRV to 1.71

Our dependency `env_logger` suffers from a bug in versions of Cargo
older than 1.71, fixed by https://github.com/rust-lang/cargo/pull/12130
(the one we suffered from before the "Cargo: don't let feature
"static" pull xenstore-rs unless "xenstore" is set" fix).

Note that some of our dependencies already required 1.70 or later.

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

README: better documentation for build requirements

Cargo: don't let feature "static" pull xenstore-rs unless "xenstore" is set

The syntax used was equivalent to "dep:xenstore-rs/static", and a bug in
Rust 1.70 (fixed by https://github.com/rust-lang/cargo/pull/12130 in 1.71)
was triggered.

Merge branch 'replace-xe-guest-utilities-latest' into 'main'

rpm: replace xe-guest-utilities-latest RPM as well

See merge request xen-project/xen-guest-agent!74

rpm: replace xe-guest-utilities-latest RPM as well

Merge branch 'post-release-0.4.0' into 'main'

Bump version to 0.5.0-dev

See merge request xen-project/xen-guest-agent!70

Bump version to 0.5.0-dev

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

ci/everycommit: adjust "git status" call to investigate build failures

This is a more useful version of of 9b16ce355debd105b65bb841b300e585e80aefad.

This allows to diagnose errors like forgetting to update Cargo.lock when
Cargo.toml is updated, as errors is reported by git-rebase before a second
`--exec` can be run.

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

Merge branch 'ci-everycommit-debug' into 'main'

ci/everycommit: add a "git status" call to investigate build failures

See merge request xen-project/xen-guest-agent!73

ci/everycommit: add a "git status" call to investigate build failures

Errors reported by git-rebase actually happen after a first build was run,
which seems to point to the build modifying source files.

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

Merge branch 'ci-fix-modifications-blocking-rebase' into 'main'

ci: cleanup after jobs making changes to source files

See merge request xen-project/xen-guest-agent!72

ci: cleanup after jobs making changes to source files

As a followup to 09aa5bc275fc023aa55c7491f2ab7e8e60bf1935, we can now see
some everycommit jobs failing to run rebase because of modified files.

This makes sure the few jobs touching source files cleanup after doing
their jobs.

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

Merge branch 'ci-fix-rebase-in-progress' into 'main'

ci: cleanup after failing build-linux-everycommit jobs

See merge request xen-project/xen-guest-agent!71

ci: cleanup after failing build-linux-everycommit jobs

Those jobs want to start a rebase, but that cannot work when a rebase is
already in progress... which can happen since we use the "fetch" Git
Strategy (now in Gitlab project CI config), and when such a job fails.
We must cleanup things after

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

Merge branch 'release-0.4.0' into 'main'

Release 0.4.0

See merge request xen-project/xen-guest-agent!69

Release version 0.4.0

release.md: don't miss CHANGELOG

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

Merge branch 'cargo-lock' into 'main'

Cargo.lock: update outdated deps

See merge request xen-project/xen-guest-agent!68

Cargo.lock: update outdated deps

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

Merge branch 'static-libxenstore' into 'main'

Statically link libxenstore

See merge request xen-project/xen-guest-agent!55

New "static" feature for libxenstore static linking

Now build linux release exe as static.

freebsd static-build configs disabled because apparently libxenstore.a
would need to be built with -fPIE.

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

Merge branch 'ci-fix-cargo-version-parsing' into 'main'

ci: more robust parsing of Cargo.toml for UPSTREAMVERSION

See merge request xen-project/xen-guest-agent!66

ci: more robust parsing of Cargo.toml for UPSTREAMVERSION

Adding a dependency as a block brings more matching lines for the
previous regexp.

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

Merge branch 'ci-fix-freebsd-netlink' into 'main'

ci: workaround inability to use specify a feature in a cargo patch

See merge request xen-project/xen-guest-agent!67

ci: workaround inability to use specify a feature in a cargo patch

For some undocumented reason we're not allowed to make use of features
added by the patched dependency we want to use.

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

Merge branch 'cargo-lock' into 'main'

Cargo.lock: update outdated deps

See merge request xen-project/xen-guest-agent!65

Cargo: relax dep on env_logger to accept 0.11

We don't need anything beyond the basic API, unlikely to break even on
new "major" releases.

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

Cargo.lock: update outdated deps

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

Merge branch 'static-bugfixes' into 'main'

ci: add missing -L flag

See merge request xen-project/xen-guest-agent!64

ci: add missing -L flag

For some reason neither cross-rs nor pkg-config tells rustc that cross
libs do live in the sysroot.  Only a problem for static linking, since
that's the only use-case we have besides dlopen.

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

Merge branch 'cargo-lock' into 'main'

Cargo.lock: update outdated deps

See merge request xen-project/xen-guest-agent!63

Cargo.lock: update outdated deps

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

freebsd/netlink: use new libc branch following 0.2

Update of crate rustix now wants libc 0.2.152, which lives on the
libc-0.2 branch while the main libc branch sails towards 0.3.  To be
able to continue building until the stuff we need gets released as
0.3, without causing our PRs to become unmergeable, a new
freebsd-netlink-0.2 was necessary.

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

Merge branch 'ci-freebsd-netlink' into 'main'

ci: add a build job for freebsd13 with netlink support

See merge request xen-project/xen-guest-agent!62

Drop old devscripts/freebsd-fetch-package.sh

Since 88d4faf7764dc581700c5192b76cf85a450add95 and
7caebf0cd73e5bb6c0ddc0d111a377bb44d49186 there is not reason to keep
this script any more.

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

ci: add a build job for freebsd13 with netlink support

Still waiting on the libc PR to land, so we need unofficial branches for
now.

We don't make it a release job because of the unofficial status of those
branches, but we still provide the executable as an artifact for
convenience.

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

Merge branch 'ci-code-checks' into 'main'

Add CI jobs for code checks

See merge request xen-project/xen-guest-agent!61

Apply a few select formatting fixes from fmt

Most of the rest mostly hampers readability.

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

ci: new check job for clippy

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

ci: separate test-builds from release-builds, as being "just checks"

We'll want to have more checks, and they will naturally fit in that new
stage, rather than delaying test builds with a separate "check" stage.

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

Apply suggestions from clippy for code simplification

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

ci: make sure to update freebsd package list before trying to install

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

Merge branch 'doc-cleanup-vif' into 'main'

Document removal of stale network info

See merge request xen-project/xen-guest-agent!60

Document removal of stale network info

Merge branch 'gtn-cleanup-vif' into 'main'

Removing all vif entries from xenstore at startup #12

Closes #12

See merge request xen-project/xen-guest-agent!58

Removing all vif entries from xenstore at startup

When interfaces reported by Xenstore are suppressed while the service is
not running, these data remain. To avoid this we start by removing all
entries from previous agent. Entries that are still valid will be
repopulated immediatly.

Fixes: #12

Merge branch 'freebsd-12-eol' into 'main'

ci: restore FreeBSD builds, now with 13.2

See merge request xen-project/xen-guest-agent!59

Revert "ci: ignore FreeBSD build failures until new container is ready"

Now that the FreeBSD jbs can run again...

This reverts commit 9619b77b712e35afb3ed752b9c560bbca08da294.

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

ci: use official package-installing scripts for freebsd

Note that devscripts/freebsd-fetch-package.sh was written for the CI,
and never worked from Cross.toml.  Switch everything to the
slightly-improved-and-still-undocumented version of the scripts
shipped in the container itself.

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

ci: switch away from EOL'd FreeBSD 12.4 to 13.2

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

Merge branch 'cargo-lock' into 'main'

Cargo.lock: update outdated deps

See merge request xen-project/xen-guest-agent!57

Cargo.lock: update outdated deps

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

Merge branch 'readme-update' into 'main'

Update README with what's next

See merge request xen-project/xen-guest-agent!54

Merge branch 'freebsd-ignore-errors' into 'main'

ci: ignore FreeBSD build failures until new container is ready

See merge request xen-project/xen-guest-agent!56

ci: ignore FreeBSD build failures until new container is ready

Updating the cross-rs container needs some time, let's not stop
things here just because that work is not ready yet.

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

Update README with what's next

Signed-off-by: Olivier Lambert <olivier.lambert@vates.fr>

Merge branch 'cargo-lock' into 'main'

Cargo.lock: update outdated deps

See merge request xen-project/xen-guest-agent!53

Cargo.lock: update outdated deps

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

Merge branch 'ci-cross-aarch64' into 'main'

WIP ci: cross-build Linux aarch64 binary

See merge request xen-project/xen-guest-agent!52

ci: cross-build Linux aarch64 binary

Testbuilding only, as we don't have an obvious use-case to support.

Builds against glibc in a Debian chroot to be able to check
the xenstore build feature.

ci: more consistency on parallel specification

Merge branch 'ci-images-full-path' into 'main'

ci: use full path for container images

See merge request xen-project/xen-guest-agent!50

Merge branch 'ci-cross-windows' into 'main'

Windows CI support

See merge request xen-project/xen-guest-agent!51

ci: cross-build Windows binary

Mostly a PoC, since we're missing crucial bits:
- Xenstore works differently on WIndows, and we don't have a crate
  supporting it yet
- pnet will not build with mingw, needs msvc, and it seems there are
  things to be done before we can use that

Use panic!() not abort!()

Hopefully we won't really need that safeguard, but it's still nice to be
able to compile non non-unix platforms.

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

ci: use full path for container images

Much easier to copy-paste them on a podman command-line, alias files
are not always up to date.

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

Merge branch 'ci-no-scheduled-pkg-source' into 'main'

ci: don't run source packaging when scheduled

See merge request xen-project/xen-guest-agent!49

ci: don't run source packaging when scheduled

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>

Merge branch 'scheduled-freebsd' into 'main'

ci/freebsd: use proper before_script template

See merge request xen-project/xen-guest-agent!48

ci/freebsd: use proper before_script template

Fixes 27fb1ae87b512efc2c55657108fa27531a26f3ea, should really
investigate why I was not able to test this scheduled job on the PR
first <sigh>.

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>