> On 04/24/2024 4:21 PM CEST Remi Collet <fedora@xxxxxxxxxxxxxxxxx> wrote: > > I can probably help for PHP reviews Thank you, appreciated! > Notice: > > - php-ralouphie-getallheaders: this is a compat layer providing a > missing function in PHP < 7.3 for php-fpm users > > Please check you really still need it ;) Good point, that dependency comes from php-guzzlehttp-psr7 which still depends on php-ralouphie-getallheaders even in newer versions :-/ > - php-guzzlehttp-guzzle6: this was version 6 > > A new package is probably needed for version 7 > > See https://bugzilla.redhat.com/show_bug.cgi?id=1982627 > (this stalled review was for 7.3.0, current is 7.8.1) Right and aws-sdk-php seem to be fine with guzzlehttp/guzzle ^7.4.5 but as you stated, not packaged in Fedora yet. > - php-aws-sdk3: is really outdated (3.191.10 => 3.305.1) > > New version probably have different dependencies Yeah going through them right now to get an understanding what current dependencies are missing in Fedora. > I suspect you may need awscrt extension which is quite > a nightmare as it bundles tons of libaws-* You suspect right, "aws/aws-crt-php": "^1.2.3". > https://git.remirepo.net/cgit/rpms/php/pecl/php-pecl-awscrt.git/tree/ Wow cool, thanks for sharing. Is there a reasons you didn't use your package to create one for Fedora? It looks like you did all the heavy lifting already. What I can already tell: I opened a can of worms with my wish to keep php-aws-sdk3 alive. It will be a challenge but a good learning opportunity too. Dom -- The Wombelix Post https://dominik.wombacher.cc -- _______________________________________________ devel mailing list -- devel@xxxxxxxxxxxxxxxxxxxxxxx To unsubscribe send an email to devel-leave@xxxxxxxxxxxxxxxxxxxxxxx Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedoraproject.org/archives/list/devel@xxxxxxxxxxxxxxxxxxxxxxx Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
