Skip to content

Makefile PATH fix - #20

Open
mtesseract wants to merge 2 commits into
mainfrom
mc/makefile-path-fix
Open

Makefile PATH fix#20
mtesseract wants to merge 2 commits into
mainfrom
mc/makefile-path-fix

Conversation

@mtesseract

@mtesseractmtesseract commented Oct 19, 2022

Copy link
Copy Markdown

Apparently the Makefile currently does not handle whitespaces correctly in PATH.

Switched to branch 'main'
Your branch is up to date with 'origin/main'.
➜ helmtest git:(main) ✗ PATH="/foo bar/bin:/usr/local/bin:/bin:/usr/bin" make go-generated-srcs env: bar/bin:/usr/local/bin:/bin:/usr/bin: No such file or directory
make: *** [deps] Error 127
➜ helmtest git:(main) ✗ PATH="/foo/bar/bin:/usr/local/bin:/bin:/usr/bin" make go-generated-srcs
+ deps
go: finding module for package github.com/stackrox/rox/pkg/buildinfo
^Cmake: *** [deps] Interrupt: 2
➜ helmtest git:(main) ✗ sb
Switched to branch 'mc/makefile-path-fix'
Your branch is up to date with 'origin/mc/makefile-path-fix'.
➜ helmtest git:(mc/makefile-path-fix) ✗ PATH="/foo bar/bin:/usr/local/bin:/bin:/usr/bin" make go-generated-srcs
+ deps
go: finding module for package github.com/stackrox/rox/pkg/buildinfo
^Cmake: *** [deps] Interrupt: 2

@porridge
porridge requested a review from a team as a code ownerApril 8, 2026 11:38
@porridge
porridge requested review from mclasmeier and removed request for a team and misbernerApril 8, 2026 11:38
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

@mtesseract