matrixfurry.com / atomic-xr
XR for Universal Blue and Fedora Atomic Desktops
fork atom

Create Fedora toolbox image based on current host machine version #1

merged opened by kaestralblades.bsky.social targeting main from kaestralblades.bsky.social/atomic-xr: 43-fixes

This pull does three things, in order:

  1. Retrieves $VERSION_ID from /etc/os-release.
  2. Asserts that the $VERSION_ID matches the Fedora versioning pattern, to assure we are pulling a valid Fedora image version.
  3. Finally, uses this $VERSION_ID to create the envision container with the appropriate Fedora image version.

This should resolve issues with version mismatches, such as the recent x264 conflict, and ensure compatibility with future Fedora versions. This should address issue #13.

Labels

None yet.

assignee

None yet.

target

None yet.

component

None yet.

Participants 2
AT URI
at://did:plc:oce3kuj4qnyw5vjdl2afm5mj/sh.tangled.repo.pull/3m5oy2mcngc22
+5 -1
Diff #0
unified split
+5 -1
src/atomic-xr/envision.nu
··· 14 14 ] { 15 15 runtime install xr (if not $no_monado {"monado"}) 16 16 17 - distrobox create envision --no-entry -Y -i registry.fedoraproject.org/fedora-toolbox:42 17 + let VERSION_ID: string = (open -r /etc/os-release | parse "{k}={v}" | str trim -c '"' | where k == VERSION_ID | get v.0) 18 + 19 + if $VERSION_ID !~ '^\d{2}$' { error make { msg: 'VERSION_ID from /etc/os-release does not match Fedora versioning pattern.' } } 20 + 21 + distrobox create envision --no-entry -Y -i registry.fedoraproject.org/fedora-toolbox:($VERSION_ID) 18 22 distrobox enter envision -- sudo dnf install -y nu 19 23 upgrade 20 24 log info "Successfully installed Envision"

History

1 round 1 comment
sign up or login to add to the discussion
kaestralblades.bsky.social submitted #0
1 commit
expand
719c81f2
Added host version checking to ensure dependency match.
expand 1 comment
pull request successfully merged