10000 minor #39745 Fix CI: Use full path to binaries in dockerized steps (j… · symfony/symfony@92ccb0a · GitHub
[go: up one dir, main page]
Skip to content

Commit 92ccb0a

Browse files
derrabusderrabus
committed
minor #39745 Fix CI: Use full path to binaries in dockerized steps (jderusse)
This PR was merged into the 4.4 branch. Discussion ---------- Fix CI: Use full path to binaries in dockerized steps | Q | A | ------------- | --- | Branch? | 4.4 | Bug fix? | yes | New feature? | no | Deprecations? | no | Tickets | - | License | MIT | Doc PR | - ~4 hours ago Github automatically adds the `PATH` env variable into docker steps. Which is an issue for the openldap, because the image stores the binaries in a vendor-prefixed folder. This PR use full path to binaries instead of relying on PATH Commits ------- 9a3fc5a Fix bitnami env PATH
2 parents 9b719ab + 9a3fc5a commit 92ccb0a

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

‎.github/workflows/tests.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -75,7 +75,7 @@ jobs:
7575
uses: docker://bitnami/openldap
7676
with:
7777
entrypoint: /bin/bash
78-
args: -c "(ldapwhoami -h localhost:3389 -D cn=admin,dc=symfony,dc=com -w symfony||sleep 5) && ldapadd -h ldap:3389 -D cn=admin,dc=symfony,dc=com -w symfony -f src/Symfony/Component/Ldap/Tests/Fixtures/data/fixtures.ldif && ldapdelete -h ldap:3389 -D cn=admin,dc=symfony,dc=com -w symfony cn=a,ou=users,dc=symfony,dc=com"
78+
args: -c "(/opt/bitnami/openldap/bin/ldapwhoami -h localhost:3389 -D cn=admin,dc=symfony,dc=com -w symfony||sleep 5) && /opt/bitnami/openldap/bin/ldapadd -h ldap:3389 -D cn=admin,dc=symfony,dc=com -w symfony -f src/Symfony/Component/Ldap/Tests/Fixtures/data/fixtures.ldif && /opt/bitnami/openldap/bin/ldapdelete -h ldap:3389 -D cn=admin,dc=symfony,dc=com -w symfony cn=a,ou=users,dc=symfony,dc=com"
7979

8080
- name: Configure composer
8181
run: |

0 commit comments

Comments
 (0)
0