Merge branch 'build-tiny-fixes'

* build-tiny-fixes:
  Fix make without arguments
  Update .PHONY targets
  Avoid copy-paste for PYO3_CONFIG_FILE
  Move steps from build into dependencies

.github/workflows/build.yml

@@ -37,16 +37,16 @@ jobs:
  # nfpm
  curl -sS -Lo /tmp/nfpm.deb "https://github.com/goreleaser/nfpm/releases/download/v2.25.0/nfpm_amd64.deb"
  sudo dpkg -i /tmp/nfpm.deb
-
- - name: Build
- run: |
- set -x
  # for building cityhash for clickhouse-rs
  sudo apt-get install -y musl-tools
  # gcc cannot do cross compile, and there is no musl-g++ in musl-tools
  sudo ln -srf /usr/bin/clang /usr/bin/musl-g++
  # musl for static binaries
  rustup target add x86_64-unknown-linux-musl
+
+ - name: Build
+ run: |
+ set -x
  make packages target=x86_64-unknown-linux-musl
  for postfix in .x86_64.rpm -x86_64.pkg.tar.zst _amd64.deb; do
  cp chdig*$postfix chdig-latest$postfix

Makefile

@@ -34,12 +34,17 @@ endif
 export PYO3_CONFIG_FILE = $(PWD)/contrib/flameshow/build/pyo3-build-config-file-$(target).txt
 $(info PYO3_CONFIG_FILE = $(PYO3_CONFIG_FILE))
 
-.PHONY: build chdig install deb rpm archlinux packages
+.PHONY: build build_completion deploy-binary chdig install run \
+ deb rpm archlinux packages
 
-contrib/flameshow/build/pyo3-build-config-file-$(target).txt:
+# This should be the first target (since ".DEFAULT_GOAL" is supported only since 3.80+)
+default: build
+.DEFAULT_GOAL: default
+
+$(PYO3_CONFIG_FILE):
  env -u PYO3_CONFIG_FILE cargo build $(cargo_build_opts) -p flameshow
 
-chdig: contrib/flameshow/build/pyo3-build-config-file-$(target).txt
+chdig: $(PYO3_CONFIG_FILE)
  cargo build $(cargo_build_opts)
 
 run: chdig
@@ -66,7 +71,5 @@ rpm: build
 archlinux: build
  CHDIG_VERSION=${CHDIG_VERSION_ARCH} nfpm package --config chdig-nfpm.yaml --packager archlinux
 
-all: build
-
 help:
  @echo "Usage: make [debug=1] [target=<TRIPLE>]"