-
Notifications
You must be signed in to change notification settings - Fork 12
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
Upgrade DFX to 0.22.0 #79
Conversation
That would only apply to the first candidate path |
REGISTRY_FOUND=$(find "$BASE_PATH" -type d -name ic_registry_local_store 2>/dev/null) | ||
|
||
if [ $(echo "$REGISTRY_FOUND" | wc -l) -gt 1 ]; then | ||
echo "Error: Multiple ic_registry_local_store directories found" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
echo "Error: Multiple ic_registry_local_store directories found" | |
echo "Error: Multiple ic_registry_local_store directories found; please start your local replica using `dfx start --clean`" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry, I thought it wouldn't let me merge until I resolved this
When upgrading DFX from v0.19.0 to v0.22.0, there was a change in where DFX stores the registry info, so I added a new place for sns-testing to look for it in. I tried it on my mac and on CI and it seems to work.
I left the other places it looked in case any of them are used in some circumstances, but I don't think they are for this version of DFX. (Perhaps they would be useful if someone wanted to use an old version of DFX though.) The dfx start docs say that: