Skip to content
This repository has been archived by the owner on Nov 5, 2018. It is now read-only.

What's the license for examples/csp.rs? #104

Closed
erickt opened this issue Oct 25, 2018 · 6 comments · Fixed by #105
Closed

What's the license for examples/csp.rs? #104

erickt opened this issue Oct 25, 2018 · 6 comments · Fixed by #105
Labels

Comments

@erickt
Copy link

erickt commented Oct 25, 2018

Hello! We saw that the code in examples/csp.rs is derived from https://www.nada.kth.se/~snilsson/concurrency/#Match, but there doesn't appear to be a license associated with this example. Could one be added, or this file be removed, or at least exclude this file from the crate?

@ghost ghost added the question label Oct 26, 2018
@ghost
Copy link

ghost commented Oct 26, 2018

The CSP example was published under Creative Commons Attribution 3.0 Unported License. I'll add attribution info.

Can I ask you for an advice on two more licensing issues?

We have a port of Go tests in the tests directory, which should include a license notice too. Would adding a Go license to that file be enough? Alternatively, I could move those tests into a different repository so that crates depending on crossbeam-channel don't include that file at all.

We similarly have a copy of std::sync::mpsc tests, where it might be a good idea to also add a Rust license. Do you think that is necessary?

@erickt
Copy link
Author

erickt commented Oct 26, 2018

Either way of these ways would work for us, but we can't give advice on which is the best approach you should take.

@ghost ghost closed this as completed in #105 Oct 30, 2018
@erickt
Copy link
Author

erickt commented Oct 30, 2018

Thanks for this @stjepang! When do you expect to do another release? We're using cargo-vendor to incorporate crates into our repo, so we can't update our crossbeam-channel until it lands on crates.io.

@ghost
Copy link

ghost commented Oct 30, 2018

@erickt I'm preparing version 0.3.0 here: #106

I expect to release it this week, but if you need it sooner than that, I can bump the version now.

@erickt
Copy link
Author

erickt commented Oct 30, 2018

This week is fine, thanks @stjepang!

@ghost
Copy link

ghost commented Nov 4, 2018

Version 0.3.0 has been released! You can see some comments on it here.

This issue was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Development

Successfully merging a pull request may close this issue.

1 participant