Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 799137 - net-misc/teamviewer: missing depenendecy on session manager
Summary: net-misc/teamviewer: missing depenendecy on session manager
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Martin Dummer
URL:
Whiteboard:
Keywords: PullRequest
Depends on:
Blocks:
 
Reported: 2021-06-29 06:06 UTC by Marcel Schilling
Modified: 2021-08-13 08:47 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 Marcel Schilling 2021-06-29 06:06:34 UTC
Installing Teamviewer on my non-systemd, no-session-manager system results in Teamviewer not launching its GUI:

```sh
sudo emerge net-misc/teamviewer
sudo rc-service teamviewerd start
teamviewer
```

```

Init...
CheckCPU: SSE2 support: yes
Checking setup...
Launching TeamViewer ...
Launching TeamViewer GUI ...
```

I checked `~/local/share/teamviewer15/logfiles/TeamViewer15_Logfile.log` and found the following error:

```
SysSessionInfoManager::GetOwnProcessSession: No session found!
```

This is under X using `startx` to launch my WM (`spectrwm`).


```sh
rc-update show | grep dbus
```

confirms `dbus` to be on the `default` runlevel,

and `/etc/X11/xinit/xinitrc` configures `startx` to lauch my via `dbus-launch`:


```
 exec dbus-launch --exit-with-session spectrwm

```

Also, `elogind` reports a seat for my user session:

```sh
loginctl
```

```
SESSION  UID USER      SEAT  TTY
      2 1001 mschilli seat0 tty1

1 sessions listed.
```


Based on this discussion, I assume the issue is me not using a session manager but `startx`: https://community.teamviewer.com/English/discussion/comment/32597

Note that this is ongoing for four years with seamingly no interest by TeamViewer to address this issue at all.

For me, this means a hard pass on TeamViewer but I'd still consider this a packagaing bug on Gentoo's end: I could get to an unworkable state without any warnings or errors.

Depending on a session manager via the ebuild would have saved me a lot of time because I would have seen `net-misc/teamviewer` pulling in a session manager and could have decided not to emerge it in the first place right then.

I am not sure if there is a virtual package providing 'some session manager', but if there is not, this bug could justify its creating IMHO.
Comment 1 Martin Dummer 2021-07-30 13:54:19 UTC
Hello Marcel,

can you clarify what you mean with "session manager"? Package names please?

Martin
Comment 2 Martin Dummer 2021-07-31 20:11:39 UTC
I tested teamviewer GUI on a clean installation with a display manager https://wiki.gentoo.org/wiki/Display_manager 
So your bug report is substantial correct, teamviewer gui will run only in a session initially started with a display manager like "lxde" or "sddm".
It will NOT run on a x-server started with the minimalisic "startx" command.

But making teamviewer depend on a display manager will not guarantee that it will work under all possible use cases. 
The dependency mechanism will not fully solve this problem.
So I am not willing to add extra dependencies for one-of-a-list of display managers (with the risk of this list being incomnplete) or starting a developer discussion for a "virtual/display-manager".

Instead, I will add a postinstall note that teamviewer gui will only work if started from a display-manager's session.
Additionally I add a "optfeature" message which reminds a user to install one of the display managers when its not installed.
This must be sufficient.
Comment 3 Marcel Schilling 2021-08-02 06:03:24 UTC
Hello Martin,

Thank you for taking care of this.
I understand why you don't want to start a virtual package for this caser and agree that your solution is a reasonable compromise.

Thanks,

Marcel
Comment 4 Martin Dummer 2021-08-02 17:00:25 UTC
I will add this to the next version teamviewer-15.20.3 the next few days.
Comment 5 Larry the Git Cow gentoo-dev 2021-08-13 08:47:05 UTC
The bug has been closed via the following commit(s):

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

commit 0466243c2a69de595dfc7c73158e758361ce81d0
Author:     Martin Dummer <martin.dummer@gmx.net>
AuthorDate: 2021-08-07 13:26:55 +0000
Commit:     Ionen Wolkens <ionen@gentoo.org>
CommitDate: 2021-08-13 08:45:42 +0000

    net-misc/teamviewer: version bump to 15.20.6
    
    * Add installation note that teamviewer gui works only in a session
      initiated by a display manager
    * Add source for arm/arm64 architecture for testing
    
    Closes: https://bugs.gentoo.org/799137
    Package-Manager: Portage-3.0.20, Repoman-3.0.3
    Signed-off-by: Martin Dummer <martin.dummer@gmx.net>
    Closes: https://github.com/gentoo/gentoo/pull/21906
    Signed-off-by: Ionen Wolkens <ionen@gentoo.org>

 net-misc/teamviewer/Manifest                  |   4 +
 net-misc/teamviewer/teamviewer-15.20.6.ebuild | 163 ++++++++++++++++++++++++++
 2 files changed, 167 insertions(+)