Skip to content

Commit fc75262

Browse files
committed
Makefile: adjust default target, broaden all target
Previously the default target ('all') built the rustls-ffi static library and the client/server example binaries. This commit adds a new default target ('default') that only builds the static lib. It also updates 'all' to include integration and connect-tests to truly be all the things.
1 parent 51ac465 commit fc75262

File tree

2 files changed

+7
-5
lines changed

2 files changed

+7
-5
lines changed

.github/workflows/test.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -64,7 +64,7 @@ jobs:
6464
- name: Build release binaries
6565
run: |
6666
make clean
67-
make CC=${{ matrix.cc }} CRYPTO_PROVIDER=${{ matrix.crypto }} PROFILE=release
67+
make CC=${{ matrix.cc }} CRYPTO_PROVIDER=${{ matrix.crypto }} PROFILE=release test
6868
- name: Verify release builds were not using ASAN
6969
if: runner.os == 'Linux' # For 'nm'
7070
run: |

Makefile

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -41,12 +41,14 @@ ifeq ($(COMPRESSION), true)
4141
LDFLAGS += -lm
4242
endif
4343

44-
all: target/client target/server
44+
default: target/$(PROFILE)/librustls_ffi.a
4545

46-
test: all
46+
all: default test integration connect-test
47+
48+
test: target/client target/server
4749
${CARGO} test ${CARGOFLAGS}
4850

49-
integration: all
51+
integration: test
5052
${CARGO} test ${CARGOFLAGS} -- --ignored
5153

5254
connect-test: target/client
@@ -97,4 +99,4 @@ format-check:
9799
cargo fmt --check
98100
sed -i -e 's/if true {/ffi_panic_boundary! {/g' src/*.rs
99101

100-
.PHONY: all clean test integration format format-check
102+
.PHONY: default all clean test integration format format-check

0 commit comments

Comments
 (0)