Skip to content

Conversation

@KristofferC
Copy link
Member

@KristofferC KristofferC commented Oct 2, 2019

Fixes #33182

Running exit(0) prevents the cleanup procedure in include to run which messes up the state of current_task().storage[:SOURCE_PATH].

@maleadt, could you try this out.

@KristofferC KristofferC added bugfix This change fixes an existing bug backport 1.3 labels Oct 2, 2019
@KristofferC KristofferC changed the title fix not messing up current_taks().storage[:SOURCE_PATH] by allowing the finally in include to run in generate_precompile.jl fix not messing up current_task().storage[:SOURCE_PATH] by allowing the finally in include to run in generate_precompile.jl Oct 2, 2019
@KristofferC KristofferC changed the title fix not messing up current_task().storage[:SOURCE_PATH] by allowing the finally in include to run in generate_precompile.jl stop hard-exiting from generate_precompile when JULIA_PRECOMPILE=0 Oct 2, 2019
Copy link
Member

@maleadt maleadt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Works perfectly!

@KristofferC KristofferC merged commit 16a09b4 into master Oct 2, 2019
@KristofferC KristofferC deleted the kc/path_no_precompile branch October 2, 2019 09:37
KristofferC added a commit that referenced this pull request Oct 3, 2019
… the finally in include to run (#33447)

(cherry picked from commit 16a09b4)
@KristofferC KristofferC mentioned this pull request Oct 3, 2019
18 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bugfix This change fixes an existing bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

JULIA_PRECOMPILE=0 build doesn't find command-line file arguments

3 participants