Skip to content
This repository was archived by the owner on Sep 9, 2020. It is now read-only.

add version command to give dep's version #51

Closed
wants to merge 2 commits into from
Closed

add version command to give dep's version #51

wants to merge 2 commits into from

Conversation

jessfraz
Copy link
Contributor

@jessfraz jessfraz commented Dec 16, 2016

This makes it so we can do this:

$ ./dep version
dep version 0.0.0 3a8f368 linux/amd64

It also adds a Makefile for running with the specific flags to input the version into the command and/or cross-compile the binaries for various platforms which we will ultimately need if we decide to release this.

Now I am unsure what the standard Go team practices are for Makefiles but I have a feeling they are not exactly fans. I am though a huge fan of make.... so let the bikeshed begin hahaha

Also I added a help command to our Makefile so you can do this:

$ make help
build                          Builds a dynamic executable
clean                          Cleanup any build binaries
cross                          Builds the cross-compiled binaries, creating a clean directory structure (eg. GOOS/GOARCH/binary)
static                         Builds a static executable

@freeformz
Copy link

I suck at advanced Makefileing, but otherwise 👍 from me.

Verified

This commit was signed with the committer’s verified signature.
jessfraz Jess Frazelle
Signed-off-by: Jess Frazelle <[email protected]>

Verified

This commit was signed with the committer’s verified signature.
jessfraz Jess Frazelle
Signed-off-by: Jess Frazelle <[email protected]>
@@ -0,0 +1,64 @@
# Set an output prefix, which is the local directory if not specified
Copy link
Contributor

Choose a reason for hiding this comment

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

Add a comment that describes what this does.

@jessfraz
Copy link
Contributor Author

jessfraz commented Jan 5, 2017

closing for now we can discuss other alternatives to my lord and savior make

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants