xt-sys/xtchain
xt-sys/xtchain
3
2
Fork 2
Code Issues 1 Pull Requests Actions Projects 1 Releases 21 Activity

Cleanup the toolchain #8

New Issue
Closed
opened 2025-03-19 15:01:37 +01:00 by harraiken · 2 comments
harraiken commented 2025-03-19 15:01:37 +01:00
Owner
Copy Link

XTChain seems to contain unnecessary tools, like MinGW integration. We do not use it to build Win32 applications, and ExectOS should have its own runtime library. Therefore, MinGW should not be needed. There may be more such unnecessary tools. The goal of the task is to slim down the toolchain by identifying all the really needed tools and removing those that are not used.

XTChain seems to contain unnecessary tools, like MinGW integration. We do not use it to build Win32 applications, and ExectOS should have its own runtime library. Therefore, MinGW should not be needed. There may be more such unnecessary tools. The goal of the task is to slim down the toolchain by identifying all the really needed tools and removing those that are not used.
harraiken added this to the XT Toolchain Development Board project 2025-03-19 15:01:37 +01:00
harraiken added the
IDEA
 label 2025-03-19 15:02:16 +01:00
perikiyoxd commented 2025-03-20 18:27:58 +01:00
Copy Link

XTChain's CMake

The XTChain-built version of CMake introduces a custom watermark by modifying the CMake_VERSION variable. This change is introduced in the patches\cmake\v3.29.3\001-add-xtc-version-suffix.patch patch.

The only functional difference in XTChain’s CMake build is the version suffix, which is not a valid reason to maintain a custom-built version.

Recommended Steps

  1. Deprecate XTChain-built CMake in favor of system-installed CMake.
  2. Remove watermark patches and update build instructions to reflect the new dependency installation strategy.
  3. Ensure CI/CD compatibility by validating that system package managers can reliably provide CMake for all supported platforms.
# XTChain's CMake The XTChain-built version of CMake introduces a custom watermark by modifying the `CMake_VERSION` variable. This change is introduced in the `patches\cmake\v3.29.3\001-add-xtc-version-suffix.patch` patch. The only functional difference in XTChain’s CMake build is the version suffix, which is not a valid reason to maintain a custom-built version. ### **Recommended Steps** 1. **Deprecate XTChain-built CMake** in favor of system-installed CMake. 2. **Remove watermark patches** and update build instructions to reflect the new dependency installation strategy. 3. **Ensure CI/CD compatibility** by validating that system package managers can reliably provide CMake for all supported platforms.
harraiken commented 2025-07-09 23:11:07 +02:00
Author
Owner
Copy Link

#10

https://git.codingworkshop.eu.org/xt-sys/xtchain/pulls/10
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
API CHANGE
Requires or requests API change
BUG
Something is not working
DUPLICATE
This issue or pull request already exists
ENHANCEMENT
New feature request or implementation
HELP WANTED
Needs some help
IDEA
Ideas and proposals
INVALID
Something is wrong
MODDING
Visual design modifications
QUESTION
More information is needed
REFACTORING
Code refactoring
SYNC
Synchronization with other project
TRANSLATION
Translation and localization issue or change
UPSTREAM
The patch needs to be sent upstream
WONTFIX
This won't be fixed
No Label
IDEA
Milestone
No items
No Milestone
Projects
Clear projects
No project XT Toolchain Development Board
Assignees
Clear assignees
belliash harraiken
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: xt-sys/xtchain#8
No description provided.
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?