Skip to content

Commit

Permalink
Merge pull request #184 from mlocati/fix-build-32bit-os
Browse files Browse the repository at this point in the history
Fix building ldap for 32-bit OSs
  • Loading branch information
mlocati authored Nov 10, 2020
2 parents 38b97ca + b9991ce commit 2ba3849
Show file tree
Hide file tree
Showing 2 changed files with 15 additions and 3 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/test-extensions.yml
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@ jobs:
runs-on: ubuntu-latest
steps:
- name: Install PHP
uses: shivammathur/setup-php@v1
uses: shivammathur/setup-php@v2
with:
php-version: "7.4"
tools: composer
Expand Down
16 changes: 14 additions & 2 deletions install-php-extensions
Original file line number Diff line number Diff line change
Expand Up @@ -801,6 +801,18 @@ getProcessorCount() {
fi
fi
}

# Get the build target tripled
# Output:
# The target triplet (eg x86_64-linux-gnu, i686-linux-gnu, i386-linux-gnu, ...)
getTargetTriplet() {
getTargetTriplet_tmp="$(gcc -print-multiarch 2>/dev/null || true)"
if test -z "$getTargetTriplet_tmp"; then
getTargetTriplet_tmp="$(gcc -dumpmachine 2>/dev/null || true)"
fi
printf '%s' "$getTargetTriplet_tmp"
}

# Retrieve the number of processors to be used when compiling an extension
#
# Arguments:
Expand Down Expand Up @@ -950,7 +962,7 @@ installBundledModule() {
ldap)
case "$DISTRO" in
debian)
docker-php-ext-configure ldap --with-libdir=lib/$(gcc -dumpmachine)
docker-php-ext-configure ldap --with-libdir=lib/$(getTargetTriplet)
;;
esac
;;
Expand Down Expand Up @@ -1302,7 +1314,7 @@ installPECLModule() {
if test $PHP_MAJMIN_VERSION -le 506; then
installPECLModule_actual="$1-4.3.0"
else
installPECLModule_machine=$(gcc -dumpmachine)
installPECLModule_machine=$(getTargetTriplet)
if ! test -e /usr/include/zstd.h || ! test -e /usr/lib/libzstd.so -o -e "/usr/lib/$installPECLModule_machine/libzstd.so"; then
installPECLModule_zstdVersion=1.4.4
installPECLModule_zstdVersionMajor=$(echo $installPECLModule_zstdVersion | cut -d. -f1)
Expand Down

0 comments on commit 2ba3849

Please sign in to comment.