Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Run composer commands as nginx #329

Merged
merged 9 commits into from
Sep 27, 2023
Merged

Run composer commands as nginx #329

Changes from 4 commits
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
7045555
Adds the "run as" part of the command
DonRichards Mar 16, 2023
a2c04c6
Update Makefile
DonRichards Apr 21, 2023
2d58fcb
Update Makefile
DonRichards Apr 21, 2023
8ecfa84
Update Makefile
DonRichards May 10, 2023
40f3381
fix typo
DonRichards May 24, 2023
9a189d1
Merge branch 'development' into force_composer_as_nginx
DonRichards Sep 27, 2023
417059a
Add wait for /var/www/drupal
DonRichards Sep 27, 2023
4efb87c
Removed hyphens in docker compose
DonRichards Sep 27, 2023
fde8c82
Additional hyphens missed in docker compose
DonRichards Sep 27, 2023
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
6 changes: 3 additions & 3 deletions Makefile
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -148,7 +148,7 @@ local: generate-secrets
fi
$(MAKE) set-files-owner SRC=$(CURDIR)/codebase ENVIRONMENT=local
docker-compose up -d --remove-orphans
docker-compose exec -T drupal with-contenv bash -lc 'composer install; chown -R nginx:nginx .'
docker-compose exec -T drupal with-contenv bash -lc "chown -R nginx:nginx . ; su nginx -s /bin/bash -c 'composer install'"
$(MAKE) remove_standard_profile_references_from_config drupal-database update-settings-php ENVIRONMENT=local
docker-compose exec -T drupal with-contenv bash -lc "drush si -y islandora_install_profile_demo --account-pass $(shell cat secrets/live/DRUPAL_DEFAULT_ACCOUNT_PASSWORD)"
$(MAKE) delete-shortcut-entities && docker-compose exec -T drupal with-contenv bash -lc "drush pm:un -y shortcut"
Expand Down Expand Up @@ -185,7 +185,7 @@ starter_dev: generate-secrets
fi
$(MAKE) set-files-owner SRC=$(CURDIR)/codebase ENVIRONMENT=starter_dev
docker-compose up -d --remove-orphans
docker-compose exec -T drupal with-contenv bash -lc 'composer install'
docker-compose exec -T drupal with-contenv bash -lc "chown -R nginx:nginx . ; su nginx -s /bin/bash -c 'composer install'"
$(MAKE) starter-finalize ENVIRONMENT=starter_dev


Expand Down Expand Up @@ -309,7 +309,7 @@ download-default-certs:

# Run Composer Update in your Drupal container
composer_update:
docker-compose exec -T drupal with-contenv bash -lc 'composer update'
docker-compose exec -T drupal with-contenv bash -lc su nginx -s /bin/bash -c 'composer update'"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should this be changed to match the other changes using the --user nginx instead of su nginx -s /bin/bash -c?



reindex-fcrepo-metadata:
Expand Down