Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 465974 - =net-misc/teamviewer-8.0.17864 - Version bump
Summary: =net-misc/teamviewer-8.0.17864 - Version bump
Status: RESOLVED WONTFIX
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: Normal enhancement
Assignee: Diego Elio Pettenò (RETIRED)
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-04-15 11:27 UTC by Ferry
Modified: 2013-04-18 13:14 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 Ferry 2013-04-15 11:27:36 UTC
Hi there,

the current ebuild (8.0.17147) is using the debian/ubuntu package. There's also an 'other distro's' tar.gz release with version 8.0.17864 which, at least by build number, appears newer.

The current version is prompting for the EULA every time. Haven't noticed any real issues besides that though.

Reproducible: Always
Comment 1 Philipp Riegger 2013-04-18 08:19:34 UTC
Some remarks to the current ebuild:

- Please don't rename the /opt/teamviewer8 directory to /opt/teamviewer-8. The path is used internally in scripts and this might lead to errors. If you really need to change the path, please make sure all the scripts are adjusted.
- Please use the wrapper scripts for the TeamViewer daemon. They not only start the daemon, but also create necessary directories in /opt/teamviewer8. The EULA bug is caused by this. There are also no logfiles written by the daemon because of this.
- Don't change the internal directory structure of the /opt/teamviewer8 directory. The application relies on it.
- There were also some links missing (Teamviewer and Teamviewer_Desktop). Without them no incoming connection is possible.
Comment 2 Julian Ospald 2013-04-18 12:32:18 UTC
(In reply to comment #1)
> Some remarks to the current ebuild:
> 
> - Please don't rename the /opt/teamviewer8 directory to /opt/teamviewer-8.
> The path is used internally in scripts and this might lead to errors. If you
> really need to change the path, please make sure all the scripts are
> adjusted.

Which scripts? I grepped them all and it is not there.

> - Please use the wrapper scripts for the TeamViewer daemon. They not only
> start the daemon, but also create necessary directories in /opt/teamviewer8.
> The EULA bug is caused by this. There are also no logfiles written by the
> daemon because of this.

The wrapper scripts are broken for our purpose. That's the reason I had to patch out the daemon parts.
Also, apps must never create directories outside of /var. I guess I will have to do some symlink stuff.

The daemon logfile is at /tmp/TeamViewer8_Logfile.log
The client logfile is at ~/.config/teamviewer8/TeamViewer8_Logfile.log

> - Don't change the internal directory structure of the /opt/teamviewer8
> directory. The application relies on it.

Afais it is mostly configurable.
The only true case for your argument would be if directories were hardcoded in the source code.

Sadly, that seems to be true for the daemon. After fixing that, logs are created in /var/log/teamviewer8 now.

> - There were also some links missing (Teamviewer and Teamviewer_Desktop).
> Without them no incoming connection is possible.

That is severe. Will fix.
Comment 3 Julian Ospald 2013-04-18 12:54:40 UTC
btw... is teamviewer-7 dead now? Can't seem to connect even to the KeepAlive server anymore.
Comment 4 Julian Ospald 2013-04-18 13:04:18 UTC
+*teamviewer-8.0.17147-r2 (18 Apr 2013)
+
+  18 Apr 2013; Julian Ospald <hasufell@gentoo.org>
+  -teamviewer-8.0.17147-r1.ebuild, +teamviewer-8.0.17147-r2.ebuild:
+  fix missing symlinks, relocate install dest, allow correct logdir
Comment 5 Julian Ospald 2013-04-18 13:14:21 UTC
also, I don't want to switch to tar.gz

that would mean to rewrite the ebuild and randomly switch between .deb and tar.gz depending on which one got the higher version number (last time tar.gz was behind)