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

tewisay: init at git-2017-04-14 #33457

Closed
wants to merge 8 commits into from
Closed

tewisay: init at git-2017-04-14 #33457

wants to merge 8 commits into from

Conversation

Chiiruno
Copy link
Contributor

@Chiiruno Chiiruno commented Jan 5, 2018

Motivation for this change

I like cowsay, so I wanted to add a fun alternative to this repository.
I'm very new to this, so if I'm not following protocol correctly, please just tell me what to do.
Making a nixpkg is a tad confusing to a beginner.

Things done
  • Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
  • Built on platform(s)
    • NixOS
    • macOS
    • other Linux distributions
  • Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
  • Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Fits CONTRIBUTING.md.

@Chiiruno Chiiruno changed the title Added tewisay and it's dependencies tewisay: init at e3fc387 Jan 5, 2018
Copy link
Member

@nlewo nlewo left a comment

Choose a reason for hiding this comment

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

Welcome to the Nix packaging!
I will try to help you in this process.

  1. Squash your commits (no merge commit in your PR)
  2. The format of a package which comes from a commit is package-name-YYYY-MM-DD. See https://nixos.org/nixpkgs/manual/#sec-package-naming
  3. Set the licence of the package you are adding.

@@ -0,0 +1,20 @@
{ stdenv, buildGoPackage, fetchgit, asciidoc, go-runewidth, pflag }:
Copy link
Member

Choose a reason for hiding this comment

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

Your package fails to build.
You have to specify Go dependencies by using the goDeps attributes. See https://nixos.org/nixpkgs/manual/#ex-buildGoPackage-3
These dependencies can be generated by some tools such as go2nix.

description = "Provides functions to get fixed width of the character or string";
homepage = https://github.com/mattn/go-runewidth;
platforms = stdenv.lib.platforms.all;
maintainers = [ stdenv.lib.maintainers.rob ];
Copy link
Member

Choose a reason for hiding this comment

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

Are you rob ? :/

Copy link
Contributor Author

@Chiiruno Chiiruno Jan 5, 2018

Choose a reason for hiding this comment

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

I wasn't sure if I could change that. Do I have to add my name to stdenv.lib.maintainers or can I just add it there?

Edit: Got it, was easier than I thought.

@@ -2412,6 +2412,8 @@ with pkgs;

go-mtpfs = callPackage ../tools/filesystems/go-mtpfs { };

go-runewidth = callPackage ../development/tools/go-runewidth { };
Copy link
Member

Choose a reason for hiding this comment

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

Useless since this is a Go dep

@@ -4024,6 +4026,8 @@ with pkgs;

pfstools = callPackage ../tools/graphics/pfstools { };

pflag = callPackage ../development/tools/pflag { };
Copy link
Member

Choose a reason for hiding this comment

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

Useless since this is a go dependency.

@Chiiruno Chiiruno changed the title tewisay: init at e3fc387 tewisay: init at git-2017-04-17 Jan 5, 2018
@Chiiruno Chiiruno changed the title tewisay: init at git-2017-04-17 tewisay: init at git-2017-04-14 Jan 5, 2018
@Chiiruno
Copy link
Contributor Author

Chiiruno commented Jan 5, 2018

I'm going to close this and then reopen with a new fork, I'm having a hard time with squashing and rebasing.

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.

None yet

5 participants