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.

Sign up for GitHub

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

Jump to bottom

Update README to use -c conda-forge #397

Merged
merged 1 commit into from
Feb 14, 2024
Merged

Update README to use -c conda-forge #397

merged 1 commit into from
Feb 14, 2024

Conversation

joshmoore
Copy link
Member

@joshmoore joshmoore mentioned this pull request Feb 13, 2024
Closed
will-moore
will-moore approved these changes Feb 13, 2024
View reviewed changes
Copy link
Member

@will-moore will-moore left a comment

Choose a reason for hiding this comment

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

Works for me!

@jburel
Copy link
Member

jburel commented Feb 13, 2024 •
edited
Loading

Does it mean the correct version of ice is installed?
The reason we reverted to use the ome conda channel was because ice 3.7 bindings were installed when using the conda-forge channel
and in many cases, building binding Ice does not work

see #369

@joshmoore
Copy link
Member Author

for me -c ome is no longer working because of arm, so I'm solely using conda-forge these days. I'm not sure how Ice 3.7 would be downloaded based on https://github.com/conda-forge/omero-py-feedstock/blob/main/recipe/meta.yaml#L35

@jburel
Copy link
Member

jburel commented Feb 13, 2024 •
edited
Loading

Things might have changed
I re-ran #332 and it works fine

@jburel jburel mentioned this pull request Feb 14, 2024
Merged
@jburel
Copy link
Member

jburel commented Feb 14, 2024

corresponding web PR ome/omero-web#533

@jburel jburel merged commit d676cac into master Feb 14, 2024
13 checks passed
@joshmoore joshmoore deleted the joshmoore-patch-1 branch February 14, 2024 12:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@will-moore will-moore will-moore approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@joshmoore @jburel @will-moore