Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 903728 - dev-lang/php - add avif USE flag to PHP >= 8.1 (working patch/ebuild attached)
Summary: dev-lang/php - add avif USE flag to PHP >= 8.1 (working patch/ebuild attached)
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal major (vote)
Assignee: PHP Bugs
URL: https://github.com/gentoo/gentoo/pull...
Whiteboard:
Keywords: EBUILD, PATCH
Depends on:
Blocks:
 
Reported: 2023-04-03 09:57 UTC by soundbastlerlive
Modified: 2023-05-16 20:21 UTC (History)
1 user (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
avif patch for php-8.2.4 (php-8.2.4-avif.diff,1.21 KB, patch)
2023-04-03 09:59 UTC, soundbastlerlive
Details | Diff
patched php-8.2.4 ebuild (php-8.2.4-r128.ebuild,21.49 KB, text/plain)
2023-04-03 10:00 UTC, soundbastlerlive
Details

Note You need to log in before you can comment on or make changes to this bug.
Description soundbastlerlive 2023-04-03 09:57:32 UTC
PHP >= 8.1. supports avif via gd. just adding --with-avif like e.g. for webp support enables this. tested using attached php-8.2.4.ebuild

Reproducible: Always

Steps to Reproduce:
imageavif()
Actual Results:  
imageavif() does not exist, phpinfo() does not list avif support for gd
Comment 1 soundbastlerlive 2023-04-03 09:59:44 UTC
Created attachment 859462 [details, diff]
avif patch for php-8.2.4
Comment 2 soundbastlerlive 2023-04-03 10:00:01 UTC
Created attachment 859463 [details]
patched php-8.2.4 ebuild
Comment 3 Michael Orlitzky gentoo-dev 2023-05-03 00:06:34 UTC
We had a pull request for this that fizzled out (see $url). The patch alone is not enough because we need new revisions for the new dependencies, and some package.use.mask entries on various platforms.
Comment 4 Larry the Git Cow gentoo-dev 2023-05-16 20:21:22 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=cc3a6140d83af5008f4d1671fb8198cd3ed63f18

commit cc3a6140d83af5008f4d1671fb8198cd3ed63f18
Author:     Michael Orlitzky <mjo@gentoo.org>
AuthorDate: 2023-05-16 19:01:44 +0000
Commit:     Michael Orlitzky <mjo@gentoo.org>
CommitDate: 2023-05-16 20:10:48 +0000

    dev-lang/php: add support for USE=avif in php-8.1 and php-8.2.
    
    This was originally a pull request by Henrik Alves that stalled. A
    follow-up bug reminded me to rebase the PR and apply it to php-8.2, but
    this is mostly Henrik's work. Thanks!
    
    Closes: https://bugs.gentoo.org/903728
    Closes: https://github.com/gentoo/gentoo/pull/26588
    Signed-off-by: Michael Orlitzky <mjo@gentoo.org>

 dev-lang/php/metadata.xml                                | 3 +++
 dev-lang/php/{php-8.1.18.ebuild => php-8.1.18-r1.ebuild} | 7 +++++--
 dev-lang/php/{php-8.2.5.ebuild => php-8.2.5-r1.ebuild}   | 7 +++++--
 3 files changed, 13 insertions(+), 4 deletions(-)

Additionally, it has been referenced in the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=1412b648e214361499e3baf9c8a72c554cdb1562

commit 1412b648e214361499e3baf9c8a72c554cdb1562
Author:     Michael Orlitzky <mjo@gentoo.org>
AuthorDate: 2023-05-16 19:00:47 +0000
Commit:     Michael Orlitzky <mjo@gentoo.org>
CommitDate: 2023-05-16 20:09:20 +0000

    profiles/arch: mask USE=avif for dev-lang/php on a few arches.
    
    Bug: https://bugs.gentoo.org/903728
    Signed-off-by: Michael Orlitzky <mjo@gentoo.org>

 profiles/arch/alpha/package.use.mask         | 1 +
 profiles/arch/hppa/package.use.mask          | 1 +
 profiles/arch/ia64/package.use.mask          | 1 +
 profiles/arch/powerpc/ppc32/package.use.mask | 1 +
 profiles/arch/s390/package.use.mask          | 1 +
 profiles/arch/sparc/package.use.mask         | 1 +
 6 files changed, 6 insertions(+)