Package Details: apt-dater-legacy 0.9.0-3

Git Clone URL: https://aur.archlinux.org/apt-dater-legacy.git (read-only, click to copy)
Package Base: apt-dater-legacy
Description: Terminal-based remote package update manager, legacy version with original text based configuration
Upstream URL: https://www.ibh.de/apt-dater
Licenses: GPL
Conflicts: apt-dater
Submitter: dundee
Maintainer: dundee
Last Packager: dundee
Votes: 2
Popularity: 0.000000
First Submitted: 2015-12-30 22:53 (UTC)
Last Updated: 2016-08-26 08:32 (UTC)

Latest Comments

dundee commented on 2016-08-26 08:33 (UTC)

Ok, forced -O0, thanks!

marco44 commented on 2016-08-16 07:36 (UTC) (edited on 2016-08-16 07:36 (UTC) by marco44)

I have found it, I think. It's a compilation problem (I bet gcc 6 or 6.1 trying to be smart): const static gchar * screen_get_sdir() { static gchar sdir[256]; if (!pw) pw = getpwuid(getuid()); if (!pw) return NULL; g_snprintf(sdir, sizeof(sdir), SCREEN_SDFORMT, SCREEN_SOCKPATH, pw->pw_name); return sdir; } gboolean screen_get_sessions(HostNode *n) { if (n->screens) { g_list_free(n->screens); n->screens = NULL; } const gchar *sdir = screen_get_sdir(); The static sdir in screen_get_sdir doesn't get deallocated at the end of the function, because of the static. sdir is not defined in screen_get_sessions after the function call when compiled with -O2 (gdb says it has been optimized out). With -O0, it works. As performance is not that crucial, and this version of apt-dater is probably not maintained anymore, I think forcing -O0 is sufficient.

dundee commented on 2016-08-09 09:57 (UTC)

cousinm: No idea. Upgrading a group of hosts and connecting to their sessions works ok for me.

marco44 commented on 2016-08-09 09:16 (UTC)

I'm having a very weird problem, I cannot attach detached sessions… For instance if I run an upgrade on a group of hosts, I cannot use n to attach to their session. apt-dater doesn't even list sessions… Do you have this problem too ? It used to work a while ago (I'd say one or two months ago)

dundee commented on 2016-08-09 08:22 (UTC)

cousinm: Yes, is required for connecting to hosts. Added, thanks!

marco44 commented on 2016-07-25 09:00 (UTC)

Shouldn't it depend on screen ?

marco44 commented on 2016-07-25 08:00 (UTC)

Hey, thanks !