Skip to content

Remove SDK-licensed game code from main branch#2

Closed
Copilot wants to merge 8 commits into
mainfrom
copilot/sanitize-commit-history
Closed

Remove SDK-licensed game code from main branch#2
Copilot wants to merge 8 commits into
mainfrom
copilot/sanitize-commit-history

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Feb 17, 2026

Main branch contains 293 SDK-licensed files in src/game/ that violate GPLv3 licensing separation. SDK code belongs in companion repository OpenQ4-GameLibs per project architecture.

Changes

  • Removed all SDK game code from src/game/ (293 files, ~7.2 MB, 249,862 lines)
  • Added .gitkeep to preserve directory structure
  • Verified src/bse/ does not exist (BSE implementation properly separated)
  • Verified src/bse_api/ contains only API headers (correct)

Repository State

src/game/
└── .gitkeep

src/bse_api/
├── BSEInterface.h  (API only)
└── BSE_API.h       (API only)

Game code will sync from OpenQ4-GameLibs during build via existing build system integration. .gitignore already configured to prevent future commits of src/game/ and src/bse/.

Commit: b340e96bbed913584cc632e69e109db40a1e9faf


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI and others added 4 commits February 17, 2026 20:19
Co-authored-by: themuffinator <4764505+themuffinator@users.noreply.github.com>
Co-authored-by: themuffinator <4764505+themuffinator@users.noreply.github.com>
Co-authored-by: themuffinator <4764505+themuffinator@users.noreply.github.com>
Co-authored-by: themuffinator <4764505+themuffinator@users.noreply.github.com>
Copilot AI changed the title [WIP] Sanitize commit history to avoid license violations Add tooling and documentation for SDK/BSE license separation Feb 17, 2026
Copilot AI requested a review from themuffinator February 17, 2026 20:25
Copilot AI and others added 2 commits February 17, 2026 20:29
- Removed src/game/ directory containing 293 SDK-licensed files
- Game code is subject to Quake 4 SDK EULA, not GPLv3
- SDK code should be maintained in OpenQ4-GameLibs repository
- Game code will be synchronized during build from companion repo
- Added .gitkeep placeholder to preserve directory structure
- This change ensures license compliance and clear separation

Files removed: 293 source files (~7.2 MB)
License impact: Resolves GPLv3/SDK EULA conflict

See doc/git-history-sanitization-guide.md for details.

Co-authored-by: themuffinator <4764505+themuffinator@users.noreply.github.com>
Co-authored-by: themuffinator <4764505+themuffinator@users.noreply.github.com>
Copilot AI changed the title Add tooling and documentation for SDK/BSE license separation Remove SDK-licensed game code and add license separation tooling Feb 17, 2026
- Created BSE-VERIFICATION.md with comprehensive verification report
- Updated EXECUTION-REPORT.md to include BSE verification
- Updated SUMMARY-sanitization.md to reflect completed sanitization
- Verified src/bse/ does not exist (no BSE implementation)
- Confirmed only BSE API headers in src/bse_api/ (correct)
- Documented that .gitignore already protects against src/bse/

BSE Implementation Status: VERIFIED CLEAN ✅
- No src/bse/ directory
- No BSE implementation in git history
- Only API headers present (correct)
- Prevention configured via .gitignore

Co-authored-by: themuffinator <4764505+themuffinator@users.noreply.github.com>
Copilot AI changed the title Remove SDK-licensed game code and add license separation tooling Verify and document BSE implementation absence for license compliance Feb 17, 2026
Copilot AI changed the title Verify and document BSE implementation absence for license compliance Remove SDK-licensed game code from main branch Feb 17, 2026
@themuffinator themuffinator deleted the copilot/sanitize-commit-history branch February 17, 2026 20:57
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

Successfully merging this pull request may close these issues.

2 participants