Makefile: add test-submodules target

...and remove "build" as dependency of "test" target. No need to waste time linking the
deft-app exe when we run tests.

.github/workflows/build-and-test.yml

@@ -40,6 +40,8 @@ jobs:
         run: |
           mkdir -p ${DYLAN}
           make install
+          ls -l ${DYLAN}/bin/
+          ldd ${DYLAN}/bin/deft
 
       - name: Exercise deft
         env:
@@ -55,3 +57,11 @@ jobs:
           ${exe} list
           ${exe} build abc-test-suite
           _build/bin/abc-test-suite
+
+      - name: Run test suite using submodules
+        env:
+          DYLAN_CATALOG: ext/pacman-catalog
+          DYLAN: dylan-root
+        run: |
+          mkdir -p ${DYLAN}
+          make test-submodules

Makefile

@@ -1,35 +1,34 @@
 # Low-tech Makefile to build and install deft.
 
 DYLAN		?= $${HOME}/dylan
-DEFT_EXE	?= dylan
 
 git_version := "$(shell git describe --tags --always --match 'v*')"
 
-.PHONY: build build-with-version clean install install-debug really-install remove-deft-artifacts test dist distclean
+.PHONY: build build-with-version clean install install-debug really-install remove-deft-artifacts test test-submodules dist distclean
 
-build:
-	$(DEFT_EXE) update
-	$(DEFT_EXE) build --unify deft-app
+build: remove-deft-artifacts
+	OPEN_DYLAN_USER_REGISTRIES=${PWD}/registry dylan-compiler -build -unify deft-app
 
 # Hack to add the version to the binary with git tag info. Don't want this to
 # be the normal build because it causes unnecessary rebuilds.
-build-with-version:
+build-with-version: remove-deft-artifacts
 	file="sources/commands/utils.dylan"; \
 	  orig=$$(mktemp); \
 	  temp=$$(mktemp); \
 	  cp -p $${file} $${orig}; \
 	  cat $${file} | sed "s,/.__./.*/.__./,/*__*/ \"${git_version}\" /*__*/,g" > $${temp}; \
 	  mv $${temp} $${file}; \
-	  $(DEFT_EXE) update; \
-	  $(DEFT_EXE) build --unify deft-app; \
+	  OPEN_DYLAN_USER_REGISTRIES=${PWD}/registry \
+	    dylan-compiler -build -unify deft-app; \
 	  cp -p $${orig} $${file}
 
 # Until the install-deft GitHub Action is no longer referring to deft-app
 # we also create a link named deft-app.
 really-install:
 	mkdir -p $(DYLAN)/bin
 	cp _build/sbin/deft-app $(DYLAN)/bin/deft
-	ln -f $(DYLAN)/bin/deft $(DYLAN)/bin/{deft-app,dylan}
+	ln -f $(DYLAN)/bin/deft $(DYLAN)/bin/deft-app
+	ln -f $(DYLAN)/bin/deft $(DYLAN)/bin/dylan
 
 install: build-with-version really-install
 
@@ -39,16 +38,25 @@ install-debug: build really-install
 # Deft needs to be buildable with submodules so that it can be built on
 # new platforms without having to manually install deps.
 test: build
-	$(DEFT_EXE) build deft-test-suite \
+	OPEN_DYLAN_USER_REGISTRIES=${PWD}/registry \
+	  dylan-compiler -build deft-test-suite \
 	  && DYLAN_CATALOG=ext/pacman-catalog _build/bin/deft-test-suite
 
-dist: distclean install
+# Deft also needs to be buildable with submodules so that it can be built on new
+# platforms without having to manually install deps.
+test-submodules: distclean
+	dylan-compiler -build deft-test-suite \
+	  && DYLAN_CATALOG=ext/pacman-catalog _build/bin/deft-test-suite
 
-clean:
-	rm -rf _build
-	rm -rf _test
+dist: distclean install
 
-distclean: clean
-	rm -f $(DYLAN)/bin/{deft,deft-app,dylan}
+# Sometimes I use deft to develop deft, so this makes sure to clean
+# up its artifacts.
+remove-deft-artifacts:
 	rm -rf _packages
 	find registry -not -path '*/generic/*' -type f -exec rm {} \;
+
+clean: remove-deft-artifacts
+	rm -rf _build _test *~
+
+distclean: clean