Skip to content

Conversation

@cmb69
Copy link
Member

@cmb69 cmb69 commented Aug 13, 2024

For now, only building pdo_oci on Windows for PHP 8.3, and running a very minimal smoke test.


@cjbj, do you think it would be feasible to actually run the test suite?

For now, only building pdo_oci on Windows for PHP 8.3, and running a
very minimal smoke test.
@cjbj
Copy link
Collaborator

cjbj commented Aug 21, 2024

Yes: https://geraldonit.com/2024/03/18/how-to-use-databases-inside-github-actions/

@cmb69
Copy link
Member Author

cmb69 commented Aug 21, 2024

From https://github.com/gvenzl/setup-oracle-free:

Only Linux runners are supported for the time being.

😢

I'll try to come up with a Linux CI job.

@cmb69
Copy link
Member Author

cmb69 commented Aug 21, 2024

I'll try to come up with a Linux CI job.

Could need some help, see https://github.com/cmb69/pecl-database-pdo_oci/actions/runs/10493187069/job/29066523362. I'm bad at Linux.

@cjbj
Copy link
Collaborator

cjbj commented Aug 22, 2024

Line 24 of "Checkout pdo_oci" says:

Deleting the contents of '/home/runner/work/pecl-database-pdo_oci/pecl-database-pdo_oci'

which occurs before the build starts.

There are a few other things to tweak.

I got a bit further. The current issue is having permissions to install the .so:

Installing shared extensions: /usr/lib/php/20230831/
cp: cannot create regular file '/usr/lib/php/20230831/#INST@3991#': Permission denied

@cjbj
Copy link
Collaborator

cjbj commented Aug 22, 2024

I got a bit further. The current issue is having permissions to install the .so:

This was an issue with leading tabs, not with the sudo command.

@shivammathur shivammathur mentioned this pull request May 31, 2025
@mvorisek
Copy link
Collaborator

mvorisek commented Jun 1, 2025

Closing in favor of #16.

As the author claims he reused some of your work, I have added your authorship via a03a465. Thanks!

@mvorisek mvorisek closed this Jun 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants