Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 553946 - gnome-extra/cinnamon-control-center: make networkmanager, modemmanager optional
Summary: gnome-extra/cinnamon-control-center: make networkmanager, modemmanager optional
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: [OLD] GNOME (show other bugs)
Hardware: All Linux
: Normal enhancement
Assignee: Gentoo Cinnamon Team (DISABLED)
URL:
Whiteboard:
Keywords: NeedPatch, UPSTREAM
: 564778 (view as bug list)
Depends on: 704532
Blocks:
  Show dependency tree
 
Reported: 2015-07-05 02:21 UTC by kisak42
Modified: 2020-05-29 00:11 UTC (History)
5 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
ebuild with optional nm/mm dependencies (cinnamon-control-center-2.6.0-r1.ebuild,3.03 KB, text/plain)
2015-09-17 16:50 UTC, Garry Filakhtov
Details
patch for making networkmanager/modemmanager optional (cinnamon-control-center-2.6.0-networkmanager-optional.patch,3.75 KB, patch)
2015-09-17 16:53 UTC, Garry Filakhtov
Details | Diff
cinnamon-control-center ebuild with optional modemmanager/networkmanager support (cinnamon-control-center-2.6.0-r1.ebuild,3.10 KB, text/plain)
2015-09-17 19:56 UTC, Garry Filakhtov
Details
patch for making networkmanager/modemmanager optional (no auto deps) (cinnamon-control-center-2.6.0-networkmanager-optional.patch,6.41 KB, patch)
2015-09-17 19:56 UTC, Garry Filakhtov
Details | Diff
cinnamon-2.8.5.ebuild (cinnamon-2.8.5.ebuild,7.90 KB, text/plain)
2015-11-21 19:14 UTC, Perfect Gentleman
Details
cinnamon-control-center-2.8.1.ebuild (cinnamon-control-center-2.8.1-r1.ebuild,2.72 KB, text/plain)
2015-11-21 19:15 UTC, Perfect Gentleman
Details
cinnamon-control-center-2.8.1.ebuild (cinnamon-control-center-2.8.1-r1.ebuild,2.76 KB, text/plain)
2015-11-21 19:23 UTC, Perfect Gentleman
Details
cinnamon-3.0.7.ebuild (cinnamon-3.0.7.ebuild,7.76 KB, text/plain)
2016-08-04 12:14 UTC, Perfect Gentleman
Details
cinnamon-control-center-3.0.1.ebuild (cinnamon-control-center-3.0.1.ebuild,2.63 KB, text/plain)
2016-08-04 12:14 UTC, Perfect Gentleman
Details
Optional networkmanager/modemmanager (cinnamon-control-center-3.6.5-r1.ebuild,2.62 KB, text/plain)
2018-05-01 03:01 UTC, Matthew Turnbull
Details
Fix non modemmanager build (cinnamon-control-center-3.6.5-no-modemmanager-fix.patch,1.19 KB, patch)
2018-05-01 03:03 UTC, Matthew Turnbull
Details | Diff
networkmanager/modemmanager optional ebuild (cinnamon-control-center-3.8.1.ebuild,2.61 KB, text/plain)
2018-07-31 20:44 UTC, Nick Reale
Details
Update for cinnamon 4.0.0 (cinnamon-control-center-4.0.0-r1.ebuild,2.51 KB, text/plain)
2018-12-18 20:55 UTC, Matthew Turnbull
Details

Note You need to log in before you can comment on or make changes to this bug.
Description kisak42 2015-07-05 02:21:50 UTC
I have been wanting to return to using cinnamon for a while now, but I have active disinterest in running networkmanager on my system. Please make networkmanager and by extension, modemmanager optional again.

This was previously discussed on #529868, but that was not the main focus of that bug report. As noted in #529868, the cinnamon devs did express interest in upstreaming this change so that we do not need to maintain a patchset. This appears to be the only package currently preventing cinnamon from being emerged without networkmanager.

Reproducible: Always
Comment 1 Garry Filakhtov 2015-09-17 16:50:12 UTC
Created attachment 412100 [details]
ebuild with optional nm/mm dependencies
Comment 2 Garry Filakhtov 2015-09-17 16:53:05 UTC
Created attachment 412102 [details, diff]
patch for making networkmanager/modemmanager optional
Comment 3 Garry Filakhtov 2015-09-17 16:55:53 UTC
I've attached ebuild and appropriate patch to make networkmanager/modemmanager optional.

