Skip to content

Commit e12bf57

Browse files
Copilotjbtronics
andcommitted
Use --no-install with require and run separate install command
Co-authored-by: jbtronics <[email protected]>
1 parent 562e477 commit e12bf57

File tree

2 files changed

+20
-6
lines changed

2 files changed

+20
-6
lines changed

.docker/frankenphp/docker-entrypoint.sh

Lines changed: 10 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -32,11 +32,18 @@ if [ "$1" = 'frankenphp' ] || [ "$1" = 'php' ] || [ "$1" = 'bin/console' ]; then
3232
# Note: COMPOSER_EXTRA_PACKAGES is intentionally not quoted to allow word splitting
3333
# This enables passing multiple package names separated by spaces
3434
# shellcheck disable=SC2086
35-
composer require $COMPOSER_EXTRA_PACKAGES --no-dev --no-interaction --no-progress --optimize-autoloader
35+
composer require $COMPOSER_EXTRA_PACKAGES --no-install --no-interaction --no-progress
3636
if [ $? -eq 0 ]; then
37-
echo "Successfully installed additional composer packages"
37+
echo "Running composer install to install packages without dev dependencies..."
38+
composer install --no-dev --no-interaction --no-progress --optimize-autoloader
39+
if [ $? -eq 0 ]; then
40+
echo "Successfully installed additional composer packages"
41+
else
42+
echo "Failed to install composer dependencies"
43+
exit 1
44+
fi
3845
else
39-
echo "Failed to install additional composer packages"
46+
echo "Failed to add additional composer packages to composer.json"
4047
exit 1
4148
fi
4249
fi

.docker/partdb-entrypoint.sh

Lines changed: 10 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -45,11 +45,18 @@ if [ -n "$COMPOSER_EXTRA_PACKAGES" ]; then
4545
# Note: COMPOSER_EXTRA_PACKAGES is intentionally not quoted to allow word splitting
4646
# This enables passing multiple package names separated by spaces
4747
# shellcheck disable=SC2086
48-
sudo -E -u www-data composer require $COMPOSER_EXTRA_PACKAGES --no-dev --no-interaction --no-progress --optimize-autoloader
48+
sudo -E -u www-data composer require $COMPOSER_EXTRA_PACKAGES --no-install --no-interaction --no-progress
4949
if [ $? -eq 0 ]; then
50-
echo "Successfully installed additional composer packages"
50+
echo "Running composer install to install packages without dev dependencies..."
51+
sudo -E -u www-data composer install --no-dev --no-interaction --no-progress --optimize-autoloader
52+
if [ $? -eq 0 ]; then
53+
echo "Successfully installed additional composer packages"
54+
else
55+
echo "Failed to install composer dependencies"
56+
exit 1
57+
fi
5158
else
52-
echo "Failed to install additional composer packages"
59+
echo "Failed to add additional composer packages to composer.json"
5360
exit 1
5461
fi
5562
fi

0 commit comments

Comments
 (0)