Merge pull request #1159 from lnis-uofu/xt_makefile

[script] serialize make all depedent targets to avoid ``make all`` bugs

.github/workflows/build.yml

@@ -127,15 +127,10 @@ jobs:
 
       - uses: hendrikmuhs/ccache-action@v1
 
-      - name: checkout submodules
-        shell: bash
-        run: |
-          make checkout
-
       - name: Build
         shell: bash
         run: |
-          make compile BUILD_TYPE=$BUILD_TYPE
+          make all BUILD_TYPE=$BUILD_TYPE
 
       # Check the cache size and see if it is over the limit
       - name: Check ccache size
@@ -216,15 +211,10 @@ jobs:
 
       - uses: hendrikmuhs/ccache-action@v1
 
-      - name: checkout submodules
-        shell: bash
-        run: |
-          make checkout
-
       - name: Build
         shell: bash
         run: |
-          make compile BUILD_TYPE=$BUILD_TYPE CMAKE_FLAGS="${{ matrix.config.cmake_flags }}"
+          make all BUILD_TYPE=$BUILD_TYPE CMAKE_FLAGS="${{ matrix.config.cmake_flags }}"
 
   ubuntu_support:
     needs: change_detect
@@ -262,15 +252,10 @@ jobs:
 
       - uses: hendrikmuhs/ccache-action@v1
 
-      - name: checkout submodules
-        shell: bash
-        run: |
-          make checkout
-
       - name: Build
         shell: bash
         run: |
-          make compile BUILD_TYPE=$BUILD_TYPE 
+          make all BUILD_TYPE=$BUILD_TYPE 
 
   debug_build:
     needs: change_detect
@@ -315,15 +300,10 @@ jobs:
 
       - uses: hendrikmuhs/ccache-action@v1
 
-      - name: checkout submodules
-        shell: bash
-        run: |
-          make checkout
-
       - name: Build
         shell: bash
         run: |
-          make compile BUILD_TYPE=${{ matrix.config.build_type }} -j ${{ matrix.config.cores }}
+          make all BUILD_TYPE=${{ matrix.config.build_type }} -j ${{ matrix.config.cores }}
 
       - name: Quick Test
         shell: bash
@@ -371,15 +351,10 @@ jobs:
 
       - uses: hendrikmuhs/ccache-action@v1
 
-      - name: checkout submodules
-        shell: bash
-        run: |
-          make checkout
-
       - name: Build
         shell: bash
         run: |
-          make compile BUILD_TYPE=${{ matrix.config.build_type }} -j ${{ matrix.config.cores }} CMAKE_FLAGS="-DOPENFPGA_ENABLE_STRICT_COMPILE=ON"
+          make all BUILD_TYPE=${{ matrix.config.build_type }} -j ${{ matrix.config.cores }} CMAKE_FLAGS="-DOPENFPGA_ENABLE_STRICT_COMPILE=ON"
 
       - name: Quick Test
         shell: bash

Makefile

@@ -66,7 +66,7 @@ prebuild:
 	echo "cd ${BUILD_DIR} && ${CMAKE_COMMAND} ${CMAKE_FLAGS} ${SOURCE_DIR}" && \
 	cd ${BUILD_DIR} && ${CMAKE_COMMAND} ${CMAKE_FLAGS} ${SOURCE_DIR}
 
-compile: prebuild
+compile: | prebuild
 # Compile the code base. By default, all the targets will be compiled
 # Following options are available
 # .. option:: CMAKE_GOALS
@@ -75,12 +75,13 @@ compile: prebuild
 	echo "Building target(s): ${CMAKE_GOALS}"
 	@+${MAKE} -C ${BUILD_DIR} ${CMAKE_GOALS}
 
-list_cmake_targets: prebuild
+list_cmake_targets: | prebuild
 # Show the targets available to be built, which can be specified through ``CMAKE_GOALS`` when compile
 	cd ${BUILD_DIR} && make help && cd -
 
-all: checkout compile
+all: checkout
 # A shortcut command to run checkout and compile in serial
+	@+${MAKE} compile
 
 format-cpp:
 # Format all the C/C++ files under this project, excluding submodules