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

[Misc]: Remove contrib folder in repo #3097

Open
tulpenkiste opened this issue Nov 13, 2024 · 1 comment
Open

[Misc]: Remove contrib folder in repo #3097

tulpenkiste opened this issue Nov 13, 2024 · 1 comment

Comments

@tulpenkiste
Copy link
Contributor

tulpenkiste commented Nov 13, 2024

The contrib folder contains 3 seperate patch files, ranging from 8 years old to 16 years old. I highly doubt these diffs even work on modern revisions, nor do they serve a purpose.

  • supertux-nogl - replacable via cmake (already done iirc)
  • supertux-coop - already implemented
  • schroedingers-boxes-pr-390 -
@tulpenkiste tulpenkiste changed the title Misc: Remove contrib folder in repo [Misc]: Remove contrib folder in repo Nov 13, 2024
@swagtoy
Copy link

swagtoy commented Nov 13, 2024

I support this when i was looking at them. However, I think we should do patches instead of maintaining our own Git submodules since we only do very minor things (and some of them unneeded nowadays)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants