Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Similar to Devel module's #1263514: patch to notify (likely technical) user what (svn-command) failed in `drush en devel` - show copy of the exec
If svn isn't installed, you get a pretty generic error:
Drush was unable to download CKEditor to /var/www/sites/all/libraries/ckeditor.
First thought to fix was permisssions, after all how often does installing subversion solve the problem? :P
The devel patch above shows the failed exec line which would help in debugging.
Comment | File | Size | Author |
---|---|---|---|
#8 | ckeditor-drush-download-fails-1507852-8.patch | 2.56 KB | vokiel |
| |||
#7 | ckeditor-drush-download-fails-1507852-7.patch | 2.57 KB | solideogloria |
|
Comments
Comment #1
dczepierga CreditAttribution: dczepierga commentedI make patch for it - pls apply this patch to last DEV version and check does it work or nor for u.
Greetings
Comment #2
mkesicki CreditAttribution: mkesicki commentedComment #3
dczepierga CreditAttribution: dczepierga commentedChanges commited to GIT (diff).
Greetings
Comment #4
mkesicki CreditAttribution: mkesicki commentedComment #5
yaach CreditAttribution: yaach commentedThis is still a problem on 7.x-1.19.
Comment #6
vokielUsing SVN won't work as we're not updating it anymore.
I've changed the code responsible for downloading CKEditor. Now it downloads archive from the https://download.cksource.com/CKEditor/CKEditor/ website.
No need to have SVN, GIT installed on the server. Untar is handled by the Drush internal commands.
Comment #7
solideogloria CreditAttribution: solideogloria commentedThe patch file contains the same patch twice (so it doesn't apply). Also it doesn't follow Drupal coding standards.
Attached is a working/fixed patch. Download using the command works.
Comment #8
vokielThank you.
Attaching another patch with your suggestions. Removed the not needed
else
part as in my initial patch.Comment #9
solideogloria CreditAttribution: solideogloria commentedComment #11
vokielThank you for the review!