Skip to content

fix(Makefile): add a glideup build target#62

Merged
arschles merged 1 commit into
deis:masterfrom
arschles:make-glideup
Feb 5, 2016
Merged

fix(Makefile): add a glideup build target#62
arschles merged 1 commit into
deis:masterfrom
arschles:make-glideup

Conversation

@arschles

@arschles arschles commented Feb 5, 2016

Copy link
Copy Markdown
Member

Fixes #58

@arschles arschles self-assigned this Feb 5, 2016
@arschles arschles added this to the v2.0-beta1 milestone Feb 5, 2016
@bacongobbler

Copy link
Copy Markdown
Member

isn't make bootstrap the equivalent? From glide --help:

update, up, install          Update or install a project's dependencies

@arschles

arschles commented Feb 5, 2016

Copy link
Copy Markdown
Member Author

@bacongobbler don't believe so.

According to the update documentation: Download or update all of the libraries listed in the glide.yaml file ... A glide.lock file will be created or updated with the dependencies pinned to specific versions.

And, according to the install documentation: This will read the glide.lock file, making sure it's tied to the glide.yaml file, and install the commit id specific versions there ... When the glide.lock file doesn't tie to the glide.yaml file, such as there being a change, it will provide an error. Running glide up will recreate the glide.lock file when updating the dependency tree.

Also, based on my experience so far, glide up has two major differences from install:

  1. If you've changed your glide.yaml file, you can run it to regenerate your glide.lock file
  2. If you've specified a dependency, but no specific version, in your glide.yaml, you can run glide update to update its version in the glide.lock

@bacongobbler

Copy link
Copy Markdown
Member

gotcha. I'm okay with this since it doesn't affect anything else.

@arschles

arschles commented Feb 5, 2016

Copy link
Copy Markdown
Member Author

thanks @bacongobbler

arschles added a commit that referenced this pull request Feb 5, 2016
fix(Makefile): add a glideup build target
@arschles arschles merged commit 20633be into deis:master Feb 5, 2016
@arschles arschles deleted the make-glideup branch February 5, 2016 17:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants