Bug#753668: How to fix ckeditor
2014-10-24 22:41 GMT+02:00 Bastien ROUCARIES : Hello, > Alternatively, one can download the package with dget using this command: > > dget -x > http://mentors.debian.net/debian/pool/main/c/ckeditor/ckeditor_4.4.4+dfsg1-2.dsc I've done that, built, checked and uloaded to sid. Note that lintian has some hints: W: ckeditor source: out-of-date-standards-version 3.9.5 (current is 3.9.6) P: ckeditor source: debian-watch-may-check-gpg-signature I: ckeditor: description-synopsis-might-not-be-phrased-properly I: ckeditor: using-first-person-in-description line 1: we E: ckeditor: privacy-breach-may-use-debian-package usr/share/doc/ckeditor/examples/jquery.html You may use libjs-jquery package. (http://code.jquery.com/jquery-1.11.0.min.js) W: ckeditor: embedded-javascript-library usr/share/javascript/ckeditor/adapters/jquery.js please use libjs-jquery I: ckeditor: package-contains-empty-directory usr/share/javascript/ckeditor/plugins/htmlwriter/samples/assets/ X: ckeditor: package-contains-broken-symlink usr/share/javascript/ckeditor/plugins/codesnippet/lib/styles ../../../../highlight/styles Regards Mathieu P -- To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
Bug#753668: How to fix ckeditor
On Fri, Oct 24, 2014 at 11:06 AM, Mathieu Parent wrote: > 2014-10-23 18:42 GMT+02:00 Bastien ROUCARIES : >> >> Le 23 oct. 2014 17:53, "Mathieu Parent" a écrit : >>> >>> 2014-10-23 17:41 GMT+02:00 Bastien ROUCARIES >>> : >>> > No it does not fix. >>> >>> It does for me. >> >> Yes buy you have not recompiled from source some part of ckeditor.js... (sed >> is here for replacing this). So it exzctly it does not work because you are >> not dfsg ok >>> >>> > Please do not upload this nmu. >>> > >>> > Could you please instead describe the symptom and write a build >>> > testsuite ? >>> >>> Just install ckeditor from sid, and create a simple html page like >>> http://docs.ckeditor.com/#!/guide/dev_installation >>> >>> ckeditor doesn't work, with a JS error stating that CKEDITOR.loader is >>> not defined. >> Ok but how can i test it without x during build ? I have tried phatomjs but >> it is not dfsg... any other idea ? >>> > I can fix it. >> Could you remove your nmu, test and sponsor my fix ? > > i will remove my NMU, but where is your fix? To access further information about this package, please visit the following URL: http://mentors.debian.net/package/ckeditor Alternatively, one can download the package with dget using this command: dget -x http://mentors.debian.net/debian/pool/main/c/ckeditor/ckeditor_4.4.4+dfsg1-2.dsc > >> >> Bastien >>> Great! >>> >>> >>> -- >>> Mathieu > > > > -- > Mathieu -- To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
Bug#753668: How to fix ckeditor
2014-10-23 18:42 GMT+02:00 Bastien ROUCARIES : > > Le 23 oct. 2014 17:53, "Mathieu Parent" a écrit : >> >> 2014-10-23 17:41 GMT+02:00 Bastien ROUCARIES >> : >> > No it does not fix. >> >> It does for me. > > Yes buy you have not recompiled from source some part of ckeditor.js... (sed > is here for replacing this). So it exzctly it does not work because you are > not dfsg ok >> >> > Please do not upload this nmu. >> > >> > Could you please instead describe the symptom and write a build >> > testsuite ? >> >> Just install ckeditor from sid, and create a simple html page like >> http://docs.ckeditor.com/#!/guide/dev_installation >> >> ckeditor doesn't work, with a JS error stating that CKEDITOR.loader is >> not defined. > Ok but how can i test it without x during build ? I have tried phatomjs but > it is not dfsg... any other idea ? >> > I can fix it. > Could you remove your nmu, test and sponsor my fix ? i will remove my NMU, but where is your fix? > > Bastien >> Great! >> >> >> -- >> Mathieu -- Mathieu -- To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
Bug#753668: How to fix ckeditor
Le 23 oct. 2014 17:53, "Mathieu Parent" a écrit : > > 2014-10-23 17:41 GMT+02:00 Bastien ROUCARIES : > > No it does not fix. > > It does for me. Yes buy you have not recompiled from source some part of ckeditor.js... (sed is here for replacing this). So it exzctly it does not work because you are not dfsg ok > > > Please do not upload this nmu. > > > > Could you please instead describe the symptom and write a build testsuite ? > > Just install ckeditor from sid, and create a simple html page like > http://docs.ckeditor.com/#!/guide/dev_installation > > ckeditor doesn't work, with a JS error stating that CKEDITOR.loader is > not defined. Ok but how can i test it without x during build ? I have tried phatomjs but it is not dfsg... any other idea ? > > I can fix it. Could you remove your nmu, test and sponsor my fix ? Bastien > Great! > > > -- > Mathieu
Bug#753668: How to fix ckeditor
2014-10-23 17:41 GMT+02:00 Bastien ROUCARIES : > No it does not fix. It does for me. > Please do not upload this nmu. > > Could you please instead describe the symptom and write a build testsuite ? Just install ckeditor from sid, and create a simple html page like http://docs.ckeditor.com/#!/guide/dev_installation ckeditor doesn't work, with a JS error stating that CKEDITOR.loader is not defined. > I can fix it. Great! -- Mathieu -- To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
Bug#753668: How to fix ckeditor
No it does not fix. Please do not upload this nmu. Could you please instead describe the symptom and write a build testsuite ? I can fix it. Bastien Le 22 oct. 2014 13:24, "Mathieu Parent" a écrit : > Hello, > > ckeditor from testing/sid is currently not working. It seems that the > build failed to include all the necessary JS snipsets. > > I see several options here: > 1. Go back to the older working version (3.7 I think). This has the > major drawback of maintaining this security-wise during jessie > lifetime. > 2. Use the not-minified and not-closur-ified version. This will be slow > 3. Package ckbuilder, and use it. This is a big work, and probably > won't pass NEW before the freeze -10days > 4. Fix the current build to behave more like ckbuilder... Any help is > welcome! > 5. Use the already minified version from upstream. This is probably > not DFSG-friendly > > IMO, 1 and 5 should be avoided and I don't know how to take option 4. > > I will try to go option 2, and upload to some delayed queue so that it > doesn't enter in sid before 2014-10-26. I have tried option 3 without > success from now (some jars in uptsream are not in debian already, > ...) > > Thoughts ? > > -- > Mathieu >
Bug#753668: How to fix ckeditor
Le 22 oct. 2014 13:24, "Mathieu Parent" a écrit : > > Hello, > > ckeditor from testing/sid is currently not working. It seems that the > build failed to include all the necessary JS snipsets. > > I see several options here: > 1. Go back to the older working version (3.7 I think). This has the > major drawback of maintaining this security-wise during jessie > lifetime. Not dfsg does not compile from source > 2. Use the not-minified and not-closur-ified version. This will be slow Prefer this but you need to rebuild some part from source. > 3. Package ckbuilder, and use it. This is a big work, and probably > won't pass NEW before the freeze -10days Last time i check ckbuilder is not dfsg need repack > 4. Fix the current build to behave more like ckbuilder... Any help is welcome! Yes > 5. Use the already minified version from upstream. This is probably > not DFSG-friendly Not dfsg > > IMO, 1 and 5 should be avoided and I don't know how to take option 4. > > I will try to go option 2, and upload to some delayed queue so that it > doesn't enter in sid before 2014-10-26. I have tried option 3 without > success from now (some jars in uptsream are not in debian already, > ...) > > Thoughts ? > > -- > Mathieu
Bug#753668: How to fix ckeditor
Hello, I attach the patches of my upcoming upload. I don't know gitpkg and the correct workflow (I mostly use gbp). Regards 2014-10-22 13:24 GMT+02:00 Mathieu Parent : > Hello, > > ckeditor from testing/sid is currently not working. It seems that the > build failed to include all the necessary JS snipsets. > > I see several options here: > 1. Go back to the older working version (3.7 I think). This has the > major drawback of maintaining this security-wise during jessie > lifetime. > 2. Use the not-minified and not-closur-ified version. This will be slow > 3. Package ckbuilder, and use it. This is a big work, and probably > won't pass NEW before the freeze -10days > 4. Fix the current build to behave more like ckbuilder... Any help is welcome! > 5. Use the already minified version from upstream. This is probably > not DFSG-friendly > > IMO, 1 and 5 should be avoided and I don't know how to take option 4. > > I will try to go option 2, and upload to some delayed queue so that it > doesn't enter in sid before 2014-10-26. I have tried option 3 without > success from now (some jars in uptsream are not in debian already, > ...) > > Thoughts ? > > -- > Mathieu -- Mathieu From 427a268ed4699cfcb01b69a651a18b82a79b15de Mon Sep 17 00:00:00 2001 From: Mathieu Parent Date: Wed, 22 Oct 2014 18:30:35 +0200 Subject: [PATCH 1/2] Install complete core sources instead of broken closure-compiled one (Closes: #753668, #756155) --- debian/ckeditor.install | 1 + debian/control | 3 +-- debian/rules| 19 --- 3 files changed, 2 insertions(+), 21 deletions(-) diff --git a/debian/ckeditor.install b/debian/ckeditor.install index 1ca4275..e146064 100644 --- a/debian/ckeditor.install +++ b/debian/ckeditor.install @@ -1,5 +1,6 @@ *.css /usr/share/javascript/ckeditor *.js /usr/share/javascript/ckeditor +core /usr/share/javascript/ckeditor adapters /usr/share/javascript/ckeditor lang /usr/share/javascript/ckeditor plugins /usr/share/javascript/ckeditor diff --git a/debian/control b/debian/control index f9448aa..bc25cf8 100644 --- a/debian/control +++ b/debian/control @@ -3,8 +3,7 @@ Section: web Priority: optional Maintainer: Frank Habermann Uploaders: Bastien Roucariès -Build-Depends: debhelper (>= 9.0~), -closure-compiler +Build-Depends: debhelper (>= 9.0~) Standards-Version: 3.9.5 Homepage: http://ckeditor.com Vcs-Browser: http://anonscm.debian.org/gitweb/?p=collab-maint/ckeditor.git diff --git a/debian/rules b/debian/rules index 8934235..ab16558 100755 --- a/debian/rules +++ b/debian/rules @@ -6,28 +6,9 @@ export CK_BASE = core/ckeditor_base %: dh $@ -override_dh_auto_build: - dh_auto_build - - # manually build ckeditor.js from core/ckeditor_base.min.js - closure-compiler --js ${CK_BASE}.js --js_output_file ${CK_BASE}.min.js - sed -e '/^\/\/[[:space:]]*AUTOGENERATED by debian/{r core/ckeditor_base.min.js' -e 'h;}' \ - -e '$${x;/./{x;q0};x;q1}' 'ckeditor.js' > ckeditor.js.inc - - -override_dh_clean: - dh_clean - test ! -f ${CK_BASE}.min.js || rm -rf ${CK_BASE}.min.js - test ! -f ckeditor.js.inc || rm -rf ckeditor.js.inc - override_dh_install: dh_install - # manually override ckeditor.js from core/ckeditor_base.min.js - cp -f ckeditor.js.inc debian/ckeditor/usr/share/javascript/ckeditor/ckeditor.js - - # remove internal file - find debian/ckeditor/usr/share/javascript/ckeditor -type f -name '_*' | xargs rm -f # remove build file find debian/ckeditor/usr/share/javascript/ckeditor -type f -name 'build-config.js' | xargs rm -f # remove extra LICENSE.md -- 2.1.1 From c2887156bb9f9a87bc89ad970adcb4cca59fd9a3 Mon Sep 17 00:00:00 2001 From: Mathieu Parent Date: Wed, 22 Oct 2014 18:43:20 +0200 Subject: [PATCH 2/2] Changelog for previous commit --- debian/changelog | 8 1 file changed, 8 insertions(+) diff --git a/debian/changelog b/debian/changelog index 22173de..9479f30 100644 --- a/debian/changelog +++ b/debian/changelog @@ -1,3 +1,11 @@ +ckeditor (4.4.4+dfsg1-1.1) unstable; urgency=medium + + * Non-maintainer upload. + * Install complete core sources instead of broken closure-compiled one +(Closes: #753668, #756155) + + -- Mathieu Parent Wed, 22 Oct 2014 18:43:11 +0200 + ckeditor (4.4.4+dfsg1-1) unstable; urgency=high * New upstream release. -- 2.1.1