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

fonts/gdouros: correct license to unfree #79679

Merged
merged 1 commit into from
Feb 29, 2020

Conversation

emilazy
Copy link
Member

@emilazy emilazy commented Feb 10, 2020

Motivation for this change

I assume the licence for these fonts changed at some point, but the current licensing conditions are actually rather byzantine and definitely non-Free. They actually prohibit redistribution, so the existing Hydra builds of these packages (example) are a liability; I'm not sure who (if anyone) would be able to purge them.

Things done
  • Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
  • 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 nixpkgs-review --run "nixpkgs-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Determined the impact on package closure size (by running nix path-info -S before and after)
  • Ensured that relevant documentation is up to date
  • Fits CONTRIBUTING.md.

@grahamc
Copy link
Member

grahamc commented Feb 10, 2020

@emilazy
Copy link
Member Author

emilazy commented Feb 10, 2020

Looks like the noncommercial requirement was added between January and March 2018; the licence forbidding redistribution first appeared in the April 20 2018 archive, so any builds on Hydra from then onwards are problematic.

@ajs124
Copy link
Member

ajs124 commented Feb 10, 2020

Is there any statement anywhere from George Douros, the maintainer of these fonts, on this?

I found multiple other projects (arch linux, some homebrew thing, some math font thing(?)) that stumbled upon this license change, but none of the discussions had any reasoning from the guy.

While he most likely had/has some reason for this license, he might also not be aware of what this change in license means. Maybe someone should try and reach out to him to figure out if this is actually what he intended and if he might want to change the licenses again.

@ofborg ofborg bot added 8.has: clean-up 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild 10.rebuild-linux: 0 This PR does not cause any packages to rebuild labels Feb 10, 2020
@emilazy
Copy link
Member Author

emilazy commented Feb 10, 2020

I haven't found anything to that effect. I'm happy to try emailing and asking if he's considered fully free font licences, but unfortunately I doubt you go from "do anything" to "do anything noncommercial" to "do almost nothing" without intending the restrictions. If a new version under a free licence is released, that would be great, but the current package version's licence should probably be fixed until then anyway.

@emilazy
Copy link
Member Author

emilazy commented Feb 10, 2020

@GrahamcOfBorg build aegan aegyptus akkadian assyrian eemusic maya symbola textfonts unidings

homepage = http://users.teilar.gr/~g1951d/;
hydraPlatforms = []; # redistribution not permitted
Copy link
Member

Choose a reason for hiding this comment

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

https://hydra.nixos.org won't build unfree packages regardless of this attribute.

Copy link
Member

@lukateras lukateras left a comment

Choose a reason for hiding this comment

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

LGTM otherwise.

@lukateras
Copy link
Member

lukateras commented Feb 27, 2020

Would like to merge this and #79681 on Feb 28 9 pm UTC. cc @grahamc

@emilazy
Copy link
Member Author

emilazy commented Feb 29, 2020

does this need backporting to 20.03 too?

@lukateras
Copy link
Member

does this need backporting to 20.03 too?

Actually, yes! (e.g. see #81337)

@lukateras lukateras merged commit c6c200f into NixOS:master Feb 29, 2020
@emilazy emilazy deleted the fix-gdouros-font-licenses branch March 3, 2020 01:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
8.has: clean-up 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild 10.rebuild-linux: 0 This PR does not cause any packages to rebuild
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants