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

Sphinxify licensing #20

Closed
jasongrout opened this issue Nov 11, 2021 · 4 comments · Fixed by #43
Closed

Sphinxify licensing #20

jasongrout opened this issue Nov 11, 2021 · 4 comments · Fixed by #43
Assignees
Milestone

Comments

@jasongrout
Copy link

jasongrout commented Nov 11, 2021

I was tracking down the history of the sphinxify licensing to clarify how this sphinxify module is BSD while the core Sage sphinxify is currently GPLv2+. Here is what I found:

I don't think there is anything to do and this ticket can be closed immediately, but I just wanted to document the history to prevent confusion.

Personal side note: If I recall correctly, I remember having a conversation about the BSD license on the original sphinxify module at SageDays 19 with Tim and a few others, where we decided it would be fine to have a BSD-licensed file in the Sage Notebook codebase, but that we'd just make sure that file explicitly stated it had the BSD license

@CAM-Gerlach
Copy link
Member

Thanks for the very detailed breakdown, @jasongrout !

@ccordoba12 Might be a good idea to add a brief note to the License section of the Readme summarizing this as it directly relates to docrepr and linking here for more information, if @jasongrout wants to submit a PR...but I leave that up to you.

@SylvainCorlay
Copy link
Member

Should we close this then?

@CAM-Gerlach
Copy link
Member

I was going to add a mention and a link to this in the Readme in #37

@ccordoba12
Copy link
Member

Sorry for chiming in so late here, but I'd like to thank @jasongrout for clarifying the licensing situation of this project.

I'd also like to add a bit more context to this part

In 2015, @ccordoba12 started what would become the docrepr project and seems to have forked this BSD-licensed sphinxify module: 37f2557#diff-47a8d80ad1d3db4de64b36827a03d3928ba6982bebcc4ea229adce0a3d1c88ad

Actually, docrepr started as an effort to create an independent project out of the code that powers Spyder's Help pane, so others can be benefit from it. Furthermore, I was the one that created that functionality, which also happens to be my first contribution to Spyder back in 2010.

I found about sphinxify at that time when looking for simple libraries that allowed to make webpages out of docstrings on the fly. As @jasongrout mentioned, the license back then was BSD. Since afterwards I didn't merge back changes from Sage's sphinxify, there shouldn't be an issue with that.

This is also why the first version of sphinxify here is a modified one of the original sphinxify in Sage. And there's another version that still lives in Spyder:

https://github.com/spyder-ide/spyder/blob/master/spyder/plugins/help/utils/sphinxify.py

One day we hope to integrate this project with Spyder as well.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment