Skip to content

Conversation

smoser
Copy link
Contributor

@smoser smoser commented Oct 4, 2023

This allows:

make BUILD_D=/tmp/build

Without the change, only a BUILD_D that was under the stacker checkout would work.

Note that in order to take advantage of shared GOCACHE and GOPATH, they must be under BUILD_D (they are by default).

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

This allows:

    make BUILD_D=/tmp/build

Without the change, only a BUILD_D that was under the stacker
checkout would work.

Note that in order to take advantage of shared GOCACHE and GOPATH,
they must be under BUILD_D (they are by default).

Signed-off-by: Scott Moser <smoser@brickies.net>
@smoser smoser requested review from rchincha and hallyn as code owners October 4, 2023 15:51
@codecov
Copy link

codecov bot commented Oct 4, 2023

Codecov Report

Merging #520 (8076490) into main (4689ad5) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #520   +/-   ##
=======================================
  Coverage   13.77%   13.77%           
=======================================
  Files          40       40           
  Lines        5671     5671           
=======================================
  Hits          781      781           
  Misses       4762     4762           
  Partials      128      128           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@smoser smoser merged commit d9f849b into project-stacker:main Oct 4, 2023
Sign up for free to 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.

2 participants