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 missing texlive packages for Sphinx #13

Merged
merged 1 commit into from
Oct 19, 2018

Conversation

Lnaden
Copy link
Contributor

@Lnaden Lnaden commented Oct 19, 2018

This should help advance omnia-md/conda-recipes#911 by pushing a change to the base docker image for the conda-build version workaround.

Once this is merged in @jchodera , all you should have to do is have this build on your Docker account (automatic?), then re-trigger all the various cuda images.

@jchodera
Copy link
Member

Thanks! I'm a bit worried about corrupting our working images, but if this rebuild is unsuccessful, it won't disrupt builds until I rebuild the CUDA images, so I think this is OK to try!

@jchodera jchodera merged commit 4642a97 into omnia-md:texlive18 Oct 19, 2018
@Lnaden
Copy link
Contributor Author

Lnaden commented Oct 19, 2018

Not sure how it would corrupt? This update would not change how the downstream's are constructed are they? Unless the source of the downloaded material changed, this should build the same way.

You could make another spinoff builds just in case? Or make local copies of the current images in case you have to push a backup

@jchodera
Copy link
Member

Not sure how it would corrupt? This update would not change how the downstream's are constructed are they? Unless the source of the downloaded material changed, this should build the same way.

That's what I was worried about---some upstream packages might have changed!

@jchodera
Copy link
Member

It broke already:
https://hub.docker.com/r/jchodera/omnia-linux-anvil/builds/bzebtyryendlkbtnvde5ul9/

Build failed: ADD failed: Get https://tycho.usno.navy.mil/timer.html: x509: certificate signed by unknown authority

Any ideas?

@Lnaden Lnaden mentioned this pull request Oct 19, 2018
@Lnaden
Copy link
Contributor Author

Lnaden commented Oct 19, 2018

See #14

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