Skip to content

Commit

Permalink
Fix merge of last 2 commits
Browse files Browse the repository at this point in the history
  • Loading branch information
vmcj committed Jul 13, 2024
1 parent ab267ec commit cd96088
Show file tree
Hide file tree
Showing 8 changed files with 9 additions and 19 deletions.
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
2 changes: 1 addition & 1 deletion Makefile
Original file line number Diff line number Diff line change
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 Down
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
12 changes: 3 additions & 9 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 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

0 comments on commit cd96088

Please sign in to comment.