Opened at 2017-03-18T16:22:17Z
Last modified at 2022-10-21T22:03:18Z
#1093 new planned
re/generate command ID header
| Reported by: | Ichthyostega | Owned by: | Ichthyostega |
|---|---|---|---|
| Priority: | nice | Milestone: | 1alpha |
| Component: | lumieraSteam | Keywords: | integration buildsystem scons command |
| Sub Tickets: | #215, #1088 | Parent Tickets: | #1087 |
Description (last modified by )
It is clear by now that commands will be defined by C++ scripts in translation units located in src/steam/cmd/*-cmd.cpp
These will be comprised of a sequence of COMMAND_DEFINITION entries, which actually define CommandSetup instances.
Obviously, to be usable by the UI, we need a series of matching declarations in the header steam/cmd.hpp
For the current stage of the project it is feasible just to require adding the necessary definition lines manually, yet on the long run we need an automated solution here: The build-system shall scan the relevant definition headers and re-write a marked section in the declaration header. In this case, it is OK that this script manipulates the source, and the results should even be checked into Git. Because, in the usual development cycle, you'd figure out that you'll need yet another command; you'd add the command script and re-build, to check the signature is properly defined. After that build run, you'd also have the declaration entry automatically inserted into cmd.hpp, which you'd refer to then in the UI code you are about to extend. Finally, when done, you'd check all changes together into Git.
Corollary: the script to write for this task needs to be idempotent
Change history (4)
comment:1 by , at 2017-03-31T16:39:16Z
| blockedby: | 215 → 215, 1088 |
|---|---|
| blocking: | 1088 |
comment:2 by , at 2017-03-31T16:41:16Z
| blocking: | → 1087 |
|---|
comment:3 by , at 2022-10-21T22:03:18Z
| Description: | modified (diff) |
|---|
comment:4 by , at 2025-12-25T00:00:00Z
| blockedby: | 215, 1088 |
|---|---|
| blocking: | 1087 |
| Parent Tickets: | → 1087 |
| Sub Tickets: | → 215, 1088 |
Migration MasterTickets ⟼ Subtickets-plugin

postponed for now