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

add template to build ice 3.6 #16

Closed
wants to merge 3 commits into from
Closed

add template to build ice 3.6 #16

wants to merge 3 commits into from

Conversation

jburel
Copy link
Member

@jburel jburel commented Dec 3, 2020

add template to build ice 3.6
used for ome/zeroc-ice-ubuntu2004#1

The same template will can be applied to various repositories where Ice is built from source.
I have kept the omero logo. I could only find a png see https://github.com/zeroc-ice/ice

Copy link
Member

@joshmoore joshmoore left a comment

Choose a reason for hiding this comment

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

No objection to merging this, but I assume this won't be a template that many people (on the team or elsewhere) will ever need to run due to the specificity of the setup. (I also hope we won't be creating too many more of these zeroc build repos.)

- uses: actions/setup-python@v2
- name: Build
run: |
docker build -t builder .
Copy link
Member

Choose a reason for hiding this comment

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

This assumes a certain API on the side of the Dockerfile.

@jburel
Copy link
Member Author

jburel commented Dec 8, 2020

Since those repositories do not change very often and we are slowing moving away from Ice. So I will do the work in the repositories and do not add a template.

@jburel jburel closed this Dec 8, 2020
@jburel jburel deleted the zeroc-ice branch March 22, 2022 10:32
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