Skip to content

Build cleanup#598

Merged
VolkerEnderlein merged 3 commits intocoin3d:masterfrom
tritao:pr/modern-gl/001-build-cleanup
Mar 30, 2026
Merged

Build cleanup#598
VolkerEnderlein merged 3 commits intocoin3d:masterfrom
tritao:pr/modern-gl/001-build-cleanup

Conversation

@tritao
Copy link
Copy Markdown
Contributor

@tritao tritao commented Jan 27, 2026

Summary

Remove legacy build/IDE artifacts and old build-system files to reduce maintenance burden and keep the tree aligned with the CMake-based build.

Changes

  • Remove large sets of legacy build/IDE files (old MSVC projects, Xcode project, autotools-era files, legacy CI configs).

Commits

  • 0cfbdb4644: Remove old unused files.
  • e37ed70140: Remove outdated build files.
  • 9f7f6d91ba: Update .gitignore

@VolkerEnderlein
Copy link
Copy Markdown
Member

Thanks for the PR. I fully agree to remove the entire build directory as it contains only obsoleted build files. The AppVeyor files are still in use to provide the full build matrix for providing the assets for releases and CI Builds.

@tritao
Copy link
Copy Markdown
Contributor Author

tritao commented Jan 27, 2026

Thanks for the PR. I fully agree to remove the entire build directory as it contains only obsoleted build files. The AppVeyor files are still in use to provide the full build matrix for providing the assets for releases and CI Builds.

Makes sense, I wasn't aware about that. I'll clean those changes up.

@VolkerEnderlein
Copy link
Copy Markdown
Member

@tritao Soft bump, any progress on the issue?

@tritao tritao force-pushed the pr/modern-gl/001-build-cleanup branch from 0cfbdb4 to dfe9651 Compare March 30, 2026 11:18
@tritao
Copy link
Copy Markdown
Contributor Author

tritao commented Mar 30, 2026

Updated, can you check this latest version?

@VolkerEnderlein
Copy link
Copy Markdown
Member

Thanks for the updated PR @tritao. I just had a look onto the PR, everything is fine. But to me it seems you did not removed the submodule build\general properly from the gitmodules. Can you check? Thanks.

@tritao tritao force-pushed the pr/modern-gl/001-build-cleanup branch from dfe9651 to 540d566 Compare March 30, 2026 17:32
@tritao
Copy link
Copy Markdown
Contributor Author

tritao commented Mar 30, 2026

Thanks for the updated PR @tritao. I just had a look onto the PR, everything is fine. But to me it seems you did not removed the submodule build\general properly from the gitmodules. Can you check? Thanks.

Updated again with that removed.

@VolkerEnderlein VolkerEnderlein merged commit 541658b into coin3d:master Mar 30, 2026
5 checks passed
@VolkerEnderlein
Copy link
Copy Markdown
Member

Thanks again @tritao for your valuable work on the Coin source. It is highly appreciated.

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