asahi-cosmic: add gvfs-mtp package

.gitea/workflows/build.yaml

@@ -49,6 +49,10 @@ jobs:
         run: |
           sudo -E ./builder.sh "${IMAGE}" "${VERSION}"
 
+      - name: Generate changelog
+        run: |
+          ./changelog.sh "${IMAGE}" "${VERSION}"
+
       - name: Build and push OCI image from rootfs
         run: |
           set -xeuo pipefail
@@ -95,10 +99,6 @@ jobs:
             containers-storage:localhost/${IMAGE}:${full_tag} \
             docker://${REGISTRY}/${IMAGE}:${VERSION}
 
-      - name: Generate changelog
-        run: |
-          ./changelog.sh "${IMAGE}" "${VERSION}"
-
       - name: Commit and push changelog
         if: github.ref == 'refs/heads/main'
         run: |

changelog.sh

@@ -1,11 +1,10 @@
 #!/usr/bin/env bash
-set -euo pipefail
+set -euox pipefail
 
 IMAGE="$1"
 VERSION="$2"
 
-# Convert to absolute path
+ROOTFS="${GITHUB_WORKSPACE}/images/${IMAGE}/rootfs"
-ROOTFS="$(realpath "images/${IMAGE}/rootfs")"
 CHANGELOG_DIR="changelogs/${IMAGE}"
 
 mkdir -p "${CHANGELOG_DIR}"
@@ -15,37 +14,65 @@ PKG_PREV="${CHANGELOG_DIR}/packages-latest.txt"
 BUILD_ID="$(cat .buildid)"
 CHANGELOG_FILE="${CHANGELOG_DIR}/${VERSION}.${BUILD_ID}.json"
 
-# Extract package list (name + evr)
+# Ensure rpmdb exists
+if [[ ! -f "${ROOTFS}/usr/lib/sysimage/rpm/Packages" ]]; then
+    echo "WARNING: rpmdb missing in rootfs (${ROOTFS})"
+    echo "Skipping changelog generation."
+    exit 0
+fi
+
+# Extract package list
 rpm -qa --root "${ROOTFS}" --qf '%{NAME} %{EVR}\n' | sort > "${PKG_CUR}"
 
+# Prepare safe temp files
+CUR_NAMES_FILE="$(mktemp)"
+PREV_NAMES_FILE="$(mktemp)"
+
+cut -d' ' -f1 "${PKG_CUR}" | sort > "${CUR_NAMES_FILE}"
+
 if [[ -f "${PKG_PREV}" ]]; then
-    CUR_NAMES="$(cut -d' ' -f1 "${PKG_CUR}" | sort)"
+    cut -d' ' -f1 "${PKG_PREV}" | sort > "${PREV_NAMES_FILE}"
-    PREV_NAMES="$(cut -d' ' -f1 "${PKG_PREV}" | sort)"
+else
+    cp "${CUR_NAMES_FILE}" "${PREV_NAMES_FILE}"
+fi
 
-    ADDED_NAMES="$(comm -13 <(echo "${PREV_NAMES}") <(echo "${CUR_NAMES}"))"
+# Safe comm operations
-    REMOVED_NAMES="$(comm -23 <(echo "${PREV_NAMES}") <(echo "${CUR_NAMES}"))"
+ADDED_NAMES="$(comm -13 "${PREV_NAMES_FILE}" "${CUR_NAMES_FILE}" || true)"
-    COMMON_NAMES="$(comm -12 <(echo "${PREV_NAMES}") <(echo "${CUR_NAMES}"))"
+REMOVED_NAMES="$(comm -23 "${PREV_NAMES_FILE}" "${CUR_NAMES_FILE}" || true)"
+COMMON_NAMES="$(comm -12 "${PREV_NAMES_FILE}" "${CUR_NAMES_FILE}" || true)"
 
