Skip to content

Commit cd96088

Browse files
committed
Fix merge of last 2 commits
1 parent ab267ec commit cd96088

File tree

8 files changed

+9
-19
lines changed

8 files changed

+9
-19
lines changed

.github/jobs/data/codespellignorefiles.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@
1010
./config.guess
1111
./gitlab/codespell.yml
1212
./.github/jobs/uploadcodecov.sh
13-
./lib/vendor
13+
./webapp/vendor
1414
./webapp/public/bundles
1515
./webapp/public/js/ace
1616
./webapp/templates/bundles

.github/jobs/syntax-check

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@ if [ ! -x /usr/bin/shellcheck ]; then
2323
fi
2424

2525
find . \( \
26-
-path ./lib/vendor -prune \
26+
-path ./webapp/vendor -prune \
2727
-o -path ./webapp/var -prune \
2828
-o -path ./output -prune \
2929
-o -path ./.git -prune \
@@ -41,7 +41,7 @@ while read -r i ; do
4141
fi
4242
if grep -q "^#\\!.*/bin/sh" "$i" && \
4343
[ "${i##*.}" != "zip" ] && \
44-
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
44+
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
4545
# shellcheck disable=SC2001
4646
echo "$i" | sed -e 's|^./|check for bashisms: |'
4747
checkbashisms "$i"

Makefile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -289,7 +289,7 @@ coverity-conf:
289289
coverity-build: paths.mk
290290
$(MAKE) build build-scripts
291291
# Secondly, delete all upstream PHP libraries to not analyze those:
292-
-rm -rf lib/vendor/*
292+
-rm -rf webapp/vendor/*
293293
@VERSION=` grep '^VERSION =' paths.mk | sed 's/^VERSION = *//'` ; \
294294
PUBLISHED=`grep '^PUBLISHED =' paths.mk | sed 's/^PUBLISHED = *//'` ; \
295295
if [ "$$PUBLISHED" = release ]; then DESC="release" ; \

lib/.gitignore

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,3 @@
11
/judge
22
/submit
3-
/vendor
43
/dj_utils.py

lib/Makefile

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -2,8 +2,6 @@ ifndef TOPDIR
22
TOPDIR=..
33
endif
44

5-
REC_TARGETS = domserver
6-
75
include $(TOPDIR)/Makefile.global
86

97
OBJECTS = $(addsuffix $(OBJEXT),lib.error lib.misc)
@@ -22,5 +20,3 @@ install-domserver:
2220
install-judgehost:
2321
$(INSTALL_DATA) -t $(DESTDIR)$(judgehost_libdir) *.php *.sh
2422
$(INSTALL_PROG) -t $(DESTDIR)$(judgehost_libdir) alert
25-
26-
domserver: SUBDIRS=vendor

webapp/.gitignore

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,7 @@
88
/var/log/*
99
!var/log/.gitkeep
1010
/vendor/
11+
!vendor/Makefile
1112

1213
###> phpunit/phpunit ###
1314
/phpunit.xml

webapp/Makefile

Lines changed: 3 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ REC_TARGETS = domserver
77
include $(TOPDIR)/Makefile.global
88

99
# Subdirectories to recurse into for REC_TARGETS
10-
SUBDIRS = config
10+
SUBDIRS = config vendor
1111

1212
maintainer-conf: .env.local
1313

@@ -18,12 +18,6 @@ maintainer-conf: .env.local
1818
@echo "# the DOMjudge Symfony application in developer mode. Adjust as needed." >> $@
1919
@echo "APP_ENV=dev" >> $@
2020

21-
# Dump autoload dependencies (including plugins)
22-
# This is needed since symfony/runtime is a Composer plugin that runs while dumping
23-
# the autoload file
24-
composer-dump-autoload:
25-
composer $(subst 1,-q,$(QUIET)) dump-autoload -o -a
26-
2721
composer-dump-autoload-dev:
2822
composer $(subst 1,-q,$(QUIET)) dump-autoload
2923

@@ -37,7 +31,7 @@ copy-bundle-assets:
3731
clean-l:
3832
-rm -rf public/bundles/nelmioapidoc
3933

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

61-
inplace-install: composer-autoclean composer-dump-autoload
55+
inplace-install: composer-autoclean
6256
maintainer-install: composer-dump-autoload-dev
6357

6458
# Install PHP dependencies

lib/vendor/Makefile renamed to webapp/vendor/Makefile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,6 @@ clean-l:
77
rm -f autoload_runtime.php
88

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

1212
domserver: autoload_runtime.php

0 commit comments

Comments
 (0)