Skip to content

Install script#43

Open
gasche wants to merge 3 commits intoOCamlPro:masterfrom
gasche:install-script
Open

Install script#43
gasche wants to merge 3 commits intoOCamlPro:masterfrom
gasche:install-script

Conversation

@gasche
Copy link
Contributor

@gasche gasche commented Oct 8, 2017

These are three small changes I made to the opam-builder install script (which I have been using interactively instead of as a complete script):

  • copy the env-initialization logic in a separate env.sh file that I can call from scripts to always start in the same environment
  • replace git@github... links by https://github... links to allow to run the script from a machine without SSH credentials
  • refresh indexes before building the switches (I now think that this is useless, but it cannot hurt and I fixed a typo along the way)

gasche added 3 commits October 5, 2017 17:18
This lets you run the script from a machine that
does not have SSH keys to access a github account.
…cript

This is helpful when replaying scripts/install-opam-builder.sh command
manually.

However, it introduces a duplication that hurts maintenance. I'm not
sure how to invoke env.sh from install-opam-builder.sh correctly.
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.

1 participant