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

change license to Apache 2.0 #624

Merged
merged 1 commit into from Aug 13, 2019
Merged

change license to Apache 2.0 #624

merged 1 commit into from Aug 13, 2019

Conversation

paweljakubas
Copy link
Contributor

Issue Number

Overview

Change license from MIT to Apache 2.0.

  • I have updated LICENSE file and changed from MIT to Apache 2.0 in each file.

Comments

There is something to be confirmed here though - it concerns the following files :

lib/bech32/src/Codec/Binary/Bech32/Internal.hs
lib/bech32/bech32.cabal
lib/bech32/src/Codec/Binary/Bech32.hs

Can the license be changed if copyright is :

copyright:     2017 Marko Bencun, 2019 IOHK                                                                                                                                                
license:       Apache-2.0

?

@paweljakubas paweljakubas self-assigned this Aug 13, 2019
@KtorZ
Copy link
Member

KtorZ commented Aug 13, 2019

Yes. The change of license only take effects at the moment you change it. So in practice, the license in 2017 & 2018 was MIT, and starting from 2019 it's Apache 2.0.

LICENSE Outdated
same "printed page" as the copyright notice for easier
identification within third-party archives.

Copyright [yyyy] [name of copyright owner]
Copy link
Member

Choose a reason for hiding this comment

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

@paweljakubas Would be good to put dates and IOHK here 😬

nix patch

improve copyright in LICENSE file
@KtorZ KtorZ merged commit 67907ab into master Aug 13, 2019
@KtorZ KtorZ deleted the paweljakubas/change-license branch August 13, 2019 16:56
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