Skip to content
This repository has been archived by the owner on Aug 17, 2020. It is now read-only.

unmatching outcome #6

Open
fedeghe opened this issue Jun 2, 2016 · 3 comments
Open

unmatching outcome #6

fedeghe opened this issue Jun 2, 2016 · 3 comments

Comments

@fedeghe
Copy link

fedeghe commented Jun 2, 2016

Hello, first of all thank You for the great work!!! Dean you are one of my heroes!!! But I'm not here for adulation: starting from the same code the node version has a different result then the one obtained in http://dean.edwards.name/packer/
( Base62 encoded, no Shrink variables )
the first breaks, the second run smooth as expected
I use the node version on a builder I wrote, it would be great if the node port could be as reliable as the online version.
Many Thanks

@evanw
Copy link
Owner

evanw commented Jun 6, 2016

My name is not Dean, sorry. This was a port I did a long time ago and the community has moved on since then. Now most people use uglify. I was keeping this port up in case people are depending on it, but it's not intended for use with new projects. I would recommend using uglify instead because it's a large community effort and is more durable than packer.

If you're still interested in using this library, I can try to fix it. It'll be easier to make sure it's fixed if you give me an example of code that is currently broken.

@fedeghe
Copy link
Author

fedeghe commented Jun 6, 2016

Then sorry Evan,

No, I think I will definitely follow you advice and migrate to uglifyjs
once for all.

Thanks

Il 6/6/16 15:42, Evan Wallace ha scritto:

My name is not Dean, sorry. This was a port I did a long time ago and
the community has moved on since then. Now most people use uglify
https://github.com/mishoo/UglifyJS2. I was keeping this port up in
case people are depending on it, but it's not intended for use with
new projects. I would recommend using uglify instead because it's a
large community effort and is more durable than packer.

If you're still interested in using this library, I can try to fix it.
It'll be easier to make sure it's fixed if you give me an example of
code that is currently broken.


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#6 (comment), or
mute the thread
https://github.com/notifications/unsubscribe/ABRUcioO3i6vgnn_2XugKN0efyPKYu41ks5qJCO7gaJpZM4IsXj7.

@czmirek
Copy link

czmirek commented Nov 15, 2016

Sure, uglify has a strong support...

But:

  • original JS code = 417 KB
  • uglified = 123 KB
  • packing the uglified code with packer = 81 KB

That's one third file size reduction and the code still works. One third is significant enough for me to post this message and give this some support.

And the only thing for this little project to work is to update the jsdom to new version which does not depend on contextify which does not work for node js 7 :)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants