Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 811078 - net-misc/remmina requires appindicator and breaks other packages
Summary: net-misc/remmina requires appindicator and breaks other packages
Status: UNCONFIRMED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Ben Kohler
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2021-08-30 07:15 UTC by Tomas Janco
Modified: 2021-09-03 14:18 UTC (History)
2 users (show)

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


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Tomas Janco 2021-08-30 07:15:26 UTC
Since 1.4.20 libappindicator is required for remmina and it installs system-wide.

However this makes all electron-based packages (i.e. slack, discord, teams) and probably others to use AppIndicator instead of whatever they were using for the systray icon.

This has a negative effect of losing the right/left-click function for status icon. Without AppIndicator, the icon has two distinct functions - left click opens the application, right click opens context menu. With AppIndicator the menu is open for both right and left click.

Remmina works fine without AppIndicator (just it does not display the status icon). I'd rather chose losing status icon for remmina than to break other packages icons if I'd have the option.

I suggest the appindicator use flag should be reintroduced.
Comment 1 Joakim Tjernlund 2021-08-30 12:01:56 UTC
Sounds right, all other pkgs I have installed has USE=appindicator:

gnome-extra/nm-applet-1.22.0 (appindicator ? dev-libs/libappindicator:3)
mate-base/mate-control-center-1.24.2 (appindicator ? dev-libs/libappindicator:3)
mate-extra/mate-polkit-1.24.0-r1 (appindicator ? dev-libs/libappindicator:3)
net-wireless/blueman-2.2.2 (appindicator ? dev-libs/libappindicator:3[introspection])
Comment 2 Ben Kohler gentoo-dev 2021-08-31 10:54:04 UTC
It works fine without libappindicator?

-- Checking for module 'ayatana-appindicator3-0.1'
--   Package 'ayatana-appindicator3-0.1', required by 'virtual:world', not found
-- Checking for module 'appindicator3-0.1'
--   Package 'appindicator3-0.1', required by 'virtual:world', not found
CMake Error at /usr/share/cmake/Modules/FindPackageHandleStandardArgs.cmake:230 (message):
  Could NOT find APPINDICATOR (missing: APPINDICATOR_LIBRARY
  APPINDICATOR_INCLUDE_DIR)
Call Stack (most recent call first):
  /usr/share/cmake/Modules/FindPackageHandleStandardArgs.cmake:594 (_FPHSA_FAILURE_MESSAGE)
  cmake/FindAPPINDICATOR.cmake:51 (find_package_handle_standard_args)
  cmake/FindOptionalPackage.cmake:27 (find_package)
  src/CMakeLists.txt:268 (find_required_package)


-- Configuring incomplete, errors occurred!
See also "/var/tmp/portage/net-misc/remmina-1.4.20/work/remmina-1.4.20_build/CMakeFiles/CMakeOutput.log".
See also "/var/tmp/portage/net-misc/remmina-1.4.20/work/remmina-1.4.20_build/CMakeFiles/CMakeError.log".
 * ERROR: net-misc/remmina-1.4.20::gentoo failed (configure phase):
 *   cmake failed
Comment 4 Tomas Janco 2021-08-31 11:14:21 UTC
I guess it works at runtime, however it does not build.
Comment 5 Ben Kohler gentoo-dev 2021-08-31 11:24:46 UTC
Then we have a bit of a conundrum here.

Shouldn't the real bug report be "electron apps are broken if libappindicator is installed" ?

I should be able to depend on a lib that my package needs..
Comment 6 Tomas Janco 2021-09-03 14:16:50 UTC
I agree electron packages are broken. And there is no way to fix them as they are often binary packages. Electron is not able to fix this either https://github.com/electron/electron/issues/14941

My argument is: If appindicator has such system-wide effect, I should by able to not use it if it's not essential, or I should by able to install it isolated

I do not see a good reason for removing the appindicator build option by remmina maintaniners. Can we have that back please? At least as a patch and use flag in gentoo package?
Comment 7 Ben Kohler gentoo-dev 2021-09-03 14:18:28 UTC
Please ask for this upstream.

I cannot add the flag back now, since it's not actually optional (unless we start carrying extra patches forever).