Skip to content

lp 1768233 workaround for plainbox_provider plugin#2140

Merged
sergiusens merged 2 commits intocanonical:masterfrom
jocave:workaround-1768233-for-plainbox-providers
May 18, 2018
Merged

lp 1768233 workaround for plainbox_provider plugin#2140
sergiusens merged 2 commits intocanonical:masterfrom
jocave:workaround-1768233-for-plainbox-providers

Conversation

@jocave
Copy link
Contributor

@jocave jocave commented May 17, 2018

  • Have you followed the guidelines for contributing?
  • Have you signed the CLA?
  • If this is a bugfix. Have you checked that there is a bug report open for the issue you are trying to fix on bug reports?
  • If this is a new feature. Have you discussed the design on the forum?
  • Have you successfully run ./runtests.sh static?
  • Have you successfully run ./runtests.sh unit?

Launchpad bug 1768233 discussed a problem that was encountered when building checkbox snaps using the plainbox-provider plugin. This PR includes the workaround discussed so builds can begin working again. Integration test included for the original report.

@codecov-io
Copy link

codecov-io commented May 17, 2018

Codecov Report

Merging #2140 into master will increase coverage by <.01%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2140      +/-   ##
==========================================
+ Coverage   91.01%   91.02%   +<.01%     
==========================================
  Files         193      193              
  Lines       12181    12182       +1     
  Branches     1820     1820              
==========================================
+ Hits        11087    11089       +2     
  Misses        752      752              
+ Partials      342      341       -1
Impacted Files Coverage Δ
snapcraft/plugins/plainbox_provider.py 100% <100%> (ø) ⬆️
snapcraft/internal/elf.py 83.17% <0%> (+0.32%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d2da431...39a328d. Read the comment docs.

@sergiusens sergiusens merged commit d932f0f into canonical:master May 18, 2018
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.

3 participants