Skip to content

Commit

Permalink
Replace phony copy-bundle-assets target by a real one
Browse files Browse the repository at this point in the history
This makes that we don't rerun the rule during
`domserver-install` if it already ran during `domserver`, so
that if you install as root, the local files are still owned
by the user who first ran `make domserver`, not root.

This does assume that if the directory `public/bundles/nelmioapidoc`
exists, then it contains the assets, and these won't automatically
get updated, for example, in the setting of a maintainer-install.
  • Loading branch information
eldering committed Mar 16, 2024
1 parent c7c5b81 commit c8b613d
Showing 1 changed file with 4 additions and 9 deletions.
13 changes: 4 additions & 9 deletions webapp/Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -9,19 +9,16 @@ include $(TOPDIR)/Makefile.global
# Subdirectories to recurse into for REC_TARGETS
SUBDIRS = config

copy-bundle-assets:
public/bundles/nelmioapidoc:
# We can not use bin/console here, as when using a fakeroot,
# the include paths are broken. We just copy in the data we need
-rm -rf public/bundles/nelmioapidoc
mkdir -p public/bundles/nelmioapidoc
cp -a ../lib/vendor/nelmio/api-doc-bundle/Resources/public/* public/bundles/nelmioapidoc/
mkdir -p $@
cp -a ../lib/vendor/nelmio/api-doc-bundle/Resources/public/* $@

clean-l:
-rm -rf public/bundles/nelmioapidoc

domserver-l:
# This must be done first to install with the rest.
$(MAKE) copy-bundle-assets
domserver-l: public/bundles/nelmioapidoc

install-domserver:
$(INSTALL_DIR) $(DESTDIR)$(domserver_webappdir);
Expand All @@ -48,5 +45,3 @@ maintainer-clean-l:
rm -rf var/$$d/$$t ; \
done ; \
done

.PHONY: copy-bundle-assets

0 comments on commit c8b613d

Please sign in to comment.