debian,vendor: import github.com/snapcore/squashfs and use #4049

Open
wants to merge 9 commits into
from

Conversation

Projects
None yet
5 participants
Collaborator

mvo5 commented Oct 16, 2017

This PR will add a new vendor dependency github.com/snapcore/squashfs
that is used to provide squashfuse for lxd.

It superseeds #4030 and still needs some love but for early review it should be ok. Some trickery needed to make github.com/snapcore/squashfuse go get able (snapcore/squashfuse@ed6f37b).

mvo5 added some commits Oct 16, 2017

debian,vendor: import github.com/snapcore/squashfs and use
This PR will add a new vendor dependency github.com/snapcore/squashfs
that is used to provide squashfuse for lxd.
Contributor

Conan-Kudo commented Oct 19, 2017

@mvo5 This is a bit less gross, but why do this through the go vendor?

@pedronis pedronis added the Blocked label Nov 9, 2017

Contributor

pedronis commented Nov 9, 2017

marked as blocked as apparently it's not fully working and we need to look into a different approach according to @mvo5

Collaborator

mvo5 commented Nov 9, 2017

This needs a forum topic with "upcoming" tag and then can be closed.

Contributor

zyga commented Nov 16, 2017

@mvo5 shall we close this or do you think it is a viable approach but just needs more work?

@zyga zyga added the Decaying label Nov 29, 2017

Contributor

Conan-Kudo commented Dec 20, 2017

Please don't do it this way. If you must vendor it, do it the more standard way (add an additional tarball to the dsc packaging and build it appropriately).

Shoehorning into weird Go things is just horrible.

Codecov Report

Merging #4049 into master will decrease coverage by <.01%.
The diff coverage is 50%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #4049      +/-   ##
==========================================
- Coverage   78.04%   78.03%   -0.01%     
==========================================
  Files         449      449              
  Lines       30907    30912       +5     
==========================================
+ Hits        24121    24122       +1     
- Misses       4774     4779       +5     
+ Partials     2012     2011       -1
Impacted Files Coverage Δ
systemd/systemd.go 83.98% <50%> (-0.6%) ⬇️
overlord/ifacestate/helpers.go 63.48% <0%> (-0.83%) ⬇️

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 7855e95...0aa81c5. Read the comment docs.

Collaborator

mvo5 commented Dec 21, 2017

@Conan-Kudo Yes, sorry for this. Unfortunately there is no clean deb package way to vendor this, this hack is about the cleanest solution I was able to find :/

Contributor

Conan-Kudo commented Dec 21, 2017

@mvo5 Last I checked, you should be able to just attach more source tarballs in the dsc description when uploading to Ubuntu. The rules file can be adjusted to actually use the second source vendor it in. In addition, this change should be considered temporary and dropped for bionic, as that release can just use regular squashfuse. Same goes for Debian sid.

There's no good reason to make a permanent hack here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment