Skip to content

Commit

Permalink
Revert changes in favour of @bedroge's suggestions
Browse files Browse the repository at this point in the history
  • Loading branch information
Neves-P committed Dec 2, 2024
1 parent 857a339 commit 13f5479
Showing 1 changed file with 1 addition and 15 deletions.
16 changes: 1 addition & 15 deletions scripts/ingest-tarball.sh
Original file line number Diff line number Diff line change
Expand Up @@ -143,13 +143,6 @@ function cvmfs_ingest_tarball() {

function check_os() {
# Check if the operating system directory is correctly set for the contents of the tarball
# Handle longer prefix in dev.eessi.io
if [ "${cvmfs_repo}" = "dev.eessi.io" ]; then
os=$(echo "${tar_first_file}" | cut -d / -f 4)
else
os=$(echo "${tar_first_file}" | cut -d / -f 3)
fi

if [ -z "${os}" ]
then
error "no operating system directory found in the tarball!"
Expand All @@ -162,13 +155,6 @@ function check_os() {

function check_arch() {
# Check if the architecture directory is correctly set for the contents of the tarball
# Handle longer prefix in dev.eessi.io
if [ "${cvmfs_repo}" = "dev.eessi.io" ]; then
arch=$(echo "${tar_first_file}" | cut -d / -f 5)
else
arch=$(echo "${tar_first_file}" | cut -d / -f 4)
fi

if [ -z "${arch}" ]
then
error "no architecture directory found in the tarball!"
Expand Down Expand Up @@ -251,7 +237,7 @@ if [ "$#" -ne 2 ]; then
error "usage: $0 <CVMFS repository name> <gzipped tarball>"
fi

export cvmfs_repo="$1"
cvmfs_repo="$1"
tar_file="$2"

# Check if the CVMFS repository exists
Expand Down

0 comments on commit 13f5479

Please sign in to comment.