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
python3Packages.pyspinel: fix build #90108
Conversation
b97a5cb
to
7d38b91
Compare
7d38b91
to
91d91cb
Compare
91d91cb
to
ba1b373
Compare
37449fb
to
4ad0890
Compare
sorry for all the round about trips.
for more examples, please look at https://github.com/NixOS/nixpkgs/blob/master/.github/CONTRIBUTING.md#submitting-changes in your case, the commit message should be:
|
4ad0890
to
7f385ce
Compare
565abf8
to
50f9e5c
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.
also, the package is missing a license, please include
1d74242
to
c4b4e98
Compare
Adopt upstream patch accepted here: openthread/pyspinel#98 Signed-off-by: Sirio Balmelli <sirio@b-ad.ch> Co-Authored-By: Jon <jonringer@users.noreply.github.com>
c4b4e98
to
ea4d917
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.
LGTM
Result of nixpkgs-review pr 90108 1
2 packages built:
- python37Packages.pyspinel
- python38Packages.pyspinel
Motivation for this change
Update to reference an upstream fix to build on python >=3.3
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)