Skip to content
This repository has been archived by the owner on Jul 17, 2023. It is now read-only.

Commit

Permalink
Merge pull request #58 from jamesjarvis/patch-1
Browse files Browse the repository at this point in the history
Fix #57 docker fqn replace with --arch on macOS
  • Loading branch information
Tatskaari authored Feb 24, 2021
2 parents b85bbd0 + 1ab1091 commit 06a74f6
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion k8s/k8s.build_defs
Original file line number Diff line number Diff line change
Expand Up @@ -60,7 +60,7 @@ def k8s_config(name:str, srcs:list, containers:list=[], params:dict=None, visibi
)

# macos sed only supports posix regex expressions so escape sequences like \b don't work
boundary_expr = "\\b" if CONFIG.OS != 'darwin' else ""
boundary_expr = "\\b" if CONFIG.HOSTOS != 'darwin' else ""
subcommands = ' '.join([
f'-e "s|{container}{boundary_expr}|$(cat $(location {fqn}))|g"'
for container, fqn in sorted(zip(containers, fqns))
Expand Down

0 comments on commit 06a74f6

Please sign in to comment.