Skip to content

Fix forgotten merge #2618

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 3 commits into from
Jul 22, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion .github/jobs/data/codespellignorefiles.txt
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@
./config.guess
./gitlab/codespell.yml
./.github/jobs/uploadcodecov.sh
./lib/vendor
./webapp/vendor
./webapp/public/bundles
./webapp/public/js/ace
./webapp/templates/bundles
Expand Down
4 changes: 2 additions & 2 deletions .github/jobs/syntax-check
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@ if [ ! -x /usr/bin/shellcheck ]; then
fi

find . \( \
-path ./lib/vendor -prune \
-path ./webapp/vendor -prune \
-o -path ./webapp/var -prune \
-o -path ./output -prune \
-o -path ./.git -prune \
Expand All @@ -41,7 +41,7 @@ while read -r i ; do
fi
if grep -q "^#\\!.*/bin/sh" "$i" && \
[ "${i##*.}" != "zip" ] && \
echo "$i" | grep -qvE '(^\./(misc-tools/dj_judgehost_cleanup.in|misc-tools/dj_make_chroot.in|config|autom4te|install-sh|sql/files/defaultdata/hs/run|sql/files/defaultdata/kt/run|lib/vendor/|output|judge/judgedaemon))'; then
echo "$i" | grep -qvE '(^\./(misc-tools/dj_judgehost_cleanup.in|misc-tools/dj_make_chroot.in|config|autom4te|install-sh|sql/files/defaultdata/hs/run|sql/files/defaultdata/kt/run|webapp/vendor/|output|judge/judgedaemon))'; then
# shellcheck disable=SC2001
echo "$i" | sed -e 's|^./|check for bashisms: |'
checkbashisms "$i"
Expand Down
7 changes: 3 additions & 4 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ export TOPDIR = $(shell pwd)

REC_TARGETS=build domserver install-domserver judgehost install-judgehost \
docs install-docs inplace-install inplace-uninstall maintainer-conf \
composer-dependencies composer-dependencies-dev
maintainer-install composer-dependencies composer-dependencies-dev

# Global Makefile definitions
include $(TOPDIR)/Makefile.global
Expand Down Expand Up @@ -198,7 +198,7 @@ inplace-conf-common: dist
# Install the system in place: don't really copy stuff, but create
# symlinks where necessary to let it work from the source tree.
# This stuff is a hack!
maintainer-install: inplace-install composer-dump-autoload-dev
maintainer-install: inplace-install
inplace-install: build domserver-create-dirs judgehost-create-dirs
inplace-install-l:
# Replace libjudgedir with symlink to prevent lots of symlinks:
Expand Down Expand Up @@ -289,7 +289,7 @@ coverity-conf:
coverity-build: paths.mk
$(MAKE) build build-scripts
# Secondly, delete all upstream PHP libraries to not analyze those:
-rm -rf lib/vendor/*
-rm -rf webapp/vendor/*
@VERSION=` grep '^VERSION =' paths.mk | sed 's/^VERSION = *//'` ; \
PUBLISHED=`grep '^PUBLISHED =' paths.mk | sed 's/^PUBLISHED = *//'` ; \
if [ "$$PUBLISHED" = release ]; then DESC="release" ; \
Expand All @@ -316,5 +316,4 @@ clean-autoconf:
$(addprefix inplace-,conf conf-common install uninstall) \
$(addprefix maintainer-,conf install) clean-autoconf config distdocs \
composer-dependencies composer-dependencies-dev \
composer-dump-autoload-dev \
coverity-conf coverity-build
1 change: 0 additions & 1 deletion lib/.gitignore
Original file line number Diff line number Diff line change
@@ -1,4 +1,3 @@
/judge
/submit
/vendor
/dj_utils.py
4 changes: 0 additions & 4 deletions lib/Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -2,8 +2,6 @@ ifndef TOPDIR
TOPDIR=..
endif

REC_TARGETS = domserver

include $(TOPDIR)/Makefile.global

OBJECTS = $(addsuffix $(OBJEXT),lib.error lib.misc)
Expand All @@ -22,5 +20,3 @@ install-domserver:
install-judgehost:
$(INSTALL_DATA) -t $(DESTDIR)$(judgehost_libdir) *.php *.sh
$(INSTALL_PROG) -t $(DESTDIR)$(judgehost_libdir) alert

domserver: SUBDIRS=vendor
1 change: 1 addition & 0 deletions webapp/.gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,7 @@
/var/log/*
!var/log/.gitkeep
/vendor/
!vendor/Makefile

###> phpunit/phpunit ###
/phpunit.xml
Expand Down
14 changes: 4 additions & 10 deletions webapp/Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ REC_TARGETS = domserver
include $(TOPDIR)/Makefile.global

# Subdirectories to recurse into for REC_TARGETS
SUBDIRS = config
SUBDIRS = config vendor

maintainer-conf: .env.local

Expand All @@ -18,12 +18,6 @@ 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
composer-dump-autoload:
composer $(subst 1,-q,$(QUIET)) dump-autoload -o -a

composer-dump-autoload-dev:
composer $(subst 1,-q,$(QUIET)) dump-autoload

Expand All @@ -37,7 +31,7 @@ copy-bundle-assets:
clean-l:
-rm -rf public/bundles/nelmioapidoc

install-domserver: composer-dump-autoload
install-domserver:
# This must be done first to install with the rest.
$(MAKE) copy-bundle-assets
$(INSTALL_DIR) $(DESTDIR)$(domserver_webappdir);
Expand All @@ -58,7 +52,7 @@ install-domserver: composer-dump-autoload
done
$(INSTALL_DATA) -t $(DESTDIR)$(domserver_webappdir) phpunit.xml.dist .env

inplace-install: composer-autoclean composer-dump-autoload
inplace-install: composer-autoclean
maintainer-install: composer-dump-autoload-dev

# Install PHP dependencies
Expand All @@ -74,7 +68,7 @@ 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:
# Make sure we're running from a clean state:
composer auto-scripts

maintainer-clean-l:
Expand Down
2 changes: 1 addition & 1 deletion lib/vendor/Makefile → webapp/vendor/Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,6 @@ clean-l:
rm -f autoload_runtime.php

autoload_runtime.php:
composer $(subst 1,-q,$(QUIET)) dump-autoload -o -a -d $(TOPDIR)
composer $(subst 1,-q,$(QUIET)) dump-autoload -o -a -d ..

domserver: autoload_runtime.php
Loading