ClojureScript

Compilation with :output-file option set fails

Details

  • Type: Defect Defect
  • Status: Closed Closed
  • Priority: Trivial Trivial
  • Resolution: Completed
  • Affects Version/s: 0.0-3196
  • Fix Version/s: 0.0-3255
  • Component/s: None
  • Labels:
    None
  • Patch:
    Code

Description

./bin/cljsc hello.cljs '{:output-file "foo.js"}'

Expected:
a file called foo.js

Actual:
Compiler throws an internal error

Notes:
The correct option is :output-to
:output-file might be removable (it is broken, nobody is using it?)
Further analysis required to determine if it is useful to keep.

  1. CLJS-742.diff
    03/Jan/14 12:14 PM
    1 kB
    Timothy Pratley
  2. cljs-742-rebased.diff
    04/Apr/15 1:05 PM
    1 kB
    Timothy Pratley

Activity

Timothy Pratley made changes -
Field Original Value New Value
Status Open [ 1 ] In Progress [ 3 ]
Hide
Timothy Pratley added a comment -

Fixes handling of compiling one IJavaScript or many

Show
Timothy Pratley added a comment - Fixes handling of compiling one IJavaScript or many
Timothy Pratley made changes -
Attachment CLJS-742.diff [ 12585 ]
Timothy Pratley made changes -
Labels patch
Timothy Pratley made changes -
Status In Progress [ 3 ] Open [ 1 ]
Hide
Timothy Pratley added a comment -

Patch is attached, updating ticket to reflect that

Show
Timothy Pratley added a comment - Patch is attached, updating ticket to reflect that
Timothy Pratley made changes -
Patch Code [ 10001 ]
Labels patch
David Nolen made changes -
Affects Version/s 0.0-3115 [ 10352 ]
Fix Version/s Next [ 10355 ]
Hide
David Nolen added a comment -

Can we get a patch rebased to master? Thanks.

Show
David Nolen added a comment - Can we get a patch rebased to master? Thanks.
Hide
Timothy Pratley added a comment -

updated for latest master

Show
Timothy Pratley added a comment - updated for latest master
Timothy Pratley made changes -
Attachment cljs-742-rebased.diff [ 14035 ]
David Nolen made changes -
Fix Version/s Next [ 10355 ]
Fix Version/s Backlog [ 10356 ]
Affects Version/s 0.0-3115 [ 10352 ]
Affects Version/s Next [ 10355 ]
David Nolen made changes -
Fix Version/s Next [ 10356 ]
Fix Version/s Backlog [ 10362 ]
Hide
David Nolen added a comment -

Looking into this some more, is there some documentation somewhere that states that :output-file is a valid option to bin/cljsc.

Show
David Nolen added a comment - Looking into this some more, is there some documentation somewhere that states that :output-file is a valid option to bin/cljsc.
David Nolen made changes -
Resolution Completed [ 1 ]
Status Open [ 1 ] Resolved [ 5 ]
David Nolen made changes -
Status Resolved [ 5 ] Closed [ 6 ]

People

Vote (0)
Watch (1)

Dates

  • Created:
    Updated:
    Resolved: