Merge webapp/vendor/Makefile into webapp/Makefile

I find it easier to follow when all the composer commands are in the
same Makefile.

Also move `composer auto-scripts` back into inplace-install-l, out of
concern for ordering issues. It was moved into a prereq of
inplace-install in commit ab267ec (Put lib/vendor close to the
webapp, 2024-06-23).

Author: tom93

Makefile

@@ -89,7 +89,7 @@ docs:                       SUBDIRS=    doc
 install-docs:               SUBDIRS=    doc
 maintainer-conf:            SUBDIRS=                                 webapp
 maintainer-install:         SUBDIRS=                                 webapp
-inplace-install:            SUBDIRS=    doc               misc-tools webapp
+inplace-install:            SUBDIRS=    doc               misc-tools
 inplace-uninstall:          SUBDIRS=    doc               misc-tools
 dist:                       SUBDIRS=        lib sql       misc-tools
 clean:                      SUBDIRS=etc doc lib sql judge misc-tools webapp
@@ -227,6 +227,8 @@ inplace-install-l:
 # because judgehost-create-dirs sets wrong permissions:
 	$(MKDIR_P) $(domserver_tmpdir)
 	chmod a+rwx $(domserver_tmpdir)
+# Make sure we're running from a clean state:
+	(cd webapp && composer auto-scripts)
 	@echo ""
 	@echo "========== Maintainer Install Completed =========="
 	@echo ""

webapp/.gitignore

@@ -8,7 +8,6 @@
 /var/log/*
 !var/log/.gitkeep
 /vendor/
-!vendor/Makefile
 
 ###> phpunit/phpunit ###
 /phpunit.xml

webapp/Makefile

@@ -7,7 +7,9 @@ REC_TARGETS = domserver
 include $(TOPDIR)/Makefile.global
 
 # Subdirectories to recurse into for REC_TARGETS
-SUBDIRS = config vendor
+SUBDIRS = config
+
+domserver: composer-dump-autoload
 
 maintainer-conf: .env.local
 
@@ -18,6 +20,16 @@ maintainer-conf: .env.local
 	@echo "# the DOMjudge Symfony application in developer mode. Adjust as needed." >> $@
 	@echo "APP_ENV=dev" >> $@
 
+# Dump autoload dependencies (including plugins)
+# This is needed since symfony/runtime is a Composer plugin that runs while dumping
+# the autoload file.
+# We skip it if autoload_runtime.php already exists, to avoid running composer
+# as root during `sudo make install-domserver`.
+composer-dump-autoload: vendor/autoload_runtime.php
+
+vendor/autoload_runtime.php:
+	composer $(subst 1,-q,$(QUIET)) dump-autoload -o -a
+
 composer-dump-autoload-dev:
 	composer $(subst 1,-q,$(QUIET)) dump-autoload
 
@@ -30,6 +42,7 @@ copy-bundle-assets:
 
 clean-l:
 	-rm -rf public/bundles/nelmioapidoc
+	-rm -f vendor/autoload_runtime.php
 
 install-domserver:
 # This must be done first to install with the rest.
@@ -52,7 +65,6 @@ install-domserver:
 	done
 	$(INSTALL_DATA) -t $(DESTDIR)$(domserver_webappdir) phpunit.xml.dist .env
 
-inplace-install: composer-autoclean
 maintainer-install: composer-dump-autoload-dev
 
 # Install PHP dependencies
@@ -67,15 +79,12 @@ endif
 composer-dependencies-dev:
 	composer $(subst 1,-q,$(QUIET)) install --prefer-dist --no-scripts --no-plugins
 
-composer-autoclean:
-# Make sure we're running from a clean state:
-	composer auto-scripts
-
 maintainer-clean-l:
 	-for d in cache log ; do \
 		for t in dev prod ; do \
 			rm -rf var/$$d/$$t ; \
 		done ; \
 	done
 
-.PHONY: copy-bundle-assets
+.PHONY: composer-dump-autoload composer-dump-autoload-dev \
+        copy-bundle-assets