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

Test new locations #446

Merged
merged 9 commits into from
Oct 5, 2021
Merged

Test new locations #446

merged 9 commits into from
Oct 5, 2021

Conversation

ChrisRackauckas
Copy link
Member

No description provided.

@ChrisRackauckas
Copy link
Member Author

@staticfloat could I get some help here? It looks like BuildKite can't install Conda anymore?

@staticfloat
Copy link
Contributor

I think Conda may require a Pkg.build() after the Pkg.instantiate().

@ChrisRackauckas
Copy link
Member Author

It should be doing that?

Pkg.instantiate()
Pkg.build()

@staticfloat
Copy link
Contributor

I think the issue is that SCIMLTutorials itself depends (transitively) on Conda, so you need to do that here too:

julia --project=. -e 'using Pkg; Pkg.instantiate()'

We didn't used to need to do this, so perhaps some downstream dependency added Conda recently.

@ChrisRackauckas
Copy link
Member Author

I am assuming that something about the instantiation is off because of #445, but I can't figure out where it's taking place

@ChrisRackauckas
Copy link
Member Author

Interesting, it looks like it did run the instantiation now (https://buildkite.com/julialang/scimltutorials-dot-jl/builds/114#58e06ad6-039e-4adf-8862-7d6ac67c2874/532-537) but that did nothing...?

@staticfloat
Copy link
Contributor

By default, we cache depots, so that you don't have to re-download every single time.

@ChrisRackauckas ChrisRackauckas deleted the ChrisRackauckas-patch-1 branch October 5, 2021 19:59
@ChrisRackauckas
Copy link
Member Author

So I need to do a new PR? I'll see.

@ChrisRackauckas ChrisRackauckas restored the ChrisRackauckas-patch-1 branch October 5, 2021 20:00
@ChrisRackauckas ChrisRackauckas merged commit cbe5048 into master Oct 5, 2021
@ChrisRackauckas ChrisRackauckas deleted the ChrisRackauckas-patch-1 branch October 5, 2021 20:01
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