Refactor binding Makefiles for Windows

Author: ingydotnet

common/binding.mk

@@ -1,5 +1,9 @@
 BINDING-NAME := $(shell basename $(shell pwd))
 
+ifeq ($(OS-NAME),windows)
+  override export PATH := "$(LIBYS)/lib:$$PATH"
+endif
+
 release:: release-deps
 	$(ROOT)/util/release release-$(BINDING-NAME)
 

fortran/Makefile

@@ -10,10 +10,6 @@ DEPS := \
   $(FPM) \
   $(LIBYAMLSTAR-SO) \
 
-ifeq ($(OS-NAME),windows)
-  override export PATH := "$(LIBYS)/lib:$$PATH"
-endif
-
 
 test: $(DEPS)
 	fpm test --flag "-L$(LIBYS)/lib"

go/Makefile

@@ -13,23 +13,12 @@ export CGO_LDFLAGS := -L $(ROOT)/libyamlstar/lib
 endif
 
 
-ifeq ($(OS-NAME),windows)
-build: $(GO) $(LIBYAMLSTAR-SO-VERSION) $(LIBYAMLSTAR-HEADER)
-	PATH="$(LIBYS)/lib:$$PATH" go build
-else
 build: $(GO) $(LIBYAMLSTAR-SO-VERSION) $(LIBYAMLSTAR-HEADER)
 	go build
-endif
 
-ifeq ($(OS-NAME),windows)
-test: $(GO) $(LIBYAMLSTAR-SO-VERSION) $(LIBYAMLSTAR-HEADER)
-	PATH="$(LIBYS)/lib:$$PATH" go test$(if $v, -v)
-	chmod -R u+w $(GOPATH) 2>/dev/null || true
-else
 test: $(GO) $(LIBYAMLSTAR-SO-VERSION) $(LIBYAMLSTAR-HEADER)
 	go test$(if $v, -v)
 	chmod -R u+w $(GOPATH) 2>/dev/null || true
-endif
 
 check: $(GO)
 	go vet

nodejs/Makefile

@@ -13,13 +13,8 @@ MAKES-REALCLEAN := \
   $(NODE-MODULES) \
 
 
-ifeq ($(OS-NAME),windows)
-test: $(NODE-MODULES) $(LIBYAMLSTAR-SO)
-	PATH="$(LIBYS)/lib:$$PATH" node test/test.js
-else
 test: $(NODE-MODULES) $(LIBYAMLSTAR-SO)
 	node test/test.js
-endif
 
 build: $(LIBYAMLSTAR-SO)
 

python/Makefile

@@ -17,13 +17,8 @@ PYTHON-VENV-SETUP := pip install pytest setuptools twine
 
 test: test-pytest
 
-ifeq ($(OS-NAME),windows)
-test-pytest: $(LIBYAMLSTAR-SO) $(PYTHON-VENV)
-	PATH="$(LIBYS)/lib:$$PATH" $(VENV) && pytest test/
-else
 test-pytest: $(LIBYAMLSTAR-SO) $(PYTHON-VENV)
 	$(VENV) && pytest test/
-endif
 
 build: $(LIBYAMLSTAR-SO)
 

rust/Makefile

@@ -8,13 +8,8 @@ MAKES-CLEAN := \
   target \
 
 
-ifeq ($(OS-NAME),windows)
-test: $(CARGO) $(LIBYAMLSTAR-SO)
-	PATH="$(LIBYS)/lib:$$PATH" cargo $@
-else
 test: $(CARGO) $(LIBYAMLSTAR-SO)
 	cargo $@
-endif
 
 build check clippy fmt: $(CARGO)
 	cargo $@