xpra icon
Bug tracker and wiki

This bug tracker and wiki are being discontinued
please use https://github.com/Xpra-org/xpra instead.

Opened 2 years ago

Last modified 17 months ago

#2709 assigned task

use terser for minification

Reported by: Antoine Martin Owned by: Antoine Martin
Priority: major Milestone: 5.0
Component: html5 Version: 3.0.x
Keywords: Cc:



The main difficulty is to ensure this is installed on all the build environment as there aren't many packages for it...

Change History (5)

comment:1 Changed 2 years ago by brief

Do you plan to use terser standalone? Or in combination with webpack (can bundle assets)?

If nodejs is used, I would also use forge.js as a dependency (as well as the other libraries if possible).

comment:2 Changed 2 years ago by Antoine Martin

Status: newassigned

Do you plan to use terser standalone? Or in combination with webpack (can bundle assets)?

I don't know.
I profoundly dislike how some of those frameworks make it difficult to deploy things.
As things stand, I can just copy the js files, by hand if necessary, and things just work. That's invaluable to me.

comment:3 Changed 2 years ago by brief

uglifyjs.terser exists in sid, ubuntu (disco, eoan, focal) only. It can be used where available.

If terser is not available, minifyjs or yui compressor is used after let/const is replaced by var.
Further JS changes (e.g. ES6) need manual adjustments to be compatible to minifying until terser can be used.

npm is not used to avoid a humongus build dependency, which we can't easily have on MacOS. Furthermore, there should only be one package manager or none (macos).

comment:4 Changed 21 months ago by Antoine Martin

Milestone: 4.15.0

comment:5 Changed 17 months ago by migration script

this ticket has been moved to: https://github.com/Xpra-org/xpra/issues/2709

Note: See TracTickets for help on using tickets.