Skip to content

Commit

Permalink
fix: scripts fail if repo is cloned into folder with whitespaces (DEV…
Browse files Browse the repository at this point in the history
  • Loading branch information
jnussbaum authored Sep 29, 2023
1 parent 21dd570 commit 08d2676
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
6 changes: 3 additions & 3 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ DOT_FIGURES = $(shell find ./ -type f -name '*.dot')
PNG_FIGURES = $(patsubst %.dot,%.dot.png,$(DOT_FIGURES))

THIS_FILE := $(abspath $(lastword $(MAKEFILE_LIST)))
CURRENT_DIR := $(shell dirname $(realpath $(firstword $(MAKEFILE_LIST))))
CURRENT_DIR := $(shell dirname "$(realpath $(firstword $(MAKEFILE_LIST)))")

include release.mk

Expand All @@ -13,7 +13,7 @@ init-submodules: ## init the documentation from each connected repo; this comman

.PHONY: update-submodules
update-submodules: ## grab the current documentation from each connected repo
$(CURRENT_DIR)/update-and-deploy.sh dsp=$(DSP) api=$(API) app=$(APP) tools=$(TOOLS) deploy=false
'$(CURRENT_DIR)/update-and-deploy.sh' dsp=$(DSP) api=$(API) app=$(APP) tools=$(TOOLS) deploy=false

.PHONY: build
build: ## build docs into the local 'site' folder
Expand All @@ -30,7 +30,7 @@ serve: ## serve docs for local viewing
.PHONY: deploy
deploy: ## build and publish docs to Github Pages with versioning from the release.mk file
@$(MAKE) install-requirements
$(CURRENT_DIR)/update-and-deploy.sh dsp=$(DSP) api=$(API) app=$(APP) tools=$(TOOLS) deploy=true
'$(CURRENT_DIR)/update-and-deploy.sh' dsp=$(DSP) api=$(API) app=$(APP) tools=$(TOOLS) deploy=true

.PHONY: install-requirements
install-requirements: ## install requirements
Expand Down
2 changes: 1 addition & 1 deletion update-and-deploy.sh
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,7 @@ for ARGUMENT in "$@"; do
echo $(pwd)
git fetch
git checkout ${VALUE}
cd ${home}
cd "${home}"
git add dsp/dsp-${KEY}
echo $sep
fi
Expand Down

0 comments on commit 08d2676

Please sign in to comment.