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

Added note to use exeflags to load environment. #33607

Merged
merged 2 commits into from
Oct 21, 2019
Merged

Added note to use exeflags to load environment. #33607

merged 2 commits into from
Oct 21, 2019

Conversation

BioTurboNick
Copy link
Contributor

Based on discussion here: https://discourse.julialang.org/t/code-loading-with-workers-and-modules/30064/2

I believe the added note is helpful because a partial solution can now be found immediately after the limitation is described. Assuming I understand right.

@BioTurboNick
Copy link
Contributor Author

Hmm, okay that advice didn't work... I'll change if I find a solution.

@SebastianM-C
Copy link
Contributor

Also see #28781 for details.

@StefanKarpinski
Copy link
Sponsor Member

Are you saying that this documentation change isn't correct? This should be closed in that case so that we don't accidentally merge incorrect documentation.

@BioTurboNick
Copy link
Contributor Author

@StefanKarpinski I made a change to make it correct after initially thinking not. It was only part of the answer.

@StefanKarpinski
Copy link
Sponsor Member

Ok, I'm still unclear on whether this should be merged or not...

@BioTurboNick
Copy link
Contributor Author

I'll leave that up to you all. What I wrote is what I would have needed to see in the docs to make it work. And it does work as currently written.

@StefanKarpinski StefanKarpinski merged commit 617f797 into JuliaLang:master Oct 21, 2019
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.

3 participants