Opened 15 years ago

Closed 14 years ago

#3697 closed Task (fixed)

CKLangTool should be launched automatically before releasing

Reported by: Wiktor Walc Owned by: Wiktor Walc
Priority: Normal Milestone: CKEditor 3.1
Component: Project : CKPackager Version:
Keywords: Confirmed Review+ Cc:

Description

the title says it all - in patches usually just the "en" language entry is added and this way CKEditor is broken in all other languages.

Attachments (3)

3697_cklangtool.patch (1.1 KB) - added by Wiktor Walc 14 years ago.
3697_ckreleaser.patch (1.4 KB) - added by Wiktor Walc 14 years ago.
3697_ckeditor.patch (1.9 KB) - added by Wiktor Walc 14 years ago.

Download all attachments as: .zip

Change History (7)

comment:1 Changed 15 years ago by Frederico Caldeira Knabben

Milestone: CKEditor 3.0CKEditor 3.1

Changed 14 years ago by Wiktor Walc

Attachment: 3697_cklangtool.patch added

Changed 14 years ago by Wiktor Walc

Attachment: 3697_ckreleaser.patch added

Changed 14 years ago by Wiktor Walc

Attachment: 3697_ckeditor.patch added

comment:2 Changed 14 years ago by Wiktor Walc

Keywords: Review? added

Basically, the idea is to allow to inject extra scripts to be executed by CKReleaser at some point (by adding extra argument --run-before-release=). We're adding here an option to execute extra code straight after files are copied to the release folder, before the real release process starts (minification etc.).

In the future we may use similar trick for example before files compression (before making zip/tar.gz files) or at any other point.

In release.bat, we're instructing CKReleaser to execute langtool.bat. CKLangTool is instructed to process files inside of the "release" directory, so the source code above _dev/releaser folder is not touched.

There are some minor modifications in CKLangTool to properly instruct CKReleaser when error occurs (errors are now printed to stderr, not to stdout).

comment:3 Changed 14 years ago by Frederico Caldeira Knabben

Keywords: Review+ added; Review? removed

comment:4 Changed 14 years ago by Wiktor Walc

Resolution: fixed
Status: newclosed

Fixed with [4591] (CKLangTool), [4592] (CKReleaser), [4593] (CKEditor).

Note: See TracTickets for help on using tickets.
© 2003 – 2022, CKSource sp. z o.o. sp.k. All rights reserved. | Terms of use | Privacy policy