Skip to content

Include license file #18

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.

Sign up for GitHub

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

Open
toddrme2178 wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Include license file #18

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
6ec178b
Create LICENSE.txt
toddrme2178 Jan 14, 2019
bd8b428
Include license and examples in sdists
toddrme2178 Jan 14, 2019
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
28 changes: 28 additions & 0 deletions LICENSE.txt
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,28 @@
This software is OSI Certified Open Source Software.
OSI Certified is a certification mark of the Open Source Initiative.

Copyright (c) 2009-2018, Enthought, Inc.
Copy link
Copy Markdown
Contributor

@stevenjkern stevenjkern Apr 16, 2019

Choose a reason for hiding this comment

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

Thanks so much for this, would you mind changing the copyright statement to the following? We're trying to keep all of our copyright headers uniform.

(C) Copyright 2009-2019 Enthought, Inc., Austin, TX
All rights reserved.

Copy link
Copy Markdown
Author

@toddrme2178 toddrme2178 May 16, 2019

Choose a reason for hiding this comment

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

I would be happy to change it. However, I have looked through all of the enthought packages and none of them use the copyright header you gave. They all use the copyright header I had originally (which is because I copied it from the other licenses you were using). Again, if you really want it changed I can do so. But that would make it inconsistent with the other packages, so I wanted to double-check to make sure that the change is correct before I do it.

Copy link
Copy Markdown
Contributor

@stevenjkern stevenjkern May 22, 2019

Choose a reason for hiding this comment

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

Thanks for checking that, @toddrme2178. Indeed the new copyright header is what we want going forward, but we do not have to make any sweeping changes of every file, just updating existing copyright headers or adding new ones whenever a file happened to be touched in a PR. Since this is such a small project, I went ahead and added headers to all the existing code a few weeks back. We just happen to be the first to implement it for this project. All of the other ETS projects will catch up slowly but surely.

All rights reserved.

Redistribution and use in source and binary forms, with or without
modification, are permitted provided that the following conditions are met:

* Redistributions of source code must retain the above copyright notice, this
list of conditions and the following disclaimer.
* Redistributions in binary form must reproduce the above copyright notice,
this list of conditions and the following disclaimer in the documentation
and/or other materials provided with the distribution.
* Neither the name of Enthought, Inc. nor the names of its contributors may
be used to endorse or promote products derived from this software without
specific prior written permission.

THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS" AND
ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED
WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE
DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT OWNER OR CONTRIBUTORS BE LIABLE FOR
ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES
(INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES;
LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON
ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
(INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS
SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
6 changes: 6 additions & 0 deletions MANIFEST.in
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,6 @@
include LICENSE.txt
include MANIFEST.in
include README.rst
include .coveragerc

graft examples