From 7fce2733ddb3dbf2ee10c258a19686e5d600d9e6 Mon Sep 17 00:00:00 2001
From: and94x <andreas@muelhau.pt>
Date: Mon, 15 Aug 2022 00:16:43 +0200
Subject: [PATCH] refactored firefox build-file

---
 build-ff.sh | 47 ++++++++++++++++++++++++++++++++---------------
 1 file changed, 32 insertions(+), 15 deletions(-)

diff --git a/build-ff.sh b/build-ff.sh
index 52ceb70..101ab0f 100644
--- a/build-ff.sh
+++ b/build-ff.sh
@@ -1,21 +1,38 @@
-#!/bin/bash
-PWD=$(pwd)
-BASE_DIR=${PWD}/base
-MANIFEST_DIR=${PWD}/firefox
-OUTPUT_DIR=${PWD}/out/artifacts/firefox
+#!/usr/bin/bash
+printf "     _                           _____         \r\n"
+printf " ___| |_ __ _ _ __ ___  _ __ ___/__   \/ \   /\ \r\n"
+printf "/ __| __/ _\` | '_ \` _ \| '_ \` _ \ /  /\/\ \ / / \r\n"
+printf "\__ \ || (_| | | | | | | | | | | / /     \ V /  \r\n"
+printf "|___/\__\__,_|_| |_| |_|_| |_| |_\/       \_/   \r\n"
+
+# Define paths
+BASE_DIR=base
+MANIFEST_DIR=firefox
+OUTPUT_DIR=out/artifacts/firefox
+
 OUTPUT_FILE=${OUTPUT_DIR}/extension.zip
 
-echo $BASE_DIR
-echo $MANIFEST_DIR
+printf "Base dir: \t%s\r\n" "$BASE_DIR"
+printf "Manifest dir: \t%s\r\n" "$MANIFEST_DIR"
+printf "Output dir: \t%s\r\n" "$OUTPUT_DIR"
+printf "\r\n"
 
-mkdir -p ${OUTPUT_DIR}
+# Create output directory, if not existing
+[ ! -d "${OUTPUT_DIR}" ] && mkdir -p "${OUTPUT_DIR}"
 
-# check if file exists before removing
+# Remove previous build, if existing
 [ -f ${OUTPUT_FILE} ] && rm -f ${OUTPUT_FILE}
 
-pushd ${BASE_DIR}
-zip -r ${OUTPUT_FILE} *
-popd
-pushd ${MANIFEST_DIR}
-zip ${OUTPUT_FILE} manifest.json
-popd
+# Add base components to zip
+pushd ${BASE_DIR} 1> /dev/null || exit
+zip -r ../${OUTPUT_FILE} * 1> /dev/null
+popd 1> /dev/null || exit
+
+# Add firefox manifest to zip
+pushd ${MANIFEST_DIR} 1> /dev/null || exit
+zip ../${OUTPUT_FILE} manifest.json 1> /dev/null
+popd 1> /dev/null || exit
+
+# Feedback
+[ -f ${OUTPUT_FILE} ] && echo "✔ Successfully created $(file ${OUTPUT_FILE})"
+[ ! -f ${OUTPUT_FILE} ] && echo "❌ Failed to build extension. See output printed above for more details."