Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Add PHP 8 images #20

Merged
merged 9 commits into from
Jul 29, 2020
Merged

Add PHP 8 images #20

merged 9 commits into from
Jul 29, 2020

Conversation

schuer
Copy link
Member

@schuer schuer commented Jun 28, 2020 •
edited
Loading

fixes #17

Tags format:

  • 5.10.1-php8.0-rc-apache
  • 5.10-php8.0-rc-apache
  • 5-php8.0-rc-apache

@schuer
Add PHP 8 images
f663214 
fixes #17

- 5.10.1-php8.0-rc-apache
- 5.10-php8.0-rc-apache
- 5-php8.0-rc-apache
@schuer schuer requested a review from bloep June 28, 2020 15:49
@schuer
Copy link
Member Author

schuer commented Jun 28, 2020

Es gibt noch Probleme mit Imagick in Alpine. Schaue ich mir bei Gelegenheit an…

@schuer schuer mentioned this pull request Jul 4, 2020
Closed
@schuer schuer changed the title Add PHP 8 images Add PHP 8 images and drop Imagick support in Alpine Jul 6, 2020
@schuer
Copy link
Member Author

schuer commented Jul 6, 2020 •
edited
Loading

Ich schlage vor, dass wir Imagick nicht länger in Alpine benutzen. Zum Einen ist es mit PHP 8 zur Zeit nicht möglich, Imagick in Alpine zu installieren, und zum Anderen ist der Maintainer von Imagick nicht überzeugt davon, Alpine weiterhin zu supporten. Es macht ihm viel Mühe, und er argumentiert zudem damit, dass Alpine darauf zielt, möglichst wenig Speicherplatz zu benötigen, und dass eine große Extension wie Imagick nicht zu diesem Konzept passt.

Ich sehe das inzwischen auch so: Wer Imagick mit REDAXO benötigt, sollte nicht unbedingt Alpine als Basis nutzen.

@bloep Was meinst du?

@schuer
Copy link
Member Author

schuer commented Jul 6, 2020

Update: Das FPM-Image baut auch nicht. :-(
Ich werde dann mal ein Issue bei Imagick anlegen, bevor es hier weitergehen kann. Schade.

@schuer schuer removed the request for review from bloep July 6, 2020 20:54
@schuer schuer mentioned this pull request Jul 7, 2020
Closed
@schuer schuer marked this pull request as draft July 28, 2020 11:51
@schuer schuer changed the title Add PHP 8 images and drop Imagick support in Alpine Add PHP 8 images Jul 29, 2020
@schuer
Copy link
Member Author

schuer commented Jul 29, 2020

Imagick in Alpine schon mal in separatem PR entfernt: #22

@schuer schuer marked this pull request as ready for review July 29, 2020 19:34
@schuer
Copy link
Member Author

schuer commented Jul 29, 2020

Imagick in PHP-8-Images vorübergehend entfernt.

@schuer schuer requested a review from bloep July 29, 2020 19:37
@schuer schuer merged commit f50c01b into master Jul 29, 2020
@delete-merged-branch delete-merged-branch bot deleted the schuer-php8 branch July 29, 2020 19:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bloep bloep Awaiting requested review from bloep

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Add PHP 8 images
1 participant
@schuer