πŸ‡¬πŸ‡§United Kingdom @T-lo

Bristol, UK
Account created on 28 March 2009, over 15 years ago
#

Recent comments

πŸ‡¬πŸ‡§United Kingdom T-lo Bristol, UK

@euphoric_mv see the readme file for the modification to your root composer file to import the correct dependancy.

πŸ‡¬πŸ‡§United Kingdom T-lo Bristol, UK

@moshe, I see the request to add GitLab CI was approved and the section now appears, however pipelines appear to be disabled.
I added an initial .gitlab-ci.yml file in this branch hoping that would trigger it but it's not run.

I wonder if someone with higher permissions on the repo needs to enable pipelines?

πŸ‡¬πŸ‡§United Kingdom T-lo Bristol, UK

@devkinetic I'd suggest PHP 8 D9 would need to be a separate issue, as we've had to switch akamai library dependencies (to https://github.com/francemediasmonde/akamaiopen-edgegrid-php-client) here to accommodate Drupal 10's switch to psr/log 3.x.

You may find switching to the main akamai-open/edgegrid-client library resolves some of your issues for PHP 8 D9

πŸ‡¬πŸ‡§United Kingdom T-lo Bristol, UK

I've just pushed commits updating deprecated functions, updating the version requirements and adding README file instructions around the repositories requirement in composer for the fork.
The new client requirement appears to be functionally compatible, no reworking required for the switch.

Given this change in akamai client to meet the psr/log requirement, we won't be able to support Drupal core under v10.0.3 going forwards, I would think that means this should be a new major version?

@moshe, I had a read through the document on DrupalCI you linked, and it seems to suggest we can only customise the "assessment" stage of the run. I think we'd need to change the "build" stage of the run, do you know if that's documented elsewhere or if there's an example somewhere?

πŸ‡¬πŸ‡§United Kingdom T-lo Bristol, UK

Sorry, I had to commit that requirements change to get my test setup building locally.
Still working through if it's a good fix.

Thanks for the info about working with a fork.

πŸ‡¬πŸ‡§United Kingdom T-lo Bristol, UK
πŸ‡¬πŸ‡§United Kingdom T-lo Bristol, UK

@rli points out the psr/log composer incompatibility which might make this quite a large re-write.

We're dependant on patriziotomato/edgegrid-client v1.2.1 which has been archived as the original Akamai repo akamai-open/edgegrid-client is active again.

patriziotomato/edgegrid-client forked from the Akamai repo akamai-open/edgegrid-client in 2017 and the version we depend on is three releases past the fork point (1.0.0).

There's a new version 2.0.0 akamai-open/edgegrid-client as well, I'll start some investigation into the compatibility of this, though it still requires psr/log 1. I'll look into contributing an upgrade of psr/log at the akamai repo.

πŸ‡¬πŸ‡§United Kingdom T-lo Bristol, UK
πŸ‡¬πŸ‡§United Kingdom T-lo Bristol, UK

As noted over at https://www.drupal.org/project/akamai/issues/3337078 πŸ“Œ Drupal 10 compatibility Fixed Composer fails to build a working dependancy set as D10 requires the psr/log 3.0.0 whereas the akamai module requires the patriziotomato/edgegrid-client v1.2.1 which then requires psr/log ^1.0

πŸ‡¬πŸ‡§United Kingdom T-lo Bristol, UK

T-lo β†’ made their first commit to this issue’s fork.

Production build 0.69.0 2024