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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Downgrade "aufs-tools" and "cgroupfs-mount" to "Suggests" #472

Merged
merged 1 commit into from May 28, 2020

Conversation

tianon
Copy link
Contributor

@tianon tianon commented May 27, 2020

Quoting from https://www.debian.org/doc/debian-policy/ch-relationships.html#binary-dependencies-depends-recommends-suggests-enhances-pre-depends (as a refresher/context):

Recommends

This declares a strong, but not absolute, dependency.

The Recommends field should list packages that would be found together with this one in all but unusual installations.

Suggests

This is used to declare that one package may be more useful with one or more others. Using this field tells the packaging system and the user that the listed packages are related to this one and can perhaps enhance its usefulness, but that installing this one without them is perfectly reasonable.

My justification for this change, given that context, is that aufs hasn't been the default graph driver for quite some time now, and I'd wager the majority of new installations these days already have cgroups via systemd or some other similar package, so it's going to be less common for users to need something like cgroupfs-mount (and pulling it in on systemd-using systems is a noop that just generates noise).

(Another way to consider this is that in a properly configured APT installation, Recommends get installed by default but Suggests do not. 馃槃)

There has been some discussion around the former in https://bugs.debian.org/948087 recently that prompted me to finally pull the trigger on opening this, but it's something I've been thinking about for quite some time now.

Signed-off-by: Tianon Gravi <admwiggin@gmail.com>
@AkihiroSuda
Copy link
Contributor

Can we just drop them?

@thaJeztah
Copy link
Member

Ah, yes, been meaning to look at this, thanks!

Can we just drop them?

I'd be ok with dropping them at some point, perhaps it's ok to keep them for now (at least the cgroup ones). With Suggests is basically just metadata, so I don't think there's any option that will automatically install these.

Copy link
Member

@thaJeztah thaJeztah left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

(unless we strongly want these to be removed altogether)

@thaJeztah
Copy link
Member

@tianon
Copy link
Contributor Author

tianon commented May 28, 2020

IMO these shouldn't be dropped completely yet -- aufs-tools is something folks should install if they still want to use aufs (which is deprecated, but still supported) and cgroupfs-mount is the type of things folks will need if their init system (or something else) doesn't set up cgroups for them, so I think there's still value in the communication these optional bits provide to users.

Once aufs is actually removed, then we should drop aufs-tools completely.

Copy link
Contributor

@silvin-lubecki silvin-lubecki left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@thaJeztah
Copy link
Member

Let's merge this 馃憤 馃

@thaJeztah thaJeztah merged commit f0626d6 into docker:master May 28, 2020
@tianon tianon deleted the suggests branch May 28, 2020 16:27
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