Posted by Dave Reid on January 5, 2010 at 3:44am
8 followers
| Project: | CKEditor - WYSIWYG HTML editor |
| Version: | 6.x-1.x-dev |
| Component: | Code |
| Category: | bug report |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | closed (fixed) |
Issue Summary
Let's make it easy for Drush users to download it!
Comments
#1
#2
Great, thanks for adding a patch. I didn't review it yet but looks good.
Just a note for me to not forget about it - the svn URL should be changed, http://svn.fckeditor.net/CKEditor/releases/stable will be valid since tomorrow or something like that (CKEditor 3.1 is coming! ;))
#3
#4
Thanks
#5
#6
This patch uses the default path sites/all/libraries/ckeditor instead of the one defined by the module sites/all/modules/ckeditor/ckeditor.
#7
The path of ckeditor isn't important, because ckeditor can find editor in libraries directory. Libraries directory is pretty good, because if you are using drush update, it delete ckeditor module directory and create new one, if we decided to download editor to ckeditor module directory, you would be forced to download editor again, after all ckeditor module updates.
#8
Thanks for the explanation ;) Very helpful. I didn't know that before, so I created a symlink to libraries/ckeditor, but it's ok now.
#9
Automatically closed -- issue fixed for 2 weeks with no activity.
#10
The "ckeditor download" command should be replaced by "ckeditor-download" due to changes in drush:
Notice: "ckeditor download" must be renamed to "ckeditor-download" in[error]
order to work with this version of drush. If you are the maintainer
for the module that defines this command, please rename it
immediately. If you are a user of this command, you may enable
spaces in commands for a while by setting "allow-spaces-in-commands"
in your drush configuration file. See example.drushrc.php.
Otherwise, great include!
#11
Changes commited to CVS.
#12
Automatically closed -- issue fixed for 2 weeks with no activity.
#13