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

Work around precompiling packages with no version entry #2251

Merged
merged 1 commit into from
Dec 2, 2020

Conversation

staticfloat
Copy link
Member

@staticfloat staticfloat commented Dec 1, 2020

While experimenting with non-precompiled stdlibs, I found that a
Project.toml that has no version entry set will cause precompile()
to fail in a nasty way.

Fixes #2146

While experimenting with non-precompiled stdlibs, I found that a
`Project.toml` that has no `version` entry set will cause `precompile()`
to fail in a nasty way.
@vtjnash
Copy link
Member

vtjnash commented Dec 1, 2020

Does this fix #2146?

@staticfloat
Copy link
Member Author

Probably; that error looks exactly like the one I worked around.

@KristofferC KristofferC merged commit f9307f8 into master Dec 2, 2020
@KristofferC KristofferC deleted the sf/precompile_no_version branch December 2, 2020 10:00
KristofferC pushed a commit that referenced this pull request Dec 2, 2020
While experimenting with non-precompiled stdlibs, I found that a
`Project.toml` that has no `version` entry set will cause `precompile()`
to fail in a nasty way.

(cherry picked from commit f9307f8)
IanButterworth added a commit to IanButterworth/Pkg.jl that referenced this pull request Dec 3, 2020
KristofferC pushed a commit that referenced this pull request Dec 3, 2020
…th no `version` entry) (#2261)

* missing change from workaround packages with no version entry #2251

* ensure one more print loop to tidy up the last task

* put a bit of random offset into the spinner animations

* include a package with no version number in tests
KristofferC pushed a commit that referenced this pull request Dec 3, 2020
…th no `version` entry) (#2261)

* missing change from workaround packages with no version entry #2251

* ensure one more print loop to tidy up the last task

* put a bit of random offset into the spinner animations

* include a package with no version number in tests

(cherry picked from commit f9b60d4)
staticfloat added a commit that referenced this pull request Jan 6, 2021
While experimenting with non-precompiled stdlibs, I found that a
`Project.toml` that has no `version` entry set will cause `precompile()`
to fail in a nasty way.

(cherry picked from commit f9307f8)
staticfloat pushed a commit that referenced this pull request Jan 6, 2021
…th no `version` entry) (#2261)

* missing change from workaround packages with no version entry #2251

* ensure one more print loop to tidy up the last task

* put a bit of random offset into the spinner animations

* include a package with no version number in tests

(cherry picked from commit f9b60d4)
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.

Packages outside sysimage without versions in Project file are not compatible with Pkg
3 participants