Args are not provided to main-cli-fn with optimizations

Description

This works when compiled with simple optimizations, and doesn't with advanced optimizations.

Proposed fix is to use nodejs/process instead of js/process here:
https://github.com/clojure/clojurescript/blob/fb90282f41799fe509b143e7870eaf4f9885de41/src/main/cljs/cljs/nodejscli.cljs#L21

Environment

None

Attachments

CLJS-2328.patch

17 Aug, 2017

Activity

David Nolen

August 18, 2017 at 1:58 PM

fixed https://github.com/clojure/clojurescript/commit/799325ac84a7cc3acdd68ce98df61328779c63d7

Yegor Timoshenko

August 17, 2017 at 11:22 PM

I see. Thanks!

António Monteiro

August 17, 2017 at 10:59 PM

`cljs.nodejs/process` is the same as `js/process`. That wasn't the problem.

`argv` was being munged so we need to access it as a string. Or provide externs for it.

Yegor Timoshenko

August 17, 2017 at 10:56 PM

Antonio, why not just use cljs.nodejs/process? It uses externs.

António Monteiro

August 17, 2017 at 10:22 PM

Patch attached with fix.

Resize work item view side panel

Completed

Details

Assignee

David Nolen

Reporter

Yegor Timoshenko

Approval

Accepted

Patch

Code

Fix versions

1.9.908

Priority

Minor

Created August 17, 2017 at 9:21 PM

Updated August 18, 2017 at 1:58 PM

Resolved August 18, 2017 at 1:58 PM

Args are not provided to *main-cli-fn* with optimizations

Description

Environment

Attachments

Activity

Details

Assignee

Reporter

Approval

Patch

Fix versions

Priority

Args are not provided to main-cli-fn with optimizations