Merge pull request #13 from Flow-IPC/pipeline_tweaks

Opportunistic: Doing TODO which said to avoid putting install-path in…

.github/workflows/main.yml

@@ -7,6 +7,7 @@ on:
   pull_request:
     branches:
       - main
+  # Make it possible to trigger workflow manually.
   workflow_dispatch:
 
 jobs:
@@ -114,12 +115,16 @@ jobs:
           - { id: gcc-13, name: gcc, version: 13, c-path: /usr/bin/gcc-13, cpp-path: /usr/bin/g++-13 }
           - { id: clang-13, name: clang, version: 13, c-path: /usr/bin/clang-13, cpp-path: /usr/bin/clang++-13 }
           - { id: clang-15, name: clang, version: 15, c-path: /usr/bin/clang-15, cpp-path: /usr/bin/clang++-15 }
-          - { id: clang-16, name: clang, version: 16, c-path: /usr/bin/clang-16, cpp-path: /usr/bin/clang++-16, install: True }
-          - { id: clang-17, name: clang, version: 17, c-path: /usr/bin/clang-17, cpp-path: /usr/bin/clang++-17, install: True }  
+          - { id: clang-16, name: clang, version: 16, c-path: /usr/bin/clang-16, cpp-path: /usr/bin/clang++-16,
+              install: True }
+          - { id: clang-17, name: clang, version: 17, c-path: /usr/bin/clang-17, cpp-path: /usr/bin/clang++-17,
+              install: True }
         build-type:
           - { id: debug, conan-name: Debug, conan-preset: debug,  }
           - { id: release, conan-name: Release, conan-preset: release }
-          - { id: relwithdebinfo-tsan, conan-name: RelWithDebInfo, conan-preset: relwithdebinfo, conan-cxxflags: [ "'-fsanitize=thread'" ], conan-cflags: [ "'-fsanitize=thread'" ], conan-ldflags: [ "'-fsanitize=thread'" ] }
+          - { id: relwithdebinfo-tsan, conan-name: RelWithDebInfo, conan-preset: relwithdebinfo,
+              conan-cxxflags: [ "'-fsanitize=thread'" ], conan-cflags: [ "'-fsanitize=thread'" ],
+              conan-ldflags: [ "'-fsanitize=thread'" ] }
           - { id: relwithdebinfo, conan-name: RelWithDebInfo, conan-preset: relwithdebinfo }
           - { id: minsizerel, conan-name: MinSizeRel, conan-preset: minsizerel }
         exclude:
@@ -206,12 +211,11 @@ jobs:
       uses: SimenB/github-actions-cpu-cores@v1
       id: cpu-cores
 
-    # TODO: It seems unusual to place install-prefix inside build-prefix.
     - name: Prepare Makefile using CMake
       run: |
         cmake \
           --preset ${{ matrix.build-type.conan-preset }} \
-          -DCMAKE_INSTALL_PREFIX=$GITHUB_WORKSPACE/build/${{ matrix.build-type.conan-name }}/install
+          -DCMAKE_INSTALL_PREFIX=$GITHUB_WORKSPACE/install/${{ matrix.build-type.conan-name }}
 
     - name: Build targets (library, demos/tests) with Makefile
       run: |
@@ -228,7 +232,7 @@ jobs:
 
     - name: Run demo/test [NetFlow echo]
       run: |
-          cd $GITHUB_WORKSPACE/build/${{ matrix.build-type.conan-name }}/install/bin/
+          cd $GITHUB_WORKSPACE/install/${{ matrix.build-type.conan-name }}/bin/
           ./net_flow_echo_srv.exec 8888 localhost & 
           sleep 1
           ./net_flow_echo_cli.exec 1 127.0.01 8888 "Hello world!"