Skip to content
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

Clarify preferred btrfs-progs dependency and fix some minor inconsistencies #9756

Merged
merged 1 commit into from
Dec 23, 2014

Conversation

tianon
Copy link
Member

@tianon tianon commented Dec 19, 2014

cc @vbatts

There's been some confusion here thinking that 3.8 is fine and that 3.16 should not be preferred (and that specifying btrfs_noversion is "OK" even with 3.16+), so I've rewritten this bit to hopefully clarify that we do in fact prefer 3.16.1+ and that lower versions are highly discouraged but supported where necessary.

…encies

Signed-off-by: Andrew "Tianon" Page <admwiggin@gmail.com>
@vbatts
Copy link
Contributor

vbatts commented Dec 21, 2014

LGTM

On Fri, Dec 19, 2014 at 5:30 PM, Tianon Gravi notifications@github.com
wrote:

cc @vbatts https://github.com/vbatts

There's been some confusion here thinking that 3.8 is fine and that 3.16
should not be preferred (and that specifying btrfs_noversion is "OK" even
with 3.16+), so I've rewritten this bit to hopefully clarify that we do in
fact prefer 3.16.1+ and that lower versions are highly discouraged but

supported where necessary.

You can merge this Pull Request by running

git pull https://github.com/tianon/docker clarify-btrfs-version-dep

Or view, comment on, or merge it at:

#9756
Commit Summary

  • Clarify preferred btrfs-progs dependency and fix some minor
    inconsistencies

File Changes

Patch Links:


Reply to this email directly or view it on GitHub
#9756.

@SvenDowideit
Copy link
Contributor

LGTM

tianon added a commit that referenced this pull request Dec 23, 2014
Clarify preferred btrfs-progs dependency and fix some minor inconsistencies
@tianon tianon merged commit 084bf8a into moby:master Dec 23, 2014
@tianon tianon deleted the clarify-btrfs-version-dep branch December 23, 2014 17:25
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.

None yet

4 participants