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

go/pup: init at 0.3.9 #11300

Merged
merged 1 commit into from
Dec 21, 2015
Merged

go/pup: init at 0.3.9 #11300

merged 1 commit into from
Dec 21, 2015

Conversation

obadz
Copy link
Contributor

@obadz obadz commented Nov 27, 2015

(also required go/color: init at 9aae6aa)

First time I package a Go package.

  • What's the date field for?
  • Is there a better way to do what I did in renameImports?
  • Am I meant to invoke this as goPackages.pup.bin?

@mention-bot
Copy link

By analyzing the blame information on this pull request, we identified @wkennington, @benley and @copumpkin to be potential reviewers

@obadz
Copy link
Contributor Author

obadz commented Nov 27, 2015

cc @jb55

sha256 = "04j3fy1vk6xap8ad7k3c05h9b5mg2n1vy9vcyg9rs02cb13d3sy0";
propagatedBuildInputs = [ net ];
buildInputs = [ go-colorable color ];
renameImports = ''
Copy link
Contributor

Choose a reason for hiding this comment

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

Do it in a preBuild or postPatch or somewhere else. renameImports is for overriding the default renaming.

@lucabrunox
Copy link
Contributor

About date, should be very clear by reading the code of buildFromGitHub.

About .bin, yes.

@obadz
Copy link
Contributor Author

obadz commented Dec 5, 2015

fixed as requested.

@obadz
Copy link
Contributor Author

obadz commented Dec 10, 2015

@lethalman, anything else before this can get merged?

(also required go/color: init at 9aae6aa)
ttuegel added a commit that referenced this pull request Dec 21, 2015
@ttuegel ttuegel merged commit 226a83e into NixOS:master Dec 21, 2015
@obadz obadz deleted the go-pup branch January 4, 2016 12:40
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.

5 participants