Skip to content

Commit 7f3a5c9

Browse files
XeSam Dodrill
authored andcommitted
fix(Makefile): remove bashism in the main Makefile
Mac OSX's make has support for `[[`, Ubuntu's does not. This fix (as suggested and written up by @carmstrong on a comment[1]) removed the bashism in favor of a more compatible syntatic trick. [1]: deis/deis#1380 (comment) Amended to fix a "$$" in place of a "$"
1 parent ce05b21 commit 7f3a5c9

1 file changed

Lines changed: 2 additions & 1 deletion

File tree

Makefile

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -48,7 +48,8 @@ install: check-fleet install-routers install-data-containers
4848
install-data-containers: check-fleet
4949
@$(foreach T, $(DATA_CONTAINER_TEMPLATES), \
5050
UNIT=`basename $(T)` ; \
51-
if [[ `$(FLEETCTL) list-units | grep $$UNIT` ]]; then \
51+
EXISTS=`$(FLEETCTL) list-units | grep $$UNIT` ; \
52+
if [ "$$EXISTS" != "" ]; then \
5253
echo $$UNIT already loaded. Skipping... ; \
5354
else \
5455
cp $(T).template . ; \

0 commit comments

Comments
 (0)