Closed (fixed)
Project:
Translation template extractor
Version:
2.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
8 Sep 2022 at 12:10 UTC
Updated:
13 Jul 2026 at 13:20 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #3
gábor hojtsyWoah, does it work otherwise then? Did you manually test?
Comment #4
lendudeInstalled potx on clean D10 install with #3285613: Drupal 10 compatibility fixes for potx applied.
Enabled potx
Installed Drush 11.5.1
Ran:
drush potx single --folder="modules/contrib/potx/"Errors seem to be all errors in the way translations are added in potx
Added the output file to the issue, looks good to me.
The D10 compatibility patch makes some changes to the commands, so not sure if this should be committed before that lands, might be better to wait for that to land
Comment #5
spadxiii commentedI think there should be a code-change as well, as drush 11 deprecated the drush.services.yml (drush 12 doesn't support it anymore): https://www.drush.org/11.x/dependency-injection/#services-files
Also, there is a new requirement that the drush commands should be in a 'Drush' subfolder, see the commands.md:
Comment #6
gábor hojtsyhttps://www.drush.org/14.x/commands/ still supports drush.services.yml (although deprecated) so I think we can keep it for backwards compatibility. However the drush compatibility info was indeed not updated in the codebase and the old Drush 8 and earlier compatibility code was not removed.
I checked the code for feature parity and ported a bit of missing feature to the modern Drush code too while I was there.
Comment #8
gábor hojtsyComment #10
gábor hojtsyComment #11
gábor hojtsy