In case if you will disable both flags: networkmanager and modemmanager - "network" module will not be built.

In case if you enable networkmanager, but disable modemmanager - you will be able to use network menu, but modemmanager functions will be not available.

Had to patch configure.ac, Makefile.am and cc-network-panel.c source file.
Comment 4 Garry Filakhtov 2015-09-17 17:00:00 UTC
Please, forward this patch to upstream, so we will get possibility to disable MM/NM oob.
Comment 5 Alexandre Rostovtsev (RETIRED) gentoo-dev 2015-09-17 17:19:33 UTC
(In reply to Igor Filakhtov from comment #2)
> Created attachment 412102 [details, diff] [details, diff]
> patch for making networkmanager/modemmanager optional

Unfortunately, we cannot accept this patch in its current form. In gentoo we must be able to disable networkmanager and modemmanager support even when building on a system where networkmanager and/or modemmanager are installed.

See https://wiki.gentoo.org/wiki/Project:Quality_Assurance/Automagic_dependencies for the reasoning.
Comment 6 Garry Filakhtov 2015-09-17 17:23:34 UTC
Alexandre,

Thanks. Will do ASAP.
Comment 7 Garry Filakhtov 2015-09-17 19:56:04 UTC
Created attachment 412118 [details]
cinnamon-control-center ebuild with optional modemmanager/networkmanager support

Disabled automatic dependencies in ebuild
Comment 8 Garry Filakhtov 2015-09-17 19:56:53 UTC
Created attachment 412120 [details, diff]
patch for making networkmanager/modemmanager optional (no auto deps)
Comment 9 Garry Filakhtov 2015-09-17 19:58:25 UTC
Attached new patch/ebuild.

Automatic dependencies are removed, --disable-networkmanager and --disable-modemmanager flags were introduced instead.

combination --enable-modemmanger --disable-networkmanager gives an error, because mm without nm makes no sense.

Please, review.
Thanks in advance!
Comment 10 Perfect Gentleman 2015-11-21 05:48:41 UTC
up
Comment 11 Perfect Gentleman 2015-11-21 19:14:05 UTC
ebuilds for Cinnamon without NetworkManagerk
Comment 12 Perfect Gentleman 2015-11-21 19:14:44 UTC
Created attachment 417510 [details]
cinnamon-2.8.5.ebuild
Comment 13 Perfect Gentleman 2015-11-21 19:15:21 UTC
Created attachment 417512 [details]
cinnamon-control-center-2.8.1.ebuild
Comment 14 Perfect Gentleman 2015-11-21 19:23:25 UTC
Created attachment 417514 [details]
cinnamon-control-center-2.8.1.ebuild
Comment 15 Pacho Ramos gentoo-dev 2016-02-07 16:49:20 UTC
*** Bug 564778 has been marked as a duplicate of this bug. ***
Comment 16 Perfect Gentleman 2016-08-04 12:14:14 UTC
Created attachment 442502 [details]
cinnamon-3.0.7.ebuild

cinnamon-3.0.7.ebuild
Comment 17 Perfect Gentleman 2016-08-04 12:14:55 UTC
Created attachment 442504 [details]
cinnamon-control-center-3.0.1.ebuild

cinnamon-control-center-3.0.1.ebuild
Comment 18 Joakim Tjernlund 2016-11-30 01:02:57 UTC
Please ?
At least remove USE modemmanager requirement. Why do one need to
force that on a DE?
Comment 19 Garry Filakhtov 2016-11-30 07:19:06 UTC
Hi @Joakim,

I have PR open for cinnamon-control-center in upstream repo: https://github.com/linuxmint/cinnamon-control-center/pull/136. Still no luck on merging it though.
Comment 20 Joakim Tjernlund 2016-11-30 08:21:39 UTC
(In reply to Garry Filakhtov from comment #19)
> Hi @Joakim,
> 
> I have PR open for cinnamon-control-center in upstream repo:
> https://github.com/linuxmint/cinnamon-control-center/pull/136. Still no luck
> on merging it though.

Hi Garry

Thanks for letting me know. I commented on upstream lets hope it
is resolved soon.
Comment 21 NHO 2017-10-29 17:20:05 UTC
This appears to be merged in upstream.
I use NM, but don't use MM, so this patch would be nice to have.
Comment 22 Matthew Turnbull 2018-05-01 03:01:21 UTC
Created attachment 529164 [details]
Optional networkmanager/modemmanager

networkmanager and modemmanager have been optional since 3.4.0

Though, at some point, a set of #ifdefs went AWOL, so there is also an included patch from 3.8.0 upstream.
Comment 23 Matthew Turnbull 2018-05-01 03:03:03 UTC
Created attachment 529166 [details, diff]
Fix non modemmanager build

Patch from upstream:
https://github.com/linuxmint/cinnamon-control-center/commit/6a916057b06fda8d3ca90520e6936f3261a915ed
Comment 24 Nick Reale 2018-07-31 20:44:36 UTC
Created attachment 541912 [details]
networkmanager/modemmanager optional ebuild

Running 3.8.1 with this ebuild compiles fine and seems to run fine, no other patches or changes seem necessary.
Comment 25 Matthew Turnbull 2018-12-18 20:55:55 UTC
Created attachment 558112 [details]
Update for cinnamon 4.0.0

Update for cinnamon 4.0.0. Patch no longer needed as this is fully supported upstream.

Can we also get the component changed back to Current? This is an ongoing ask applicable to the current cinnamon versions.
Comment 26 Larry the Git Cow gentoo-dev 2020-05-29 00:11:31 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=8da71837558fd68d32a72b8b7b5fb88071e78ecc

commit 8da71837558fd68d32a72b8b7b5fb88071e78ecc
Author:     Matthew Turnbull <sparky@bluefang-logic.com>
AuthorDate: 2020-05-13 05:52:22 +0000
Commit:     Matt Turner <mattst88@gentoo.org>
CommitDate: 2020-05-29 00:10:46 +0000

    gnome-extra/cinnamon: 4.4.8 version bump
    
    Update cinnamon to 4.4.8. Re-work python support
    as python-single-r1 with python3_{6,7} compat.
    Clean-up dependencies and make networkmanager optional.
    
    Closes: https://bugs.gentoo.org/553946
    Closes: https://bugs.gentoo.org/704532
    Closes: https://bugs.gentoo.org/662546
    Closes: https://bugs.gentoo.org/709016
    Bug: https://bugs.gentoo.org/718316
    Package-Manager: Portage-2.3.99, Repoman-2.3.22
    Signed-off-by: Matthew S. Turnbull <sparky@bluefang-logic.com>
    Signed-off-by: Matt Turner <mattst88@gentoo.org>

 gnome-extra/cinnamon/Manifest              |   1 +
 gnome-extra/cinnamon/cinnamon-4.4.8.ebuild | 176 +++++++++++++++++++++++++++++
 gnome-extra/cinnamon/metadata.xml          |   8 ++
 3 files changed, 185 insertions(+)

Additionally, it has been referenced in the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=db2ce06e3503ba5c9c7c793d0b42405b406355ee

commit db2ce06e3503ba5c9c7c793d0b42405b406355ee
Author:     Matthew Turnbull <sparky@bluefang-logic.com>
AuthorDate: 2020-05-13 03:19:55 +0000
Commit:     Matt Turner <mattst88@gentoo.org>
CommitDate: 2020-05-29 00:10:46 +0000

    gnome-extra/cinnamon-control-center: 4.4.0 version bump
    
    Update cinnamon-control-center to 4.4.0. CUPS dependencies
    belong in cinnamon-settings-daemon. Make networkmanager
    and modemmanager optional. Clean-up dead configure options.
    
    Bug: https://bugs.gentoo.org/553946
    Bug: https://bugs.gentoo.org/704532
    Package-Manager: Portage-2.3.99, Repoman-2.3.22
    Signed-off-by: Matthew S. Turnbull <sparky@bluefang-logic.com>
    Signed-off-by: Matt Turner <mattst88@gentoo.org>

 gnome-extra/cinnamon-control-center/Manifest       |  1 +
 .../cinnamon-control-center-4.4.0.ebuild           | 87 ++++++++++++++++++++++
 gnome-extra/cinnamon-control-center/metadata.xml   | 12 +++
 3 files changed, 100 insertions(+)