Skip to content
This repository has been archived by the owner on Feb 1, 2019. It is now read-only.

windows support #41

Merged
merged 1 commit into from
Apr 15, 2014
Merged

windows support #41

merged 1 commit into from
Apr 15, 2014

Conversation

remco138
Copy link
Contributor

Windows does not know how to handle bash nor the ./ prefix (isn't this unnecessary anyway?)
This pull request includes a port of the bash script to batch and a platform condition in the plugin.

The first eval prints garbage from ghc starting up, after that all should work fine, not sure if this is a windows specific issue, if so, I could look into that too.

@heyLu
Copy link
Contributor

heyLu commented Apr 15, 2014

Oh, that's cool, thank you!

The printouts from the initial run are from building light-haskell and it's dependencies. (Because it's in a sandbox it has to compile all dependencies, which are quite a few.)

So, thanks again for doing this! :)

heyLu added a commit that referenced this pull request Apr 15, 2014
@heyLu heyLu merged commit fc8a4e9 into jetaggart:master Apr 15, 2014
heyLu added a commit that referenced this pull request Apr 15, 2014
it was missing the import of the platform namespace and the compiled
version of the code.
@heyLu
Copy link
Contributor

heyLu commented Apr 15, 2014

Just two small tips for the future:

  • to run this on other machines without evaling the code you'll have to commit the haskell_compiled.js` as well
  • and you need to :require new namespaces

(I mess this up all the time, that's probably one drawback of LightTable: That stuff you do interactively is not also saved automatigically.)

I've fixed those two things and released 0.2.6 with windows support.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants