Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[idea] Align build output for build step and object name #2564

Closed
Challanger524 opened this issue Jan 29, 2025 · 2 comments
Closed

[idea] Align build output for build step and object name #2564

Challanger524 opened this issue Jan 29, 2025 · 2 comments

Comments

@Challanger524
Copy link

Examples:

original:

[build] [8/19] Building ASM object dep/boost/libs/context/CMakeFiles/boost_context.dir/src/asm/jump_x86_64_ms_pe_gas.asm.obj
[build] [9/19] Building CXX object dep/boost/libs/container/CMakeFiles/boost_container.dir/src/unsynchronized_pool_resource.cpp.obj
[build] [10/19] Building ASM object dep/boost/libs/context/CMakeFiles/boost_context.dir/src/asm/ontop_x86_64_ms_pe_gas.asm.obj
[build] [11/19] Building C object dep/boost/libs/container/CMakeFiles/boost_container.dir/src/alloc_lib.c.obj
[build] [12/19] Building CXX object dep/boost/libs/context/CMakeFiles/boost_context.dir/src/fcontext.cpp.obj

aligned: build step, object name

[build] [ 8/19] Building ASM object dep/boost/libs/context/CMakeFiles/boost_context.dir/src/asm/jump_x86_64_ms_pe_gas.asm.obj
[build] [ 9/19] Building CXX object dep/boost/libs/container/CMakeFiles/boost_container.dir/src/unsynchronized_pool_resource.cpp.obj
[build] [10/19] Building ASM object dep/boost/libs/context/CMakeFiles/boost_context.dir/src/asm/ontop_x86_64_ms_pe_gas.asm.obj
[build] [11/19] Building C   object dep/boost/libs/container/CMakeFiles/boost_container.dir/src/alloc_lib.c.obj
[build] [12/19] Building CXX object dep/boost/libs/context/CMakeFiles/boost_context.dir/src/fcontext.cpp.obj

These params should be known from the beginning, so what if some kind of std::setw()+precalculation was used for this?
Vote with 👍 👎 and/or point me to the source code responsible for the message printing, input files storing and build steps calculation.

overkill: will break filepaths on copying, but looks even more pretty

[build] [ 8/19] Building ASM object dep/boost/libs/context  /CMakeFiles/boost_context  .dir/src/asm/jump_x86_64_ms_pe_gas.asm.obj
[build] [ 9/19] Building CXX object dep/boost/libs/container/CMakeFiles/boost_container.dir/src/unsynchronized_pool_resource.cpp.obj
[build] [10/19] Building ASM object dep/boost/libs/context  /CMakeFiles/boost_context  .dir/src/asm/ontop_x86_64_ms_pe_gas.asm.obj
[build] [11/19] Building C   object dep/boost/libs/container/CMakeFiles/boost_container.dir/src/alloc_lib.c.obj
[build] [12/19] Building CXX object dep/boost/libs/context  /CMakeFiles/boost_context  .dir/src/fcontext.cpp.obj
@jhasse
Copy link
Collaborator

jhasse commented Jan 29, 2025

The status message is out of control of Ninja - it's getting set by CMake.

@jhasse jhasse closed this as completed Jan 29, 2025
@Challanger524
Copy link
Author

Challanger524 commented Jan 29, 2025

The status message is out of control of Ninja - it's getting set by CMake.

Oh, good to know. Simpler to bury this idea after seeing 5K open issues on Kitware GitLab

Opened, if somebody need: https://gitlab.kitware.com/cmake/cmake/-/issues/26650

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

No branches or pull requests

2 participants