Clojure

Unable to set compiler options via system properties except for AOT compilation

Details

  • Type: Defect Defect
  • Status: Open Open
  • Priority: Major Major
  • Resolution: Unresolved
  • Affects Version/s: Release 1.5
  • Fix Version/s: Release 1.7
  • Component/s: None
  • Labels:
  • Patch:
    Code
  • Approval:
    Screened

Description

The code that converts JVM system properties into keys under the *compiler-options* var is present only inside the clojure.lang.Compile class. This is a problem when using a debugger inside an IDE and not AOT compiling; specifying -Dclojure.compiler.disable-locals-clearing=true has no effect here when it would be most useful!

Patch: CLJ-1274.patch
Screened by: Stu

Activity

Hide
Stuart Halloway added a comment -

Patch is straightforward, marking screened.

I am left wondering if other options that are set only in Compile.java ought also to be moved.

Show
Stuart Halloway added a comment - Patch is straightforward, marking screened. I am left wondering if other options that are set only in Compile.java ought also to be moved.
Hide
Howard Lewis Ship added a comment - - edited

Changes initialization of *compiler-options* to occur statically inside Compiler; now available to all forms of Clojure, not just AOT compilation; however, the initial *compiler-options* value is now defined as a root binding, rather than a per-thread binding, which has slightly different semantics.

Show
Howard Lewis Ship added a comment - - edited Changes initialization of *compiler-options* to occur statically inside Compiler; now available to all forms of Clojure, not just AOT compilation; however, the initial *compiler-options* value is now defined as a root binding, rather than a per-thread binding, which has slightly different semantics.
Hide
Howard Lewis Ship added a comment -

Obviously, that's supposed to be *compiler-options*.

Show
Howard Lewis Ship added a comment - Obviously, that's supposed to be *compiler-options*.

People

Vote (3)
Watch (1)

Dates

  • Created:
    Updated: