Skip to content

Merge v1.20 into v1.x #1733

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.

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

Merged
merged 1 commit into from
Oct 18, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
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
3 changes: 3 additions & 0 deletions .github/actions/windows/build/action.yml
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,9 @@ inputs:
description: "Thread-safety (nts or ts)"
required: true
outputs:
vs:
description: "The Visual Studio version"
value: ${{ steps.prepare-build-env.outputs.vs }}
build-dir:
description: "The build directory to be used"
value: ${{ steps.prepare-build-env.outputs.build-dir }}
Expand Down
3 changes: 3 additions & 0 deletions .github/actions/windows/prepare-build/action.yml
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,9 @@ inputs:
description: "Thread-safety (nts or ts)"
required: true
outputs:
vs:
description: "The Visual Studio version"
value: ${{steps.setup-php.outputs.vs}}
build-dir:
description: "The build directory to be used"
value: ${{steps.get-build-dir.outputs.build_dir}}
Expand Down
35 changes: 14 additions & 21 deletions .github/workflows/build-windows-package.yml
Original file line number Diff line number Diff line change
Expand Up @@ -38,6 +38,8 @@ jobs:
defaults:
run:
shell: cmd
outputs:
vs: ${{ steps.build-driver.outputs.vs }}

steps:
- uses: actions/checkout@v4
Expand Down Expand Up @@ -110,31 +112,22 @@ jobs:
with:
filenames: php_mongodb.dll

# Copy the signature file from the release asset directory to avoid directory issues in the ZIP file
- name: "Copy signature file"
run: cp ${RELEASE_ASSETS}/php_mongodb.dll.sig .
- name: "Generate file name for DLL and archive"
run:
echo FILENAME="php_mongodb-${{ inputs.version }}-${{ inputs.php }}-${{ inputs.ts }}-${{ needs.build.outputs.vs }}-${{ inputs.arch == 'x64' && 'x64_86' || inputs.arch }}" >> "$GITHUB_ENV"

- name: "Set compiler environment variable"
# In this step, we:
# - update the extension DLL file name to match the expectation of pie
# - copy the signature file from the release asset directory to avoid directory issues in the archive
# - rename the signature file to match the extension DLL file
- name: "Copy signature file and use correct file names"
run: |
case "$PHP_VERSION" in
"7.4")
COMPILER="vc15"
;;
"8.0" | "8.1" | "8.2" | "8.3")
COMPILER="vs16"
;;
"8.4")
COMPILER="vs17"
;;
esac
echo "COMPILER=${COMPILER}" >> "$GITHUB_ENV"
shell: bash
env:
PHP_VERSION: ${{ inputs.php }}
mv php_mongodb.dll ${{ env.FILENAME }}.dll
cp ${RELEASE_ASSETS}/php_mongodb.dll.sig ${{ env.FILENAME }}.dll.sig

- name: "Create and upload release asset"
if: ${{ inputs.upload_release_asset }}
run: |
ARCHIVE=php_mongodb-${{ inputs.version }}-${{ inputs.php }}-${{ inputs.ts }}-${{ env.COMPILER }}-${{ inputs.arch }}.zip
zip ${ARCHIVE} php_mongodb.dll php_mongodb.dll.sig php_mongodb.pdb CREDITS CONTRIBUTING.md LICENSE README.md THIRD_PARTY_NOTICES
ARCHIVE=${{ env.FILENAME }}.zip
zip ${ARCHIVE} ${{ env.FILENAME }}.dll ${{ env.FILENAME }}.dll.sig php_mongodb.pdb CREDITS CONTRIBUTING.md LICENSE README.md THIRD_PARTY_NOTICES
gh release upload ${{ inputs.version }} ${ARCHIVE}