[PR #317] [MERGED] Umk: Added parameter for providing UppHub location. #308

New issue

Closed

opened 2026-05-05 03:45:22 -06:00 by gitea-mirror · 0 comments

gitea-mirror commented

2026-05-05 03:45:22 -06:00

Owner

📋 Pull Request Information

Original PR: https://github.com/ultimatepp/ultimatepp/pull/317
Author: @klugier
Created: 11/1/2025
Status: ✅ Merged
Merged: 11/2/2025
Merged by: @mirek-fidler

Base: master ← Head: klugier/upp-hub-dir-as-umk-option

📝 Commits (4)

19f2a00 UMK: Added parameter for providing UppHub location.
c4cfd31 .remove fat
20f8063 .review
a52f726 .docs

📊 Changes

5 files changed (+74 additions, -37 deletions)

View changed files

📝 uppsrc/ide/app.tpp/umk_en-us.tpp (+16 -7)
📝 uppsrc/ide/app.tpp/umk_en-us.tppi (+10 -10)
📝 uppsrc/umk/UppHub.cpp (+27 -0)
📝 uppsrc/umk/umake.cpp (+20 -20)
📝 uppsrc/umk/umake.h (+1 -0)

📄 Description

This Pull Request introduces a new command-line argument, --upp_hub_dir ${dir}, to specify the directory containing UppHub packages. This enhancement ensures build independence and process repeatability by allowing users to explicitly control the package source. Here is the sample usage:

umk --hub-dir ./3p/hub ./,3p/uppsrc UppTerm 3p/umk/CLANG.bm -brvh +GUI{{build_flags}} build/UppTerm

Beyond the core changes, minor refactoring was completed. UppHub operations for UMK are now located in a dedicated namespace, and an ExitCodes enum was introduced to improve readability by replacing cryptic exit status numbers.

As part of the VSCode integration effort, this change is necessary to ensure UppHub packages reside in a consistent location within the workspace.

Skipping documentation updates for now. They will be included in a separate PR after this change is accepted.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/ultimatepp/ultimatepp/pull/317 **Author:** [@klugier](https://github.com/klugier) **Created:** 11/1/2025 **Status:** ✅ Merged **Merged:** 11/2/2025 **Merged by:** [@mirek-fidler](https://github.com/mirek-fidler) **Base:** `master` ← **Head:** `klugier/upp-hub-dir-as-umk-option` --- ### 📝 Commits (4) - [`19f2a00`](https://github.com/ultimatepp/ultimatepp/commit/19f2a00b93dc2155a357a01643f55c8f6a9cdf8c) UMK: Added parameter for providing UppHub location. - [`c4cfd31`](https://github.com/ultimatepp/ultimatepp/commit/c4cfd3195978966c2e6a493f7d574c776767d657) .remove fat - [`20f8063`](https://github.com/ultimatepp/ultimatepp/commit/20f8063200d840c257d0a8a1ffa849a3299ea1b4) .review - [`a52f726`](https://github.com/ultimatepp/ultimatepp/commit/a52f72631a1110377c8ce8ba5396518a5c8990ac) .docs ### 📊 Changes **5 files changed** (+74 additions, -37 deletions) <details> <summary>View changed files</summary> 📝 `uppsrc/ide/app.tpp/umk_en-us.tpp` (+16 -7) 📝 `uppsrc/ide/app.tpp/umk_en-us.tppi` (+10 -10) 📝 `uppsrc/umk/UppHub.cpp` (+27 -0) 📝 `uppsrc/umk/umake.cpp` (+20 -20) 📝 `uppsrc/umk/umake.h` (+1 -0) </details> ### 📄 Description This Pull Request introduces a new command-line argument, --upp_hub_dir ${dir}, to specify the directory containing UppHub packages. This enhancement ensures build independence and process repeatability by allowing users to explicitly control the package source. Here is the sample usage: ``` umk --hub-dir ./3p/hub ./,3p/uppsrc UppTerm 3p/umk/CLANG.bm -brvh +GUI{{build_flags}} build/UppTerm ``` Beyond the core changes, minor refactoring was completed. UppHub operations for UMK are now located in a dedicated namespace, and an ExitCodes enum was introduced to improve readability by replacing cryptic exit status numbers. As part of the VSCode integration effort, this change is necessary to ensure UppHub packages reside in a consistent location within the workspace. Skipping documentation updates for now. They will be included in a separate PR after this change is accepted. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

gitea-mirror

2026-05-05 03:45:22 -06:00

closed this issue
added the
pull-request
label

No labels

pull-request

No milestone

No project

No assignees

1 participant

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: github-starred/ultimatepp#308

No description provided.

Rows
Columns