Skip to content

Commit 2798778

Browse files
authored
Optimized Composer dependency adding (#33)
* Optimized Composer dependency adding * Don't execute recipes for all dependencies
1 parent 15f176f commit 2798778

File tree

1 file changed

+6
-7
lines changed

1 file changed

+6
-7
lines changed

bin/prepare_project_edition.sh

Lines changed: 6 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -85,7 +85,6 @@ docker exec install_dependencies composer recipes:install ibexa/${PROJECT_EDITIO
8585

8686
# Install Docker stack
8787
docker exec install_dependencies composer require --dev ibexa/docker:^0.1.x-dev --no-scripts
88-
docker exec install_dependencies composer sync-recipes ibexa/docker
8988

9089
# Add other dependencies if required
9190
if [ -f ./${DEPENDENCY_PACKAGE_NAME}/dependencies.json ]; then
@@ -102,15 +101,15 @@ if [ -f ./${DEPENDENCY_PACKAGE_NAME}/dependencies.json ]; then
102101
echo ">> Private or fork repository detected, adding VCS to Composer repositories"
103102
docker exec install_dependencies composer config repositories.$(uuidgen) vcs "$REPO_URL"
104103
fi
105-
docker exec install_dependencies composer require ${PACKAGE_NAME}:"$REQUIREMENT" --no-scripts --no-install || true
104+
jq --arg package "$PACKAGE_NAME" --arg requirement "$REQUIREMENT" '.["require"] += { ($package) : ($requirement) }' composer.json > composer.json.new
105+
mv composer.json.new composer.json
106106
done
107107

108-
docker exec install_dependencies composer install --no-scripts
108+
docker exec install_dependencies composer update --no-scripts
109109

110-
for ((i=0;i<$COUNT;i++)); do
111-
PACKAGE_NAME=$(cat dependencies.json | jq -r .[$i].package)
112-
docker exec install_dependencies composer sync-recipes ${PACKAGE_NAME} --force
113-
done
110+
# Execute recipes from BehatBundle and docker again, because they use copy-from-package
111+
docker exec install_dependencies composer sync-recipes ibexa/docker --force
112+
docker exec install_dependencies composer sync-recipes ezsystems/behatbundle --force
114113
fi
115114

116115
# Create a default Behat configuration file

0 commit comments

Comments
 (0)