Re: [Pkg-javascript-devel] Bug#750715: ipython: FTBFS against uglify 2.x series - uglifyjs -nc is not a supported option

2014-07-04 Thread Leo Iannacone
Hi Julian,

please consider to test this patch.

Thanks!
Index: debian/patches/packaged-js.patch
===
--- debian/patches/packaged-js.patch(revision 29613)
+++ debian/patches/packaged-js.patch(working copy)
@@ -6,7 +6,7 @@
  mkdir -p bootstrap/js
  cat js/bootstrap-transition.js js/bootstrap-alert.js js/bootstrap-button.js js/bootstrap-carousel.js js/bootstrap-collapse.js js/bootstrap-dropdown.js js/bootstrap-modal.js js/bootstrap-tooltip.js js/bootstrap-popover.js js/bootstrap-scrollspy.js js/bootstrap-tab.js js/bootstrap-typeahead.js js/bootstrap-affix.js  bootstrap/js/bootstrap.js
 -./node_modules/.bin/uglifyjs -nc bootstrap/js/bootstrap.js  bootstrap/js/bootstrap.min.tmp.js
-+uglifyjs -nc bootstrap/js/bootstrap.js  bootstrap/js/bootstrap.min.tmp.js
++uglifyjs bootstrap/js/bootstrap.js  bootstrap/js/bootstrap.min.tmp.js
  echo /*!\n* Bootstrap.js by @fat  @mdo\n* Copyright 2012 Twitter, Inc.\n* http://www.apache.org/licenses/LICENSE-2.0.txt\n*/;  bootstrap/js/copyright.js
  cat bootstrap/js/copyright.js bootstrap/js/bootstrap.min.tmp.js  bootstrap/js/bootstrap.min.js
  rm bootstrap/js/copyright.js bootstrap/js/bootstrap.min.tmp.js

___
Pkg-javascript-devel mailing list
Pkg-javascript-devel@lists.alioth.debian.org
http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/pkg-javascript-devel

Re: [Pkg-javascript-devel] Bug#750715: ipython: FTBFS against uglify 2.x series - uglifyjs -nc is not a supported option

2014-06-10 Thread Leo Iannacone
On 9 June 2014 16:29, Julian Taylor jtaylor.deb...@googlemail.com wrote:
 tags 750715 + help
 thanks

 As far as I understand uglify 1 and 2 are two different projects.
 Why must the version 2 package replace the version 1 pakage?
 that is not a viable strategy for hard to test javascript code.
 It seems to me it would be simpler to just have two packages.

From the homepage of UglifyJS (https://github.com/mishoo/UglifyJS):

I started working on UglifyJS's successor, version 2. It's almost a
full rewrite (except for the parser which is heavily modified,
everything else starts from scratch). I've detailed my reasons in the
README, see the project page.

https://github.com/mishoo/UglifyJS2

Version 1 will continue to be maintained for fixing show-stopper bugs,
but no new features should be expected.


They share the same namespace so installing both at same time would
raise a package conflict.

In my point of view, and according with manapage, it should be better
go head with v2 and have as more as possible packages compatible with
this version.

As you can see, we have very few packages, at the moment, not
compatible with uglifyjs2:

 https://wiki.debian.org/Javascript/Nodejs/Tasks/UglifyJS2.x


 Anyhow I don't know much about javascript so I probably need a patch.

Really easy, just remove '-nc' option while calling uglifyjs in
debian/patches/packaged-js.patch:

-+ uglifyjs -nc bootstrap/js/bootstrap.js  bootstrap/js/bootstrap.min.tmp.js
++ uglifyjs bootstrap/js/bootstrap.js  bootstrap/js/bootstrap.min.tmp.js

It is also compatible with uglifyjs v1, so you don't have to wait for
uglifyjs2 in unstable to apply this change.



That's all.

Cheers,

Leo.

-- 
Ubuntu Member - http://launchpad.net/~l3on
Home Page - http://leoiannacone.com
GPG Key Id - 0xD282FC25

___
Pkg-javascript-devel mailing list
Pkg-javascript-devel@lists.alioth.debian.org
http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/pkg-javascript-devel