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
sdrangel : init at 4.11.7 #67357
sdrangel : init at 4.11.7 #67357
Conversation
de6cead
to
8c3635c
Compare
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.
This is my first PR / nixpkgs contribution so that list is a little overwhelming ^
We'll get through it step by step
f5446ae
to
ce4a05c
Compare
a50c3c1
to
a5a8634
Compare
Made the latest change and it now uses the nixpkgs serialdv and cm256cc |
@GrahamcOfBorg build sdrangel |
Oh just changed the rev to a number rather than hash |
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.
Some minor cleanups. Looks pretty good now. This may be the last round.
3d2fcb2
to
921015c
Compare
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.
Congrats to the completion of your first pull request and thanks for packaging this nice program.
Wew nice; thanks for everything :) |
Motivation for this change
Sdrangel was missing from nixpkgs
Things done
Added sdrangel derivation and myself to the maintainers list
sandbox
innix.conf
on non-NixOS)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)Notify maintainers
@markuskowa
This is my first PR / nixpkgs contribution so that list is a little overwhelming ^