#156 scripts: Update to latest upstream from template repo
Merged a year ago by bcotton. Opened a year ago by jflory7.
Fedora-Council/ jflory7/council-docs update/scripts  into  main

file modified
+21 -15
@@ -3,25 +3,30 @@ 

  image="docker.io/antora/antora"

  cmd="--html-url-extension-style=indexify site.yml"

  

- if [ "$(uname)" == "Darwin" ]; then

+ if uname | grep -iwq darwin; then

      # Running on macOS.

      # Let's assume that the user has the Docker CE installed

      # which doesn't require a root password.

      echo ""

      echo "This build script is using Docker container runtime to run the build in an isolated environment."

      echo ""

-     docker run --rm -it -v $(pwd):/antora $image $cmd

+     docker run --rm -it -v "$(pwd):/antora" "${image}" ${cmd}

  

- elif [ "$(expr substr $(uname -s) 1 5)" == "Linux" ]; then

+ elif uname | grep -iq linux; then

      # Running on Linux.

+     # there isn't an antora/aarch64 container, antora can be installed locally

      # Check whether podman is available, else faill back to docker

      # which requires root.

  

-     if [ -f /usr/bin/podman ]; then

+     if [ -f /usr/local/bin/antora ]; then

+         /usr/local/bin/antora "${cmd}"

+     elif uname -m | grep -iwq aarch64; then

+         echo "no antora/aarch64 container try just \`npm install -g @antora/cli @antora/site-generator-default\`"

+     elif [ -f /usr/bin/podman ]; then

          echo ""

          echo "This build script is using Podman to run the build in an isolated environment."

          echo ""

- 	podman run --rm -it -v $(pwd):/antora:z $image $cmd

+         podman run --rm -it -v "$(pwd):/antora:z" "${image}" ${cmd}

  

      elif [ -f /usr/bin/docker ]; then

          echo ""
@@ -29,18 +34,19 @@ 

          echo ""

  

          if groups | grep -wq "docker"; then

- 	    docker run --rm -it -v $(pwd):/antora:z $image $cmd

- 	else

+             docker run --rm -it -v "$(pwd):/antora:z" "${image}" ${cmd}

+         else

+             echo "You might be asked for your password."

+             echo "You can avoid this by adding your user to the 'docker' group,"

+             echo "but be aware of the security implications."

+             echo "See https://docs.docker.com/install/linux/linux-postinstall/"

              echo ""

-             echo "This build script is using $runtime to run the build in an isolated environment. You might be asked for your password."

-             echo "You can avoid this by adding your user to the 'docker' group, but be aware of the security implications. See https://docs.docker.com/install/linux/linux-postinstall/."

-             echo ""

-             sudo docker run --rm -it -v $(pwd):/antora:z $image $cmd

- 	fi

+             sudo docker run --rm -it -v "$(pwd):/antora:z" "${image}" ${cmd}

+         fi

      else

          echo ""

- 	echo "Error: Container runtime haven't been found on your system. Fix it by:"

- 	echo "$ sudo dnf install podman"

- 	exit 1

+         echo "Error: Container runtime haven't been found on your system. Fix it by:"

+         echo "$ sudo dnf install podman"

+         exit 1

      fi

  fi

file modified
+5 -4
@@ -9,14 +9,15 @@ 

  echo "  * Engineering .... http://localhost:8080/engineering/"

  echo "  * Mindshare ...... http://localhost:8080/mindshare/"

  

- if [ "$(uname)" == "Darwin" ]; then

+ if [ "$(uname)" = "Darwin" ]; then

      # Running on macOS.

      # Let's assume that the user has the Docker CE installed

      # which doesn't require a root password.

-     docker run --rm -v $(pwd):/antora:ro -v $(pwd)/nginx.conf:/etc/nginx/conf.d/default.conf:ro -p 8080:80 nginx

+     docker run --rm -v "$(pwd):/antora:ro" -v "$(pwd)/nginx.conf:/etc/nginx/conf.d/default.conf:ro" -p 8080:80 nginx

  

- elif [ "$(expr substr $(uname -s) 1 5)" == "Linux" ]; then

+ elif [ "$(expr substr "$(uname -s)" 1 5)" = "Linux" ]; then

      # Running on Linux.

      # Fedora Workstation has python3 installed as a default, so using that

-     python3 -m http.server --directory ./public 8080

+     cd ./public

+     python3 -m http.server 8080

  fi

This commit updates the build.sh and preview.sh scripts to the state they are in upstream at pagure.io/fedora-docs/template.

Metadata Update from @jflory7:
- Pull-request tagged with: type - dev tools

a year ago

Pull-Request has been merged by bcotton

a year ago