linux-trace-devel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] kernelshark: generate header file in the build directory
       [not found] <a5350178-93c4-4aad-a503-a622704fe764.ref@yahoo.com>
@ 2025-10-24 19:58 ` Mircea Cirjaliu
  2025-10-27 19:47   ` Yordan Karadzhov
  0 siblings, 1 reply; 2+ messages in thread
From: Mircea Cirjaliu @ 2025-10-24 19:58 UTC (permalink / raw)
  To: linux-trace-devel; +Cc: y.karadz

Mixing versioned and generated files will bloat git listings.
Put KsCmakeDef.hpp together with the rest of the generated files.

Signed-off-by: Mircea Cirjaliu <mircea.cirjaliu@yahoo.com>
---
 CMakeLists.txt       | 4 ++--
 build/cmake_clean.sh | 2 +-
 src/CMakeLists.txt   | 2 +-
 3 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/CMakeLists.txt b/CMakeLists.txt
index 54ff97c..84215dd 100644
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -129,7 +129,7 @@ else (CMAKE_BUILD_TYPE MATCHES Package)
 endif (CMAKE_BUILD_TYPE MATCHES Package)
 
 include_directories(${KS_DIR}/src/
-                    ${KS_DIR}/build/src/)
+                    ${KS_DIR}/build/include/)
 
 message("")
 message(STATUS "C flags      : " ${CMAKE_C_FLAGS})
@@ -163,7 +163,7 @@ if (_DOXYGEN_DOC AND DOXYGEN_FOUND)
 endif ()
 
 configure_file(${KS_DIR}/build/deff.h.cmake
-               ${KS_DIR}/src/KsCmakeDef.hpp)
+               ${KS_DIR}/build/include/KsCmakeDef.hpp)
 
 configure_file(${KS_DIR}/build/ks.desktop.cmake
                ${KS_DIR}/${KS_APP_NAME}.desktop)
diff --git a/build/cmake_clean.sh b/build/cmake_clean.sh
index 67afd6c..2c05d3e 100755
--- a/build/cmake_clean.sh
+++ b/build/cmake_clean.sh
@@ -6,6 +6,7 @@ rm -f CTestTestfile.cmake
 rm -f DartConfiguration.tcl
 rm -rf CMakeFiles/
 rm -rf src/
+rm -rf include/
 rm -rf examples/
 rm -rf tests/
 rm -rf Testing/
@@ -14,7 +15,6 @@ rm -f ../lib/*
 rm ../kernelshark.desktop
 rm ../libkshark.pc
 rm ../org.freedesktop.kshark-record.policy
-rm -f ../src/KsCmakeDef.hpp
 rm -f CMakeDoxyfile.in
 rm -f CMakeDoxygenDefaults.cmake
 rm -rf .qt/
diff --git a/src/CMakeLists.txt b/src/CMakeLists.txt
index 0643365..da31f8d 100644
--- a/src/CMakeLists.txt
+++ b/src/CMakeLists.txt
@@ -71,7 +71,7 @@ if (OPENGL_FOUND)
                   "${KS_DIR}/src/libkshark-plot.h"
                   "${KS_DIR}/src/KsPlotTools.hpp"
                   "${KS_DIR}/src/KsPlugins.hpp"
-                  "${KS_DIR}/src/KsCmakeDef.hpp"
+                  "${KS_DIR}/build/include/KsCmakeDef.hpp"
             DESTINATION ${KS_INCLUDS_DESTINATION}
                 COMPONENT libkshark-devel)
 
-- 
2.43.0

^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH] kernelshark: generate header file in the build directory
  2025-10-24 19:58 ` [PATCH] kernelshark: generate header file in the build directory Mircea Cirjaliu
@ 2025-10-27 19:47   ` Yordan Karadzhov
  0 siblings, 0 replies; 2+ messages in thread
From: Yordan Karadzhov @ 2025-10-27 19:47 UTC (permalink / raw)
  To: Mircea Cirjaliu, linux-trace-devel

The patch is applied. Thanks!

Yordan

On 10/24/25 22:58, Mircea Cirjaliu wrote:
> Mixing versioned and generated files will bloat git listings.
> Put KsCmakeDef.hpp together with the rest of the generated files.
> 
> Signed-off-by: Mircea Cirjaliu <mircea.cirjaliu@yahoo.com>
> ---
>   CMakeLists.txt       | 4 ++--
>   build/cmake_clean.sh | 2 +-
>   src/CMakeLists.txt   | 2 +-
>   3 files changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/CMakeLists.txt b/CMakeLists.txt
> index 54ff97c..84215dd 100644
> --- a/CMakeLists.txt
> +++ b/CMakeLists.txt
> @@ -129,7 +129,7 @@ else (CMAKE_BUILD_TYPE MATCHES Package)
>   endif (CMAKE_BUILD_TYPE MATCHES Package)
>   
>   include_directories(${KS_DIR}/src/
> -                    ${KS_DIR}/build/src/)
> +                    ${KS_DIR}/build/include/)
>   
>   message("")
>   message(STATUS "C flags      : " ${CMAKE_C_FLAGS})
> @@ -163,7 +163,7 @@ if (_DOXYGEN_DOC AND DOXYGEN_FOUND)
>   endif ()
>   
>   configure_file(${KS_DIR}/build/deff.h.cmake
> -               ${KS_DIR}/src/KsCmakeDef.hpp)
> +               ${KS_DIR}/build/include/KsCmakeDef.hpp)
>   
>   configure_file(${KS_DIR}/build/ks.desktop.cmake
>                  ${KS_DIR}/${KS_APP_NAME}.desktop)
> diff --git a/build/cmake_clean.sh b/build/cmake_clean.sh
> index 67afd6c..2c05d3e 100755
> --- a/build/cmake_clean.sh
> +++ b/build/cmake_clean.sh
> @@ -6,6 +6,7 @@ rm -f CTestTestfile.cmake
>   rm -f DartConfiguration.tcl
>   rm -rf CMakeFiles/
>   rm -rf src/
> +rm -rf include/
>   rm -rf examples/
>   rm -rf tests/
>   rm -rf Testing/
> @@ -14,7 +15,6 @@ rm -f ../lib/*
>   rm ../kernelshark.desktop
>   rm ../libkshark.pc
>   rm ../org.freedesktop.kshark-record.policy
> -rm -f ../src/KsCmakeDef.hpp
>   rm -f CMakeDoxyfile.in
>   rm -f CMakeDoxygenDefaults.cmake
>   rm -rf .qt/
> diff --git a/src/CMakeLists.txt b/src/CMakeLists.txt
> index 0643365..da31f8d 100644
> --- a/src/CMakeLists.txt
> +++ b/src/CMakeLists.txt
> @@ -71,7 +71,7 @@ if (OPENGL_FOUND)
>                     "${KS_DIR}/src/libkshark-plot.h"
>                     "${KS_DIR}/src/KsPlotTools.hpp"
>                     "${KS_DIR}/src/KsPlugins.hpp"
> -                  "${KS_DIR}/src/KsCmakeDef.hpp"
> +                  "${KS_DIR}/build/include/KsCmakeDef.hpp"
>               DESTINATION ${KS_INCLUDS_DESTINATION}
>                   COMPONENT libkshark-devel)
>   

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2025-10-27 19:47 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
     [not found] <a5350178-93c4-4aad-a503-a622704fe764.ref@yahoo.com>
2025-10-24 19:58 ` [PATCH] kernelshark: generate header file in the build directory Mircea Cirjaliu
2025-10-27 19:47   ` Yordan Karadzhov

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).