Skip to content

Commit

Permalink
Clarify Artistic-2.0 in license
Browse files Browse the repository at this point in the history 
Closes #243
  • Loading branch information
@0xpr03
0xpr03 authored Mar 6, 2021
1 parent ed83b54 commit a0f4855
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion Cargo.toml
View file
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ documentation = "https://docs.rs/notify"
homepage = "https://github.com/notify-rs/notify"
repository = "https://github.com/notify-rs/notify.git"
readme = "README.md"
license = "CC0-1.0"
license = "CC0-1.0 OR Artistic-2.0"
keywords = ["events", "filesystem", "notify", "watch"]
categories = ["filesystem"]
authors = [
Expand Down

8 comments on commit a0f4855

@JohnTitor
Copy link
Member

Choose a reason for hiding this comment

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

@0xpr03 I think it should be CC0-1.0 AND Artistic-2.0.

@0xpr03
Copy link
Member Author

@0xpr03 0xpr03 commented on a0f4855 Mar 6, 2021

Choose a reason for hiding this comment

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

We specifically make it a dual license ?

@0xpr03
Copy link
Member Author

@0xpr03 0xpr03 commented on a0f4855 Mar 6, 2021

Choose a reason for hiding this comment

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

So I used the rand version for specification

@JohnTitor
Copy link
Member

@JohnTitor JohnTitor commented on a0f4855 Mar 6, 2021 •
edited
Loading

Choose a reason for hiding this comment

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

https://github.com/notify-rs/notify#license says we license the code under both.

@0xpr03
Copy link
Member Author

@0xpr03 0xpr03 commented on a0f4855 Mar 6, 2021

Choose a reason for hiding this comment

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

Yeah but https://github.com/rust-random/rand#license does it the same way, I thought thats the normal form, you license it as both, so you can choose "this or that".

@0xpr03
Copy link
Member Author

@0xpr03 0xpr03 commented on a0f4855 Mar 6, 2021

Choose a reason for hiding this comment

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

Anyway if you think that's not right please change it accordingly, I'll create a PR next time.

@JohnTitor
Copy link
Member

Choose a reason for hiding this comment

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

Hm, it's unclear to me that whether we allow users to select licenses at their option. Since we want to drop CC-0 in the future, it's more better to use WITH, according to https://doc.rust-lang.org/cargo/reference/manifest.html#the-license-and-license-file-fields?

@0xpr03
Copy link
Member Author

@0xpr03 0xpr03 commented on a0f4855 Mar 8, 2021

Choose a reason for hiding this comment

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

The thing is that we dual-license every new code. So at least at some point everything should be dual licensed..

Please sign in to comment.