slimit icon indicating copy to clipboard operation
slimit copied to clipboard

Python 3 compatibility fixes

Open lelit opened this issue 11 years ago • 3 comments

The following changes were needed to successfully use slimit with Python 3 in my own project.

I had to use current (not yet released) ply though, with a tiny fix of mine for its issue 44.

lelit avatar Feb 09 '14 09:02 lelit

Meanwhile #102 was merged already. Should this be closed? :thinking:

gforcada avatar Oct 02 '18 06:10 gforcada

Probably yes. I switched to https://github.com/calmjs/calmjs.parse months ago, so I can't say if #102 implements everything proposed here, though.

lelit avatar Oct 02 '18 08:10 lelit

They should all be more or less implemented since I cut it from the @lelit fork, however I did make some significant changes as some parts of the fork, such as at places where some new changes were done inefficiently. The other major thing is that that there are a pile of bug fixes that simply couldn't easily be done without correcting some fundamental flaws, such as parsing and the implementation of source map generation. The changelog file has references to all of the issues reported on slimit and the calmjs.parse issue tracker.

Since @gforcada is now here in this thread, if Plone 5 is still going to maintain the existing native JavaScript minification function, the minify print function just had a slight import/function name change if the latest calmjs.parse release is to be used; a brief example is in the documentation.

metatoaster avatar Oct 02 '18 09:10 metatoaster