-
Notifications
You must be signed in to change notification settings - Fork 441
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
go plugin: do not remove install /bin directory before build #2957
Conversation
Packages configured in a part's stage-packages are installed prior to build and installed to $SNAPCRAFT_PART_INSTALL. Cleaning $SNAPCRAFT_PART_INSTALL/bin will purge any files that may have been installed there. Running go build should overwrite any previously installed files, which will be OK for incremental builds. However, as the go plugin does not enable out_of_source_build, the entire build directory is purged on every build anyways. Add spread test for coverage. Signed-off-by: Chris Patterson <chris.patterson@canonical.com>
LGTM |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, we now need an extra parameter for os.makedirs
though
tests/spread/plugins/go/snaps/go-use-stage-packages/go-hello/main.go
Outdated
Show resolved
Hide resolved
Codecov Report
@@ Coverage Diff @@
## master #2957 +/- ##
=========================================
Coverage ? 88.17%
=========================================
Files ? 229
Lines ? 16569
Branches ? 2560
=========================================
Hits ? 14610
Misses ? 1425
Partials ? 534
Continue to review full report at Codecov.
|
Packages configured in a part's stage-packages are installed prior
to build and installed to $SNAPCRAFT_PART_INSTALL. Cleaning
$SNAPCRAFT_PART_INSTALL/bin will purge any files that may have been
installed there.
Running go build should overwrite any previously installed files,
which will be OK for incremental builds. However, as the go plugin
does not enable out_of_source_build, the entire build directory is
purged on every build anyways.
Add spread test for coverage.
Signed-off-by: Chris Patterson chris.patterson@canonical.com
Reported by @stgraber