+# Helper: get version from file using awk (exact match)
+get_ver() {
+    local pkg="$1"
+    local file="$2"
+    awk -v p="$pkg" '$1 == p {print $2}' "$file"
+}
+
+# Build lists
 ADDED_LIST=()
 while read -r name; do
     [[ -z "$name" ]] && continue
-        ver="$(grep -E "^${name} " "${PKG_CUR}" | awk '{print $2}')"
+    ver="$(get_ver "$name" "$PKG_CUR")"
     ADDED_LIST+=("${name} ${ver}")
 done <<< "${ADDED_NAMES}"
 
 REMOVED_LIST=()
 while read -r name; do
     [[ -z "$name" ]] && continue
-        ver="$(grep -E "^${name} " "${PKG_PREV}" | awk '{print $2}')"
+    ver="$(get_ver "$name" "$PKG_PREV")"
     REMOVED_LIST+=("${name} ${ver}")
 done <<< "${REMOVED_NAMES}"
 
 UPDATED_LIST=()
 while read -r name; do
     [[ -z "$name" ]] && continue
-        old_ver="$(grep -E "^${name} " "${PKG_PREV}" | awk '{print $2}')"
+    old_ver="$(get_ver "$name" "$PKG_PREV")"
-        new_ver="$(grep -E "^${name} " "${PKG_CUR}" | awk '{print $2}')"
+    new_ver="$(get_ver "$name" "$PKG_CUR")"
-        if [[ "${old_ver}" != "${new_ver}" ]]; then
+
+    # Skip if either version is missing
+    [[ -z "$old_ver" || -z "$new_ver" ]] && continue
+
+    if [[ "$old_ver" != "$new_ver" ]]; then
         UPDATED_LIST+=("${name} ${old_ver} -> ${new_ver}")
     fi
 done <<< "${COMMON_NAMES}"
@@ -54,6 +81,7 @@ if [[ -f "${PKG_PREV}" ]]; then
 REMOVED_COUNT="${#REMOVED_LIST[@]}"
 UPDATED_COUNT="${#UPDATED_LIST[@]}"
 
+# Write JSON
 {
   echo "{"
   echo "  \"image\": \"${IMAGE}\","
@@ -83,34 +111,6 @@ if [[ -f "${PKG_PREV}" ]]; then
   echo "}"
 } > "${CHANGELOG_FILE}"
 
-else
-    ADDED_LIST=()
-    while read -r name ver; do
-        ADDED_LIST+=("${name} ${ver}")
-    done < "${PKG_CUR}"
-
-    ADDED_COUNT="${#ADDED_LIST[@]}"
-
-    {
-      echo "{"
-      echo "  \"image\": \"${IMAGE}\","
-      echo "  \"version\": \"${VERSION}\","
-      echo "  \"build_id\": \"${BUILD_ID}\","
-      echo "  \"added_count\": ${ADDED_COUNT},"
-      echo "  \"removed_count\": 0,"
-      echo "  \"updated_count\": 0,"
-      echo "  \"added\": ["
-      for i in "${!ADDED_LIST[@]}"; do
-        sep=$([[ $i -lt $((ADDED_COUNT-1)) ]] && echo "," || echo "")
-        printf '    "%s"%s\n' "${ADDED_LIST[$i]}" "${sep}"
-      done
-      echo "  ],"
-      echo "  \"removed\": [],"
-      echo "  \"updated\": []"
-      echo "}"
-    } > "${CHANGELOG_FILE}"
-fi
-
 mv "${PKG_CUR}" "${PKG_PREV}"
 
 echo "Changelog written to ${CHANGELOG_FILE}"

images/asahi-cosmic/packages.yaml

@@ -25,6 +25,7 @@ packages:
   - plymouth-system-theme
   - system-config-printer
   - xdg-desktop-portal-gtk
+  - gvfs-mtp
  # Ensure that we have a kernel. Kernel packages are not in any comps group
   # - kernel
   # - kernel-modules