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

Pysmbc #29939

Merged
merged 2 commits into from Oct 1, 2017
Merged

Pysmbc #29939

merged 2 commits into from Oct 1, 2017

Conversation

symphorien
Copy link
Member

Motivation for this change

https://askubuntu.com/questions/548361/browse-button-disabled-in-windows-printer-via-samba-for-printer-shares

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
    • Linux
  • N/A 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.

@orivej orivej merged commit 4f37b89 into NixOS:master Oct 1, 2017
@Moredread
Copy link
Contributor

Moredread commented Oct 1, 2017

This commit seems to break Travis and nox review builds, e.g. at https://travis-ci.org/NixOS/nixpkgs/jobs/282048362.

The license is wrong, "gplv2" instead of gpl2 or more correctly I guess gpl2Plus, if you look at the license notices in the c files and setup.py.

@orivej
Copy link
Contributor

orivej commented Oct 1, 2017

Fixed in c1e872d

@orivej
Copy link
Contributor

orivej commented Oct 1, 2017

Changed to gpl2Plus, thank you!

@symphorien symphorien deleted the pysmbc branch May 18, 2019 16:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants