Skip to content

Commit

Permalink
ci: Removing tar --owner and --group options and creating seperate st…
Browse files Browse the repository at this point in the history
…ep instead

The --owner and --group options did not work, so trying a separate step instead for setting permissions on the tar file before it is uploaded.

Signed-off-by: Tyler Erickson <tyler.erickson@seagate.com>
  • Loading branch information
vonericsen committed Oct 15, 2024
1 parent a6edfc2 commit 9c39124
Showing 1 changed file with 14 additions and 10 deletions.
24 changes: 14 additions & 10 deletions .github/workflows/meson.yml
Original file line number Diff line number Diff line change
Expand Up @@ -79,7 +79,7 @@ jobs:
cxx: "g++",
release_name: "linux-x86_64-gcc",
release_extension: ".tar.xz",
archive_command: "tar --owner=root --group=root -cvfJ"
archive_command: "tar cvfJ"
}
- {
name: "Ubuntu Clang",
Expand All @@ -88,7 +88,7 @@ jobs:
cxx: "clang++",
release_name: "linux-x86_64",
release_extension: ".tar.xz",
archive_command: "tar --owner=root --group=root -cvfJ"
archive_command: "tar cvfJ"
}
- {
name: "MUSL Cross Compile x86_64",
Expand All @@ -101,7 +101,7 @@ jobs:
cross_compiler_arch: "x86_64",
release_name: "linux-x86_64-portable",
release_extension: ".tar.xz",
archive_command: "tar --owner=root --group=root -cvfJ"
archive_command: "tar cvfJ"
}
- {
name: "MUSL Cross Compile i686",
Expand All @@ -113,7 +113,7 @@ jobs:
cross_compiler_arch: "i686",
release_name: "linux-i686-portable",
release_extension: ".tar.xz",
archive_command: "tar --owner=root --group=root -cvfJ"
archive_command: "tar cvfJ"
}
- {
name: "MUSL Cross Compile aarch64",
Expand All @@ -125,7 +125,7 @@ jobs:
cross_compiler_arch: "aarch64",
release_name: "linux-aarch64-portable",
release_extension: ".tar.xz",
archive_command: "tar --owner=root --group=root -cvfJ"
archive_command: "tar cvfJ"
}
- {
name: "MUSL Cross Compile armv7l",
Expand All @@ -137,7 +137,7 @@ jobs:
cross_compiler_arch: "armv7l",
release_name: "linux-armv7l-portable",
release_extension: ".tar.xz",
archive_command: "tar --owner=root --group=root -cvfJ"
archive_command: "tar cvfJ"
}
- {
name: "MUSL Cross Compile armv6",
Expand All @@ -149,7 +149,7 @@ jobs:
cross_compiler_arch: "armv6",
release_name: "linux-armv6-portable",
release_extension: ".tar.xz",
archive_command: "tar --owner=root --group=root -cvfJ"
archive_command: "tar cvfJ"
}
- {
name: "MUSL Cross Compile armv5l",
Expand All @@ -161,7 +161,7 @@ jobs:
cross_compiler_arch: "armv5l",
release_name: "linux-armv5l-portable",
release_extension: ".tar.xz",
archive_command: "tar --owner=root --group=root -cvfJ"
archive_command: "tar cvfJ"
}
- {
name: "MUSL Cross Compile powerpc64",
Expand All @@ -173,7 +173,7 @@ jobs:
cross_compiler_arch: "powerpc64",
release_name: "linux-powerpc64-portable",
release_extension: ".tar.xz",
archive_command: "tar --owner=root --group=root -cvfJ"
archive_command: "tar cvfJ"
}
- {
name: "MUSL Cross Compile powerpc64le",
Expand All @@ -185,7 +185,7 @@ jobs:
cross_compiler_arch: "powerpc64le",
release_name: "linux-powerpc64le-portable",
release_extension: ".tar.xz",
archive_command: "tar --owner=root --group=root -cvfJ"
archive_command: "tar cvfJ"
}
outputs: #where hashes need to be stored for slsa provenance
#NOTE: Only doing this for builds with "publish_release: true"
Expand Down Expand Up @@ -273,6 +273,10 @@ jobs:
${{ matrix.config.archive_command }} "${DESTDIR}${ARCHIVE_EXT}" $DESTDIR
shell: bash

- name: Set ownership of tar archive to root:root
if: ${{ matrix.config.os != 'windows-latest' }}
run: sudo chown root:root build/"${DESTDIR}${ARCHIVE_EXT}"

# add `GOBIN` to the `PATH` otherwise nfpm in next step can't be found
- uses: actions/setup-go@v5
if: ${{ matrix.config.create_package }}
Expand Down

0 comments on commit 9c39124

Please sign in to comment.