[systemd-devel] [PATCH] sysv-generator: handle Provides: for non-virtual facility names

2015-01-05 Thread Michael Biebl
See attached patch. The commit message reads: The list of provided facility names as specified via Provides: in the LSB header was originally implemented by adding those facilities to the Names= property via unit_add_name(). In commit 95ed3294c632f5606327149f10cef1eb34422862 the internal SysV

Re: [systemd-devel] rsync output is not captured in the journal

2015-01-05 Thread Igor Bukanov
It could be that this is SELinux-related. If I copy the rsync binary to a new location and change the test service unit to use it, then the log appeared as it should. However, I have no indications of any rsync-related issues in any SELinux logs. Another thing is that if I run under strace the

[systemd-devel] [PATCH] libsystemd-network: fix typo in lldp

2015-01-05 Thread Torstein Husebø
--- src/libsystemd-network/lldp-internal.h | 2 +- src/libsystemd-network/sd-lldp.c | 12 ++-- 2 files changed, 7 insertions(+), 7 deletions(-) diff --git a/src/libsystemd-network/lldp-internal.h b/src/libsystemd-network/lldp-internal.h index c7235c11ee..8e09ee8f3a 100644 ---

[systemd-devel] logind: support of shared devices

2015-01-05 Thread Oleg Samarin
Hello! The default systemd behavior is to grant/revoke access on devices attached to a seat when the user activates/deactivates a session on the seat. But sometimes it needs the user has an access to some device when he/she activates a session with ANY seat. For example, I have two seats:

Re: [systemd-devel] rsync output is not captured in the journal

2015-01-05 Thread Igor Bukanov
On 5 January 2015 at 12:21, Mantas Mikulėnas graw...@gmail.com wrote: Run it under `strace -D` – the problem may be related to rsync's parent being pid1. Under strace -D the problem disappeared as well. The problem also disappears if I run rsync like in: ExecStart=/usr/bin/bash -c 'exec

Re: [systemd-devel] [PATCH] libsystemd-network: fix typo in lldp

2015-01-05 Thread Lennart Poettering
On Mon, 05.01.15 09:55, Torstein Husebø (torst...@huseboe.net) wrote: Applied! Thanks! --- src/libsystemd-network/lldp-internal.h | 2 +- src/libsystemd-network/sd-lldp.c | 12 ++-- 2 files changed, 7 insertions(+), 7 deletions(-) diff --git

Re: [systemd-devel] [RFC PATCH] journal: pass uid.gid in the stream header

2015-01-05 Thread Zbigniew Jędrzejewski-Szmek
On Mon, Jan 05, 2015 at 02:12:45PM +0100, Lennart Poettering wrote: On Thu, 01.01.15 04:40, Zbigniew Jędrzejewski-Szmek (zbys...@in.waw.pl) wrote: Sounds generally OK. A disadvantage of the solution implemented here, otoh, is that both systemd and journald must be restarted for it to

Re: [systemd-devel] rsync output is not captured in the journal

2015-01-05 Thread Igor Bukanov
I attach strace log for systemd. I have in my test unit: # cat test.service [Service] ExecStart=/usr/bin/bash -c 'echo rsync copy' ExecStart=/var/rsync -vv /etc/hostname /tmp/h6 ExecStart=/usr/bin/bash -c 'echo system rsync' ExecStart=/usr/bin/rsync -vv /etc/hostname /tmp/h2

Re: [systemd-devel] [PATCH v4] Do not clear parent mount flags when setting up namespaces

2015-01-05 Thread Lennart Poettering
On Sun, 04.01.15 20:51, Topi Miettinen (toiwo...@gmail.com) wrote: When setting up a namespace, mount flags like noexec, nosuid and nodev are cleared, so the mounts always have exec, suid and dev flags enabled. Copy source directory mount flags to target mount when remounting the bind

Re: [systemd-devel] is a unix shell (zsh, bash, sh) still required by systemd?

2015-01-05 Thread Lennart Poettering
On Wed, 31.12.14 00:16, ivo welch (ivo.we...@gmail.com) wrote: [2] on http://freedesktop.org/wiki/Software/systemd/, this list is noted as appropriate not only for for general development but for technical discussion, too. alas, because this is not about developing systemd, but developing

Re: [systemd-devel] rsync output is not captured in the journal

2015-01-05 Thread Lennart Poettering
On Sun, 04.01.15 21:06, Igor Bukanov (i...@mir2.org) wrote: Hello, I have a strange case when there is no output in the journal from rsync when the command is run from a service. This is on Fedora 21 with systemd 217. Consider the following trivial test unit: # cat

[systemd-devel] [PATCH] bus-proxyd: don't allow to acquire org.freedesktop.DBus name

2015-01-05 Thread Lukasz Skalski
--- src/bus-proxyd/bus-proxyd.c | 4 1 file changed, 4 insertions(+) diff --git a/src/bus-proxyd/bus-proxyd.c b/src/bus-proxyd/bus-proxyd.c index a7818f5..fc70cce 100644 --- a/src/bus-proxyd/bus-proxyd.c +++ b/src/bus-proxyd/bus-proxyd.c @@ -829,6 +829,10 @@ static int process_driver(sd_bus

Re: [systemd-devel] [PATCH] po: update Italian translation

2015-01-05 Thread Lennart Poettering
On Mon, 05.01.15 10:35, Daniele Medri (dme...@gmail.com) wrote: Applied! Thanks! --- po/it.po | 207 +++ 1 file changed, 127 insertions(+), 80 deletions(-) diff --git a/po/it.po b/po/it.po index 90a5fe0..93a1e79 100644 ---

Re: [systemd-devel] [PATCH] sysv-generator: handle Provides: for non-virtual facility names

2015-01-05 Thread Zbigniew Jędrzejewski-Szmek
On Mon, Jan 05, 2015 at 10:30:11AM +0100, Michael Biebl wrote: See attached patch. The commit message reads: The list of provided facility names as specified via Provides: in the LSB header was originally implemented by adding those facilities to the Names= property via unit_add_name().

Re: [systemd-devel] [RFC PATCH] journal: pass uid.gid in the stream header

2015-01-05 Thread Lennart Poettering
On Thu, 01.01.15 04:40, Zbigniew Jędrzejewski-Szmek (zbys...@in.waw.pl) wrote: Sounds generally OK. A disadvantage of the solution implemented here, otoh, is that both systemd and journald must be restarted for it to take effect. This is something I am concerned about. This will break

Re: [systemd-devel] [RFC PATCH] journal: pass uid.gid in the stream header

2015-01-05 Thread Lennart Poettering
On Mon, 05.01.15 14:55, Zbigniew Jędrzejewski-Szmek (zbys...@in.waw.pl) wrote: On Mon, Jan 05, 2015 at 02:12:45PM +0100, Lennart Poettering wrote: On Thu, 01.01.15 04:40, Zbigniew Jędrzejewski-Szmek (zbys...@in.waw.pl) wrote: Sounds generally OK. A disadvantage of the solution

Re: [systemd-devel] [RFC PATCH] journal: pass uid.gid in the stream header

2015-01-05 Thread Lennart Poettering
On Mon, 05.01.15 15:02, Zbigniew Jędrzejewski-Szmek (zbys...@in.waw.pl) wrote: What we can do instead is to implement daemon-reexec equivalent for journald. It would simply reexec itself to a new binary and pass all the fds. Some serialization/de-serialization protocol would be necessary to

Re: [systemd-devel] systemd: Program terminated with signal SIGFPE, Arithmetic exception.

2015-01-05 Thread Lennart Poettering
On Tue, 30.12.14 16:51, Paul Menzel (paulepan...@users.sourceforge.net) wrote: Dear systemd folks, using Debian Sid/unstable with systemd 215-8, I still have problems that there are time-outs between systemd and D-Bus, even with the current D-Bus 1.8.12 [1]. Experiencing this again, where

Re: [systemd-devel] Delegate=yes and user instance's resource controlling

2015-01-05 Thread Lennart Poettering
On Thu, 25.12.14 23:38, Ivan Shapovalov (intelfx...@gmail.com) wrote: Hi! Judging from commit a931ad47a8623163a29d898224d8a8c1177ffdaf, the systemd user instance is intentionally restricted from touching !systemd hierarchies. So, things like systemctl --user set-property foo.service

Re: [systemd-devel] systemd: Program terminated with signal SIGFPE, Arithmetic exception.

2015-01-05 Thread Paul Menzel
Am Montag, den 05.01.2015, 17:57 +0100 schrieb Lennart Poettering: On Tue, 30.12.14 16:51, Paul Menzel (paulepan...@users.sourceforge.net) wrote: using Debian Sid/unstable with systemd 215-8, I still have problems that there are time-outs between systemd and D-Bus, even with the current

Re: [systemd-devel] [PATCH] sysv-generator: handle Provides: for non-virtual facility names

2015-01-05 Thread Michael Biebl
Hi, 2015-01-05 14:35 GMT+01:00 Zbigniew Jędrzejewski-Szmek zbys...@in.waw.pl: On Mon, Jan 05, 2015 at 10:30:11AM +0100, Michael Biebl wrote: +link = strjoin(arg_dest, /, alias, NULL); +if (!link) +return log_oom(); path_join()? I borrowed from the existing

Re: [systemd-devel] [PATCH] sysv-generator: handle Provides: for non-virtual facility names

2015-01-05 Thread Zbigniew Jędrzejewski-Szmek
On Tue, Jan 06, 2015 at 12:01:35AM +0100, Michael Biebl wrote: Hi, 2015-01-05 14:35 GMT+01:00 Zbigniew Jędrzejewski-Szmek zbys...@in.waw.pl: On Mon, Jan 05, 2015 at 10:30:11AM +0100, Michael Biebl wrote: +link = strjoin(arg_dest, /, alias, NULL); +if (!link) +

Re: [systemd-devel] [PATCH] Clarify path escaping and reference systemd-escape

2015-01-05 Thread Zbigniew Jędrzejewski-Szmek
On Thu, Jan 01, 2015 at 04:35:34PM -0500, Chris Atkinson wrote: See Bug 87688 (https://bugs.freedesktop.org/show_bug.cgi?id=87688). This patch adds more detail to the description of how path escaping operates and provides a pointer to the systemd-escape program. Either would serve to answer

Re: [systemd-devel] Relative links in tmpfiles.d/etc.conf

2015-01-05 Thread Zbigniew Jędrzejewski-Szmek
On Sat, Jan 03, 2015 at 12:07:59PM +0100, Umut Tezduyar Lindskog wrote: Hi, Our /etc is a sym link and due to that all the links created by tmpfiles.d/etc.conf are wrong. Is there a reason why the links are relative? They are relative so that if you're looking into a container (or a system

Re: [systemd-devel] [PATCH] bus-proxyd: don't allow to acquire org.freedesktop.DBus name

2015-01-05 Thread Daniel Mack
On 01/05/2015 02:45 PM, Lukasz Skalski wrote: --- src/bus-proxyd/bus-proxyd.c | 4 1 file changed, 4 insertions(+) Applied, thanks! diff --git a/src/bus-proxyd/bus-proxyd.c b/src/bus-proxyd/bus-proxyd.c index a7818f5..fc70cce 100644 --- a/src/bus-proxyd/bus-proxyd.c +++

[systemd-devel] [PATCH] bus-proxyd: overwrite 'sender' field in process_driver() function

2015-01-05 Thread Lukasz Skalski
To keep compatibility with dbus-1, before we use synthetic_reply_method_*() functions, we should overwrite 'sender' field - otherwise functions sd_bus_message_new_method_return() and sd_bus_message_new_method_error() never set 'destination' field. Thanks to this we have proper 'destination'

Re: [systemd-devel] [PATCH 2/4] systemctl: edit: improve error messages, report actual error for units which could not be loaded

2015-01-05 Thread Zbigniew Jędrzejewski-Szmek
This is a minor thing, but makes things easier, especially when using gitk or git format-patch: please keep the patch subject at around 80 characters (=72 is best). I know we don't always to this, sometimes it just isn't possible, but in this case something like 'systemctl: report actual load

Re: [systemd-devel] systemctl action; tmpfiles 'D'

2015-01-05 Thread Lennart Poettering
On Mon, 05.01.15 00:33, Elias Probst (m...@eliasprobst.eu) wrote: systemd-tmpfiles --create fails with chmod(/var/run/samba) failed: No such file or directory after d7b8eec7 https://bugs.freedesktop.org/show_bug.cgi?id=87953 Fixed this one, too! Thanks! Lennart -- Lennart Poettering,

Re: [systemd-devel] [PATCH 3/4] systemctl: edit: further reword error messages

2015-01-05 Thread Zbigniew Jędrzejewski-Szmek
On Fri, Dec 19, 2014 at 05:08:09PM +0300, Ivan Shapovalov wrote: --- src/systemctl/systemctl.c | 22 +++--- 1 file changed, 11 insertions(+), 11 deletions(-) diff --git a/src/systemctl/systemctl.c b/src/systemctl/systemctl.c index 658793e..20c367c 100644 ---