Skip to content

Commit

Permalink
chore: separating heaptrack from debug build
Browse files Browse the repository at this point in the history
  • Loading branch information
gabrielmer committed Jan 23, 2025
1 parent dfd06fa commit 8688839
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 3 deletions.
2 changes: 1 addition & 1 deletion Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -83,7 +83,7 @@ HEAPTRACKER_INJECT ?= 0
ifeq ($(HEAPTRACKER), 1)
# Needed to make nimbus-build-system use the Nim's 'heaptrack_support' branch
DOCKER_NIM_COMMIT := NIM_COMMIT=heaptrack_support
TARGET := debug
TARGET := prod-with-heaptrack

ifeq ($(HEAPTRACKER_INJECT), 1)
# the Nim compiler will load 'libheaptrack_inject.so'
Expand Down
9 changes: 7 additions & 2 deletions ci/Jenkinsfile.release
Original file line number Diff line number Diff line change
Expand Up @@ -56,7 +56,12 @@ pipeline {
)
booleanParam(
name: 'DEBUG',
description: 'Enable debug features (heaptrack).',
description: 'Enable debug features',
defaultValue: false
)
booleanParam(
name: 'HEAPTRACK',
description: 'Enable heaptrack build',
defaultValue: false
)
}
Expand All @@ -73,7 +78,7 @@ pipeline {
"--build-arg=NIMFLAGS='${params.NIMFLAGS} -d:postgres ' " +
"--build-arg=LOG_LEVEL='${params.LOWEST_LOG_LEVEL_ALLOWED}' " +
"--build-arg=DEBUG='${params.DEBUG ? "1" : "0"} ' " +
"--target=${params.DEBUG ? "debug" : "prod"} ."
"--target=${params.HEAPTRACK ? "prod-with-heaptrack" : "prod"} ."
)
} }
}
Expand Down

0 comments on commit 8688839

Please sign in to comment.