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
slackcat: init at 1.6 #96222
slackcat: init at 1.6 #96222
Conversation
@GrahamcOfBorg How can I get this PR merged? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All packages need meta information
@dali99 Thanks for your review. Do I need to add myself as a maintainer or the package developer themself? |
You add yourself For more information you can read In your case you should have two commits
With the relevant information to The other commit should be
adding the package itself. There should be no more commits (so you need to force push your branch) |
You seem to have based your PR on a branch from May in 2019 The
Just commenting so that you'll be aware of this. I'd rebase on current master and add at least |
add callPackage minor refactor of rev Update pkgs/tools/networking/slackcat/default.nix Co-authored-by: Daniel Løvbrøtte Olsen <daniel.olsen99+GitHub@gmail.com> slackcat: init at 1.6
maintainers: add nasadorian
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Built sucessfully!
🎉
@dali99 Is there a way to kick the build? I'm not sure what happened here. |
ah yes we should ping someone |
@GrahamcOfBorg eval |
Co-authored-by: Sandro <sandro.jaeckel@gmail.com>
owner = "bcicen"; | ||
repo = "${pname}"; | ||
rev = "v${version}"; | ||
sha256 = "1ha9b2dffpxqs159ldqdh1aibn3316xab19vdn2yjgk8r508wm31"; | ||
}; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I noticed they have a go.mod https://github.com/bcicen/slackcat/blob/0b70f6e0ab9c41e4782bfbe74e934ddb49d520b8/go.mod, do you mind trying to do a buildGoModule
instead? package is deprecated
|
||
src = fetchFromGitHub { | ||
owner = "bcicen"; | ||
repo = "${pname}"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
repo = "${pname}"; | |
repo = pname; |
Please follow CONTRIBUTING.md and manual#submitting-changes-making-patches and squash the fix-up commits.
|
@@ -0,0 +1,22 @@ | |||
{ stdenv, buildGoPackage, fetchFromGitHub }: | |||
|
|||
buildGoPackage rec { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please use buildGoModule which does not require use to use a deps.nix.
I marked this as stale due to inactivity. → More info |
Closing due to inactivity from author. |
Motivation for this change
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)