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

[WIP] proposed changes adding the MIT license #69

Closed
wants to merge 1 commit into from

Conversation

lsmith77
Copy link
Member

@@ -187,7 +210,7 @@
same "printed page" as the copyright notice for easier
identification within third-party archives.

Copyright [yyyy] [name of copyright owner]
Copyright 2010-2013 Liip and others
Copy link
Member

Choose a reason for hiding this comment

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

do we need to update this every year?

Copy link
Member Author

Choose a reason for hiding this comment

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

i fear yes .. its kind of ridiculous imho ..

Choose a reason for hiding this comment

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

Without warranty as far as I know the year is only informational. If it is missing it doesn't change anything. It's possible, that it depends on the country.

@dbu
Copy link
Member

dbu commented Aug 13, 2013

thanks lukas. did you look into the tool to automatically change all headers? would we then reset every header to the sample header in SessionInterface?

@lsmith77
Copy link
Member Author

i did not yet look into JMSCommandBundle for updating the license headers

@dbu
Copy link
Member

dbu commented Aug 28, 2013

ups, sorry overlooked this. i pushed the license change in a separate branch.

* Copyright 2008-2013 Karsten Dambekalns <karsten@typo3.org>
*
* For the full copyright and license information, please view the LICENSE.txt
* file that was distributed with this source code.
Copy link
Member

Choose a reason for hiding this comment

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

i did not add this but rather remove license information in the tests folder to be consistent.

@dbu dbu closed this Aug 28, 2013
@dbu dbu deleted the dual_license branch August 28, 2013 20:24
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