- Issue created by @Maico de Jong
- Status changed to Needs review
almost 2 years ago 8:52am 7 February 2023 - 🇳🇱Netherlands Maico de Jong
Module works fine on Drupal 10 with a small change mentioned in #3272447: Update to PSR/log v3 →
- 🇮🇳India hardikpandya
I was not able to apply the patch in #2. Got this error:
error: patch failed: log_stdout.info.yml:2 error: log_stdout.info.yml: patch does not apply
- First commit to issue fork.
- @makazimtingwa opened merge request.
- 🇺🇸United States MakaziMtingwa
Created the issue fork to install this project in Drupal 10 with:
composer require drupal/log_stdout:dev-3339853-drupal-10-compatibility
At present, this will install patch #2. You may need to include patch #3 locally.
- 🇮🇹Italy arturopanetta Grotteria (RC)
I tested patch #2 and it works, I can redirect logs to stdout container. Patch #3 which I was unable to apply as it tries to remove some lines of code related to module versioning.
I used to:
- Drupal 10.0.9
- log_stdout branch 1.x
- patches #2 - 🇦🇺Australia gigimaor Narangba/Brisbane
This one is working for me after upgrading to Drupal 10.
- 🇺🇸United States TolstoyDotCom L.A.
The patch in #8 looks like it works. I also ran
phpcs --standard=Drupal
and didn't see any warnings. - 🇺🇸United States JonMcL Brooklyn, NY
Patch in #8 works with with the 8.x-1.3 branch, even though this issue is tagged with the 8.x-2.x-dev branch.
- 🇺🇸United States TolstoyDotCom L.A.
8.x-2.x-dev (Last updated: 2 Sep 2020) is marked as only working with D8
8.x-1.x-dev (Last updated: 4 Oct 2021) is marked as working with ^8.8 || ^9 and the patch adds D10 compatibility. That's what I tested.
The main difference between the two dev versions seems to be that 2.x-dev uses a slightly different output format and it also adds user info to the output.
- @scarer opened merge request.
- Status changed to Fixed
about 1 year ago 9:56am 13 September 2023 - 🇮🇹Italy Bladedu Italy
Merged and created release 1.4 which is D10 compatible.
Thank you!
- 🇬🇧United Kingdom rm-rf
I'm afraid you will still need to update the composer.json in order for this module to install correctly via Composer on a D10 install.
- Status changed to Needs work
about 1 year ago 11:15am 13 September 2023 - 🇮🇹Italy Bladedu Italy
You are right. I didn't check the patch properly, I'm afraid.
- @bladedu opened merge request.
- Status changed to Needs review
about 1 year ago 11:40am 13 September 2023 - 🇬🇧United Kingdom rm-rf
That looks good and the module now installs correctly via Composer on Drupal 10.
- 🇳🇿New Zealand fraserthompson Dunedin, New Zealand
Also confirming that the dev branch works with Composer on Drupal 10 but the current stable release doesn't and throws an error.
- 🇮🇹Italy Bladedu Italy
I apologize for the inconvenience. I messed up with tag 1.4 and I couldn't amend it.
I've just releases 1.5 which can be pulled with latest Drupal 10 release.