Skip to content

Strict handling of UTF8 files (fixes fd issue)#2338

Merged
paf31 merged 2 commits into
purescript:masterfrom
chrisdone:strict-utf8-files
Sep 29, 2016
Merged

Strict handling of UTF8 files (fixes fd issue)#2338
paf31 merged 2 commits into
purescript:masterfrom
chrisdone:strict-utf8-files

Conversation

@chrisdone

Copy link
Copy Markdown
Contributor

I found that on OS X when trying to build the basic purescript-react example I got this error:

psc: /Users/chris/Projects/psc/purescript-dom/src//DOM/HTML/Location.js: openFile: resource exhausted (Too many open files)

This was fixed in #1674 for psc-bundle, but actually my error comes from psc itself, because it uses readUTF8File in various places and the fix was placed only in psc-bundle. Instead, I've fixed the problem at the source by using the already-present utf8-string package to simply decode the strict ByteString into a String. This is also what hindent does.

This fixes the issue on my OS X.

@paf31

paf31 commented Sep 29, 2016

Copy link
Copy Markdown
Contributor

Looks good, thanks! Could you please update CONTRIBUTORS.md and I'll get this merged for 0.10.1?

@garyb

garyb commented Sep 29, 2016

Copy link
Copy Markdown
Member

I'll get this merged for 0.10.1

Isn't it just going to be 0.10? We've not released a non-RC yet...

@paf31

paf31 commented Sep 29, 2016

Copy link
Copy Markdown
Contributor

Last time we used 0.9.1 since people had been using the 0.9 release candidate for a while, but I don't mind.

@chrisdone

Copy link
Copy Markdown
Contributor Author

Done!

@paf31

paf31 commented Sep 29, 2016

Copy link
Copy Markdown
Contributor

I'm not sure why GitHub thinks there is a conflict here.

@chrisdone

Copy link
Copy Markdown
Contributor Author

I rebased. =)

@paf31 paf31 merged commit 2cad878 into purescript:master Sep 29, 2016
@chrisdone chrisdone deleted the strict-utf8-files branch September 30, 2016 05:37
@paf31 paf31 mentioned this pull request Oct 1, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants