When I'm trying tl-notify -u on eudemo I get: tl-notify -u astrand "Test" error: user 'astrand' has no sessions But I have a session: astrand thinlinc/3 2014-05-20 16:33 (:3)
tl-notify is crawling /proc/x/cmdline searching for matching tl-run-profile to get the display. This has been changed when tl-run-profile become a python script which uses execv. We should change this into something more proper way to enum local session of an agent.
For now there is no way to get the DBUS_SESSION_BUS_ADDRESS for a session, before we had a known process which we harvested the environment from. We need to handle this and the appropriated way seems to store the session bus address in thinlinc user session directory when xstartup.d dbus launch script executes.
The approach i used; xmlrpc function get_sessions() is not usable due to it requires a privileged query and each agent needs to be explicitly defined in /vsm/allowed_clients to be used within a cluster. It does work on a single setup master/agent.
tl-notify rewritten in vommit 29061 and tl-dbus-launch.sh in commit 29062.
The approach should slightly be changed: - Dump full environment to $(TLSESSIONDATA)/xstartup.env due to there are others who needs environment information and not only the dbus session address. - Make tl-notify read DBUS_SESSION_BUS_ADDRESS from env. file dump
(In reply to comment #5) > - Dump full environment to $(TLSESSIONDATA)/xstartup.env due to there are > others who needs environment information and not only the dbus session > address. Commit 29072.
(In reply to comment #5) > The approach should slightly be changed: > > - Dump full environment to $(TLSESSIONDATA)/xstartup.env due to there are > others who needs environment information and not only the dbus session > address. > > - Make tl-notify read DBUS_SESSION_BUS_ADDRESS from env. file dump Commit 29073 reverts commit 29062. Commit 29074 makes tl-notify to read the new environment file.
(In reply to comment #7) > (In reply to comment #5) > > The approach should slightly be changed: > > > > - Dump full environment to $(TLSESSIONDATA)/xstartup.env due to there are > > others who needs environment information and not only the dbus session > > address. > > > > - Make tl-notify read DBUS_SESSION_BUS_ADDRESS from env. file dump > > Commit 29073 reverts commit 29062. > > Commit 29074 makes tl-notify to read the new environment file. Commit 29075 fixes a bug and cleanups env. key/value parsing.
From https://www.cendio.com/bugzilla/show_bug.cgi?id=5265#c3: However tl-notify have double meaning error reporting when xstartup.env is removed from session directory; /opt/thinlinc/sbin/tl-notify hello error: failed to read xstartup.env ([Errno 2] No such file or directory: '/var/opt/thinlinc/sessions/cendio/1/xstartup.env') Message sent to 1 sessions on this agent.
(In reply to comment #9) > From https://www.cendio.com/bugzilla/show_bug.cgi?id=5265#c3: > > However tl-notify have double meaning error reporting when xstartup.env is > removed from session directory; > > /opt/thinlinc/sbin/tl-notify hello > error: failed to read xstartup.env ([Errno 2] No such file or directory: > '/var/opt/thinlinc/sessions/cendio/1/xstartup.env') > Message sent to 1 sessions on this agent. Fixed in commit 29427.
Build 4515 works fine on SLED11.
