[PATCH] Move captive portal to fedora-release-workstation

Josh Boyer jwboyer at fedoraproject.org
Mon Oct 6 12:30:36 UTC 2014


On Mon, Oct 6, 2014 at 8:21 AM, Stephen Gallagher <sgallagh at redhat.com> wrote:
>
>
> Just a bit of an update; the new upgrade plan should be able to resolve
> this issue without the patch (and also in a way that is likely
> acceptable to all groups).
>
> We can remove the explicit Requires: NM-captive-portal-fedora from both
> gnome-shell and fedora-release workstation, because the new 'fedup
> --network 21 --product=workstation' command will automatically install
> it as long as it's part of the @^workstation-product-environment group
> in comps (which a quick inspection shows is not currently the case but
> is a two-line change that I will submit right now).

I don't see a need to remove it from fedora-release-workstation now
that it is already in and built.

Also, unless I'm misunderstanding something, the environment group
doesn't handle cases where someone installs Workstation, the removes
pieces of what we consider "core" functionality.  At that point they
are no longer running Workstation.  (I'm not sure we have a good
handle on that overall anyway, but removing the current Requires is
fairly pointless.)

> Of course, this approach has the same issue as this patch does, which is
> that it will only ensure that this new package is added to the
> Workstation upgrades and not to standard upgrades...

I still don't think that is bad, given that is the entire reason for
the patch in the first place.

josh


More information about the desktop mailing list