haskell / containers

Assorted concrete container types
https://hackage.haskell.org/package/containers
315 stars 178 forks source link

`haddock-autolink` submodule: Licensing, inclusion #891

Open sidkshatriya opened 1 year ago

sidkshatriya commented 1 year ago

While cloning the main ghc repository (using git clone https://gitlab.haskell.org/ghc/ghc.git --recurse-submodules) I noticed that the containers repository was being cloned. That's OK because the containers library is a submodule of the ghc repo.

However, I noticed that the haddock-autolink submodule of containers is also being cloned. This is because haddock-autolink is a submodule of the containers repo.

[submodule "containers/docs/_extensions/haddock-autolink"]
    path = containers/docs/_extensions/haddock-autolink
    url = https://github.com/m-renaud/haddock-autolink

( https://github.com/haskell/containers/blob/b216597c83755131aac0f335744e2f803eaa9399/.gitmodules )

Here is what I noticed:

Request maintainers of containers suggest if any of these two issues need to be addressed.

cc: @m-renaud

treeowl commented 1 year ago

@m-renaud is definitely more qualified to answer this then I am, or probably anyone else. I know literally nothing about that submodule.

m-renaud commented 1 year ago

For background: haddock-autolink is only used in the generation of the Markdown docs which live at https://haskell-containers.readthedocs.io/en/latest/ (but is also used by other doc sets outside of container).

To your questions: the code is very simple, so if desired could be inlined into containers, or hoisted to the haskell GitHub organization.

treeowl commented 1 year ago

@m-renaud It sounds like the most urgent matter is for you to add a suitable license for haddock-autolink. Would you be able to do that soon? Beyond that, I'm going to continue to hide under my rock and let you maintain that side of things.

m-renaud commented 1 year ago

@treeowl sorry for the delay! PTAL at https://github.com/m-renaud/haddock-autolink/pull/2 (BSD 3-Clause which to my understanding is compatible).

treeowl commented 1 year ago

@m-renaud What does PTAL mean? Can you submit a commit to get the license in the submodule?

sidkshatriya commented 1 year ago

PTAL means "Please take another look" IIUC (if I understand correctly).

BTW the PR is not merged yet.