8000 PocketFFT missing LICENSE.md · Issue #14552 · numpy/numpy · GitHub
[go: up one dir, main page]

Skip to content

PocketFFT missing LICENSE.md #14552

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

Closed
sandrotosi opened this issue Sep 20, 2019 · 4 comments · Fixed by #14557
Closed

PocketFFT missing LICENSE.md #14552

sandrotosi opened this issue Sep 20, 2019 · 4 comments · Fixed by #14557
Assignees

Comments

@sandrotosi
Copy link
Contributor

Hello,
numpy/fft/README.md says

License

3-clause BSD (see LICENSE.md)

but no LICENSE.md file is present in the repo (and thus the release tarball too).

Could you please include that file too? thanks!

@rgommers
Copy link
Member

Thanks @sandrotosi.

Since none of this code is in an external package anymore, it may make more sense to remove the mention of a license from that README rather than add a separate license file. It can simply be covered by the NumPy BSD license. @mreineck would that be fine with you? If not, can you point me to the LICENSE.md file that's in the README?

@rgommers rgommers added this to the 1.17.3 release milestone Sep 20, 2019
@rgommers rgommers self-assigned this Sep 20, 2019
@sandrotosi
Copy link
Contributor Author

it still lives here: https://gitlab.mpcdf.mpg.de/mtr/pocketfft (included the LICENSE.md file)

@mreineck
Copy link
Contributor

I'm fine with either change! Sorry, can't produce a patch right now, will be back at a decent computer in a few days.

@rgommers
Copy link
Member

thanks @mreineck. I'll open a PR now.

rgommers added a commit to rgommers/numpy that referenced this issue Sep 20, 2019
Closes numpygh-14552

The Pocketfft author (mreineck) agreed on numpygh-14552 that Pocketfft can
simply fall under the NumPy license, no separate license file needed.

[ci skip]
rgommers added a commit to rgommers/numpy that referenced this issue Sep 21, 2019
Closes numpygh-14552

The Pocketfft author (mreineck) agreed on numpygh-14552 that Pocketfft can
simply fall under the NumPy license, no separate license file needed.

[ci skip]

(cherry picked from commit 43bc819)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants
0