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

Copy License to install #149

Merged
merged 2 commits into from
Jun 22, 2016
Merged

Conversation

danhedron
Copy link
Member

This only covers Linux and FreeBSD.

Anyone who knows what the right thing to do for windows and macOS is invited to tell me what to do.

@JayFoxRox
Copy link
Collaborator

What I said earlier on IRC mostly referred to including it with the build at all.
This is important if we have artifacts for download from appveyor or travis.

However, this is still fine I guess. But it's really up to how each distro manages it.
My understanding is that some distros have extra room for the license file in their package description.

I believe on Windows it's common practice to either zip up the project and have a license.txt OR show it in the installer - there is no designated folder to my knowledge.
There are some common GUI styling guides by Apple and Microsoft though and those might mention where to display such information.
The Windows one mentions the license terms for the Setup for example: https://msdn.microsoft.com/en-us/library/windows/desktop/dn742461(v=vs.85).aspx

@danhedron
Copy link
Member Author

I've added the file to the build directory too.

re the install: A lot of packages provide a LICENSE/COPYING file along with their license metadata, it seems reasonable to do.

We probably won't have an installer for some time, so copying to the build directory is fine by me for now.

@danhedron danhedron merged commit d36b813 into rwengine:master Jun 22, 2016
@danhedron danhedron deleted the org/copy-license branch July 4, 2016 22:40
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

2 participants