Skip to content

Conversation

@drizuid
Copy link
Member

@drizuid drizuid commented Dec 6, 2024

No description provided.

Copy link
Member Author

@drizuid drizuid left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

---draft until baseimage-alpine-nginx:3.21---

non-dependent upstream discussion - mikespub-org/seblucas-cops#123

if base image supports php84, we would make more changes;

would change to

[php84-dom](https://pkgs.alpinelinux.org/package/v3.21/community/x86_64/php84-dom)
[php84-gd](https://pkgs.alpinelinux.org/package/v3.21/community/x86_64/php84-gd)
[php84-intl](https://pkgs.alpinelinux.org/package/v3.21/community/x86_64/php84-intl)
[php84-pdo_sqlite](https://pkgs.alpinelinux.org/package/v3.21/community/x86_64/php84-pdo_sqlite)
[php84-sqlite3](https://pkgs.alpinelinux.org/package/v3.21/community/x86_64/php84-sqlite3)

and some dockerfile actions would point to php84 rather than php83

@drizuid drizuid marked this pull request as draft December 6, 2024 20:53
@LinuxServer-CI
Copy link
Contributor

I am a bot, here are the test results for this PR:
https://ci-tests.linuxserver.io/lspipepr/cops/3.4.6-pkg-852e095d-dev-c33485f181fdd80058fe26bf302b8a295eabffc8-pr-61/index.html
https://ci-tests.linuxserver.io/lspipepr/cops/3.4.6-pkg-852e095d-dev-c33485f181fdd80058fe26bf302b8a295eabffc8-pr-61/shellcheck-result.xml

Tag Passed
amd64-3.4.6-pkg-852e095d-dev-c33485f181fdd80058fe26bf302b8a295eabffc8-pr-61
arm64v8-3.4.6-pkg-852e095d-dev-c33485f181fdd80058fe26bf302b8a295eabffc8-pr-61

@drizuid drizuid marked this pull request as ready for review December 16, 2024 17:26
@drizuid drizuid requested a review from a team December 16, 2024 17:28
@drizuid
Copy link
Member Author

drizuid commented Dec 17, 2024

tested and confirmed working in upgrade and new install by upstream dev

@drizuid drizuid merged commit ddefebd into master Dec 17, 2024
5 checks passed
@drizuid drizuid deleted the 3.21-rebase branch December 17, 2024 12:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

4 participants