Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support for standard library #129

Merged
merged 4 commits into from
Apr 21, 2015
Merged

Support for standard library #129

merged 4 commits into from
Apr 21, 2015

Conversation

supercooldave
Copy link

Added support for standard library.
Default directory location, which is baked into compiler when compiler is built.
One module is placed in "prototype" library and a corresponding test has been added.

@kaeluka
Copy link
Contributor

kaeluka commented Apr 20, 2015

The change works for me, tests pass on OS X. Apart from the comments regarding the environment variable, I have no nag.

The comment regarding the env variable is not meant to imply it's a bad idea, btw. This may well be what we should keep doing for baking similar paths into the frontend in the future. If someone knows nicer ways, let's hear them!

@supercooldave
Copy link
Author

The compiler binds to a directory within the git directory structure. If you have two git directories, then the compilers within those will bind to the possibly different local versions of the libraries.

To support a bit more flexibility, I'll add a flag that allows you to specify the location of the std lib.

In the future, we will put the standard lib in ~/.encore, but we first want to do some research on the best way of handling packages (or bundles, as we call them). So, if there's a masters or bachelors student looking for a project, then doing a survey of package managers would be useful.

@kaeluka
Copy link
Contributor

kaeluka commented Apr 21, 2015

Alright, that makes sense!

kaeluka pushed a commit that referenced this pull request Apr 21, 2015
Support for standard library
@kaeluka kaeluka merged commit cee9257 into parapluu:master Apr 21, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants