A ticket + patch that fixes the regression welcome.

David


On Wed, Feb 26, 2014 at 1:58 PM, Travis Vachon <[email protected]>wrote:

> Hm - sorry to chime in late on this, but I'm not sure I follow the
> logic in the original question - :preamble and :target :nodejs weren't
> mutually exclusive, it was just the case that the default :preamble
> was the hashbang when using :target :nodejs
>
> As patched, it's no longer possible to exclude the hashbang when
> targetting nodejs, which breaks deployment in the parse.com cloud code
> environment. Couldn't you have used :target :nodejs and included both
> the hashbang and the sourcemap comment in the preamble?
>
> On Fri, Feb 21, 2014 at 2:14 PM, Michael Glaesemann
> <[email protected]> wrote:
> >
> > On Feb 21, 2014, at 13:56, David Nolen wrote:
> >
> >> JIRA ticket + patch welcome. Make sure to send in your Clojure CA,
> thanks!
> >
> > http://dev.clojure.org/jira/browse/CLJS-771
> >
> > As for the CA, I'm already listed on http://clojure.org/contributing.
> >
> > Cheers!
> >
> >
> >>
> >> David
> >>
> >>
> >> On Fri, Feb 21, 2014 at 1:52 PM, Michael Glaesemann <
> [email protected]>wrote:
> >>
> >>> Currently the :preamble and :target :nodejs compiler options are
> mutually
> >>> exclusive.
> >>>
> >>> I recently had a situation where I wanted to use a preamble with node:
> I
> >>> came across
> >>> a nodejs module for [source map support](
> >>> http://github.com/evanw/node-source-map-support).
> >>>
> >>> It requires adding a bit of code at the top of the compiled file to
> >>>
> >>>    //# sourceMappingURL=path/to/source.map
> >>>    require('source-map-support').install();
> >>>
> >>> While the source map support is still rudimentary, I don't see a reason
> >>> not to allow
> >>> nodejs targets to have a preamble.
> >>>
> >>> I've attached a patch that adds the preamble directly after the node
> >>> hashbang.
> >>>
> >>> If this looks like something that would be generally useful, I'm happy
> to
> >>> create a JIRA issue and attach the patch.
> >>>
> >>> Cheers,
> >>>
> >>> Michael Glaesemann
> >>> grzm seespotcode net
> >>>
> >>>
> >>> --
> >>> Note that posts from new members are moderated - please be patient with
> >>> your first post.
> >>> ---
> >>> You received this message because you are subscribed to the Google
> Groups
> >>> "ClojureScript" group.
> >>> To unsubscribe from this group and stop receiving emails from it, send
> an
> >>> email to [email protected].
> >>> To post to this group, send email to [email protected].
> >>> Visit this group at http://groups.google.com/group/clojurescript.
> >>>
> >>
> >> --
> >> Note that posts from new members are moderated - please be patient with
> your first post.
> >> ---
> >> You received this message because you are subscribed to the Google
> Groups "ClojureScript" group.
> >> To unsubscribe from this group and stop receiving emails from it, send
> an email to [email protected].
> >> To post to this group, send email to [email protected].
> >> Visit this group at http://groups.google.com/group/clojurescript.
> >
> > Michael Glaesemann
> > grzm seespotcode net
> >
> >
> >
> > --
> > Note that posts from new members are moderated - please be patient with
> your first post.
> > ---
> > You received this message because you are subscribed to the Google
> Groups "ClojureScript" group.
> > To unsubscribe from this group and stop receiving emails from it, send
> an email to [email protected].
> > To post to this group, send email to [email protected].
> > Visit this group at http://groups.google.com/group/clojurescript.
>
> --
> Note that posts from new members are moderated - please be patient with
> your first post.
> ---
> You received this message because you are subscribed to the Google Groups
> "ClojureScript" group.
> To unsubscribe from this group and stop receiving emails from it, send an
> email to [email protected].
> To post to this group, send email to [email protected].
> Visit this group at http://groups.google.com/group/clojurescript.
>

-- 
Note that posts from new members are moderated - please be patient with your 
first post.
--- 
You received this message because you are subscribed to the Google Groups 
"ClojureScript" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/clojurescript.

Reply via email to