Opened 11 years ago

Closed 8 years ago

#3597 closed New Feature (wontfix)

Usability improvements in CKReleaser

Reported by: Wiktor Walc Owned by: Wiktor Walc
Priority: Normal Milestone:
Component: Project : CKPackager Version:
Keywords: Cc:

Description

Originally reported by Alfonso in #3546

  - The "Creating documentation..." step takes quite a while. If isn't
 possible to output something during that process I would put a warning
 ("Please wait, it might take some minutes.")

  - After that the "Checking compressed code..." is repeated over and over
 again. It should state which file is checking or simply adding a dot like
 other steps so it's nicer and don't lead to think that it has started an
 endless loop. Also it will allow to check previous messages instead of
 filling the screen buffer.

Plus:

  • it should be also much easier to run CKReleaser in a verbose

mode (release.bat -v should enable verbose mode)

  • package.sh should start CKPackager in verbose mode just like package.bat
  • CKPackager in verbose mode should list paths instead of file names (it shows plenty of plugin.js which is rather useless in debugging)

Attachments (5)

3597_ckeditor.patch (1.5 KB) - added by Wiktor Walc 11 years ago.
3597_ckpackager.patch (2.0 KB) - added by Wiktor Walc 11 years ago.
3597_ckreleaser.patch (1.5 KB) - added by Wiktor Walc 11 years ago.
ckpackager.jar (904.6 KB) - added by Wiktor Walc 11 years ago.
ckreleaser.jar (1.1 MB) - added by Wiktor Walc 11 years ago.

Download all attachments as: .zip

Change History (8)

Changed 11 years ago by Wiktor Walc

Attachment: 3597_ckeditor.patch added

Changed 11 years ago by Wiktor Walc

Attachment: 3597_ckpackager.patch added

Changed 11 years ago by Wiktor Walc

Attachment: 3597_ckreleaser.patch added

Changed 11 years ago by Wiktor Walc

Attachment: ckpackager.jar added

Changed 11 years ago by Wiktor Walc

Attachment: ckreleaser.jar added

comment:1 Changed 11 years ago by Wiktor Walc

Keywords: Review? added

The attached .jar files are just to make testing 3597_ckpackager.patch and 3597_ckreleaser.patch easier.

Patch from #3546 is not included.

comment:2 Changed 9 years ago by Frederico Caldeira Knabben

Status: reviewreview_passed

I'm not really running the code, but I think the changes are minor and are ok. The patch needs to be adapted to the current code and then run to see if works well. This can be done on commit, directly.

comment:3 Changed 8 years ago by Wiktor Walc

Resolution: wontfix
Status: review_passedclosed
Note: See TracTickets for help on using tickets.
© 2003 – 2019 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy