Opened 3 years ago

Last modified 3 years ago

#5020 closed change

[emscripten] Improve compile script — at Initial Version

Reported by: trev Assignee: trev
Priority: P4 Milestone:
Module: Core Keywords:
Cc: Blocked By:
Blocking: #4122 Platform: Unknown / Cross platform
Ready: yes Confidential: no
Tester: Unknown Verified working: no
Review URL(s):

https://codereview.adblockplus.org/29390650/
https://codereview.adblockplus.org/29390657/
https://codereview.adblockplus.org/29390663/
https://codereview.adblockplus.org/29392768/

Description

Background

The script currently used to compile Emscripten code was meant as a temporary solution and has a number of shortcomings that are easy to fix.

What to change

  • Use os.path.join() consistently.
  • Don't use current directory, resolve paths relative to script directory instead.
  • Add command line option to set Emscripten path and only use ../emscripten as fallback.
  • Add --debug command line flag to disable code optimizations and enable memory tracing.
  • Implement a Windows-compatible solution to retrieve Emscripten environment.

Change History (0)

Note: See TracTickets for help on using tickets.