Improve error handling

LaKabane · Jan 17, 2024 · 4355294 · 4355294
1 parent 14fc1dd
commit 4355294
Showing 1 changed file with 13 additions and 9 deletions.
diff --git a/deb/build.sh b/deb/build.sh
@@ -12,24 +12,28 @@ die () {
 )
 
 (
-    mkdir ../build
-    cd ../build
+    mkdir ../build || die "Failed to create build directory"
+    cd ../build || die "Failed to change to build directory"
     cmake -DCMAKE_BUILD_TYPE=Release -DCMAKE_INSTALL_PREFIX=/usr .. || die "Failed to configure project"
     cmake --build .. || die "Failed to build project"
     cmake --install .. --prefix "../deb/package/" || die "Failed to install project into package directory"
 )
 
-mkdir -p package/DEBIAN/
+mkdir -p package/DEBIAN/ || die "Failed to create meta data directory"
+
 VERSION="$(git describe --long | sed 's/^libtuntap-//')"
-cp control.template package/DEBIAN/control
-sed "s/%VERSION%/$VERSION/" -i package/DEBIAN/control
-sed "s/%ARCHITECTURE%/$(dpkg --print-architecture)/" -i package/DEBIAN/control
+[ "$VERSION" ] || die "Failed to get version string"
+ARCH="$(dpkg --print-architecture)"
+[ "$ARCH" ] || die "Failed to get architecture string"
+
+cp control.template package/DEBIAN/control || die "Failed to copy control template"
+sed "s/%VERSION%/$VERSION/" -i package/DEBIAN/control || die "Failed to replace version in control template"
+sed "s/%ARCHITECTURE%/$ARCH/" -i package/DEBIAN/control || die "Failed to replace architecture in control template"
 
-mkdir -p package/usr/share/doc/libtuntap-dev/
-cp copyright package/usr/share/doc/libtuntap-dev/copyright
+mkdir -p package/usr/share/doc/libtuntap-dev/ || die "Failed to create package meta data directory"
+cp copyright package/usr/share/doc/libtuntap-dev/copyright || die "Failed to copy copyright file"
 
 echo "Building package..."
 dpkg-deb --root-owner-group --build package libtuntap-dev.deb || die "Failed to build package"
 
 echo "Package $(dpkg-deb --show libtuntap-dev.deb | sed "s/\t/ /") successful builded!"
-