[systemd-devel] [PATCH 1/3] test-strv.c: added strv_split

2013-10-31 Thread Daniel Buch
--- src/test/test-strv.c | 16 1 file changed, 16 insertions(+) diff --git a/src/test/test-strv.c b/src/test/test-strv.c index f32d02e..e96200e 100644 --- a/src/test/test-strv.c +++ b/src/test/test-strv.c @@ -156,6 +156,21 @@ static void test_strv_quote_unquote(const char* const

[systemd-devel] [PATCH 2/3] test-strv.c: added strv_split_newline

2013-10-31 Thread Daniel Buch
--- src/test/test-strv.c | 16 1 file changed, 16 insertions(+) diff --git a/src/test/test-strv.c b/src/test/test-strv.c index e96200e..9b51d82 100644 --- a/src/test/test-strv.c +++ b/src/test/test-strv.c @@ -171,6 +171,21 @@ static void test_strv_split(void) { } } +s

[systemd-devel] [PATCH 3/3] test-strv.c: added test_strv_remove_prefix

2013-10-31 Thread Daniel Buch
--- src/test/test-strv.c | 17 + 1 file changed, 17 insertions(+) diff --git a/src/test/test-strv.c b/src/test/test-strv.c index 9b51d82..e468859 100644 --- a/src/test/test-strv.c +++ b/src/test/test-strv.c @@ -199,6 +199,22 @@ static void test_strv_split_nulstr(void) { a

[systemd-devel] test-strv.c

2013-10-31 Thread Daniel Buch
Hi there, By these 3 patches test_strv.c should be complete Sorry for the tap mess i pushed, wont happen again ;) ___ systemd-devel mailing list systemd-devel@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/systemd-devel

Re: [systemd-devel] [PATCH] SMACK: assign * label to /tmp when using SMACK.

2013-10-31 Thread Karel Zak
On Tue, Oct 29, 2013 at 01:27:07PM +0100, Lennart Poettering wrote: > On Tue, 29.10.13 16:02, WaLyong Cho (walyong@samsung.com) wrote: > > > >> Hmm, here's an idea: there has been a long standig feature request to > > >> add a configurable boolean to mount unit files that controls > > >> /bin/

Re: [systemd-devel] [systemd-commits] src/timedate

2013-10-31 Thread Lennart Poettering
On Wed, 30.10.13 16:39, Lennart Poettering (lenn...@poettering.net) wrote: > Wut? timedated runs after everything is mounted, so there's really no > point in allowing specification of NTP units in the rootprefix. What's > the use case here? > > I am going to revert if that's OK? Done! Lennart

Re: [systemd-devel] [PATCH] build: lookup for the "mount" binary

2013-10-31 Thread Harald Hoyer
On 10/30/2013 09:25 PM, Cristian Rodríguez wrote: > El 30/10/13 15:34, Harald Hoyer escribió: >> On 10/30/2013 07:27 PM, Cristian Rodríguez wrote: >>> El mié 30 oct 2013 15:18:48 CLST, Tom Gundersen escribió: On Wed, Oct 30, 2013 at 7:12 PM, Cristian Rodríguez wrote: > Real executabl

[systemd-devel] [PATCH] systemctl: provide better error message when start limit is exceeded.

2013-10-31 Thread Vaclav Pavlin
From: Václav Pavlín --- src/core/job.c| 12 ++-- src/core/job.h| 1 + src/systemctl/systemctl.c | 5 + 3 files changed, 16 insertions(+), 2 deletions(-) diff --git a/src/core/job.c b/src/core/job.c index e5dcef7..44b6b23 100644 --- a/src/core/job.c +++ b/sr

[systemd-devel] Print a note about reset-failed if start limit is exceeded

2013-10-31 Thread Vaclav Pavlin
Could you please check this patch, I am not sure about overall JobResult usage. I added one type of result to distinguish fail in favor of start limit from other reasons. But I am not completely sure if there could be a case where JOB_FAILED is awaited, but JOB_FAILED_LIMIT would be passed inst

[systemd-devel] [PATCH] selinux: fix selinux check for transient units

2013-10-31 Thread Vaclav Pavlin
From: Václav Pavlín --- src/core/selinux-access.c | 59 ++- src/core/selinux-access.h | 15 +--- 2 files changed, 70 insertions(+), 4 deletions(-) diff --git a/src/core/selinux-access.c b/src/core/selinux-access.c index 0a3ee18..5908a79 100644

Re: [systemd-devel] [PATCH] build: lookup for the "mount" binary

2013-10-31 Thread Cristian Rodríguez
El jue 31 oct 2013 07:43:54 CLST, Harald Hoyer escribió: On 10/30/2013 09:25 PM, Cristian Rodríguez wrote: El 30/10/13 15:34, Harald Hoyer escribió: On 10/30/2013 07:27 PM, Cristian Rodríguez wrote: El mié 30 oct 2013 15:18:48 CLST, Tom Gundersen escribió: On Wed, Oct 30, 2013 at 7:12 PM, Cri

[systemd-devel] [PATCH] Make hibernation test work for swap files

2013-10-31 Thread Jan Janssen
Suspend to disk works for swap files too (even if it is located on an ecrypted file system): https://www.kernel.org/doc/Documentation/power/swsusp-and-swap-files.txt --- src/shared/sleep-config.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/shared/sleep-config.c b/src/sh

[systemd-devel] [PATCH] zsh-completion: add missing options for systemd-run

2013-10-31 Thread Ronny Chevalier
--- shell-completion/zsh/_systemd-run | 2 ++ 1 file changed, 2 insertions(+) diff --git a/shell-completion/zsh/_systemd-run b/shell-completion/zsh/_systemd-run index 0c81c54..d550f22 100644 --- a/shell-completion/zsh/_systemd-run +++ b/shell-completion/zsh/_systemd-run @@ -22,6 +22,8 @@ _argume

[systemd-devel] [PATCH] fix compiler warnings

2013-10-31 Thread Ronny Chevalier
multiple warnings like src/socket-proxy/socket-proxyd.c: In function ‘transfer_data_cb’: src/socket-proxy/socket-proxyd.c:237:25: warning: format ‘%ld’ expects argument of type ‘long int’, but argument 6 has type ‘size_t’ [-Wformat=] log_debug("Buffer now has %ld bytes fu

[systemd-devel] [wgiokas@wst420: Re: [PATCH] zsh-completion: add missing options for systemd-run]

2013-10-31 Thread William Giokas
Aand I hit the wrong button. Here it is to the list. - Forwarded message from William Giokas - On Thu, Oct 31, 2013 at 07:24:28PM +0100, Ronny Chevalier wrote: > --- > shell-completion/zsh/_systemd-run | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/shell-completion/zsh/_

Re: [systemd-devel] [PATCH] SMACK: assign * label to /tmp when using SMACK.

2013-10-31 Thread Karel Zak
On Tue, Oct 29, 2013 at 01:27:07PM +0100, Lennart Poettering wrote: > On Tue, 29.10.13 16:02, WaLyong Cho (walyong@samsung.com) wrote: > > > >> Hmm, here's an idea: there has been a long standig feature request to > > >> add a configurable boolean to mount unit files that controls > > >> /bin/

[systemd-devel] [PATCH v2] zsh-completion: add missing options for systemd-run

2013-10-31 Thread Ronny Chevalier
--- shell-completion/zsh/_systemd-run | 17 + 1 file changed, 17 insertions(+) diff --git a/shell-completion/zsh/_systemd-run b/shell-completion/zsh/_systemd-run index 0c81c54..4bfbd19 100644 --- a/shell-completion/zsh/_systemd-run +++ b/shell-completion/zsh/_systemd-run @@ -18,1

Re: [systemd-devel] [PATCH] SMACK: assign * label to /tmp when using SMACK.

2013-10-31 Thread Kok, Auke-jan H
On Thu, Oct 31, 2013 at 12:00 PM, Karel Zak wrote: > On Tue, Oct 29, 2013 at 01:27:07PM +0100, Lennart Poettering wrote: >> On Tue, 29.10.13 16:02, WaLyong Cho (walyong@samsung.com) wrote: >> >> > >> Hmm, here's an idea: there has been a long standig feature request to >> > >> add a configurab

Re: [systemd-devel] [PATCH v2] zsh-completion: add missing options for systemd-run

2013-10-31 Thread William Giokas
On Thu, Oct 31, 2013 at 08:46:29PM +0100, Ronny Chevalier wrote: > --- > shell-completion/zsh/_systemd-run | 17 + > 1 file changed, 17 insertions(+) > > diff --git a/shell-completion/zsh/_systemd-run > b/shell-completion/zsh/_systemd-run > index 0c81c54..4bfbd19 100644 > --- a/s

Re: [systemd-devel] [PATCH] fix compiler warnings

2013-10-31 Thread David Strauss
Thank. I'll commit this once I'm at my desk. On Oct 31, 2013 11:26 AM, "Ronny Chevalier" wrote: > multiple warnings like > > src/socket-proxy/socket-proxyd.c: In function ‘transfer_data_cb’: > src/socket-proxy/socket-proxyd.c:237:25: warning: format ‘%ld’ expects > argument of type ‘long int’, bu

Re: [systemd-devel] [PATCH v2] zsh-completion: add missing options for systemd-run

2013-10-31 Thread Ronny Chevalier
2013/10/31 William Giokas <1007...@gmail.com>: > On Thu, Oct 31, 2013 at 08:46:29PM +0100, Ronny Chevalier wrote: >> --- >> shell-completion/zsh/_systemd-run | 17 + >> 1 file changed, 17 insertions(+) >> >> diff --git a/shell-completion/zsh/_systemd-run >> b/shell-completion/zsh/

Re: [systemd-devel] [PATCH v2] zsh-completion: add missing options for systemd-run

2013-10-31 Thread William Giokas
On Thu, Oct 31, 2013 at 09:54:05PM +0100, Ronny Chevalier wrote: > 2013/10/31 William Giokas <1007...@gmail.com>: > > Much better. Now one thing that would make this amazing is if you could > > make __machines an autoloaded file, possibly _sd_machines, that can be > > used in multiple zsh completio

Re: [systemd-devel] [PATCH] fix compiler warnings

2013-10-31 Thread David Strauss
Committed. Did you happen to be on 32-bit? I didn't see any warnings on my own builds (64-bit Fedora 19). ___ systemd-devel mailing list systemd-devel@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/systemd-devel

Re: [systemd-devel] [PATCH] fix compiler warnings

2013-10-31 Thread Ronny Chevalier
2013/10/31 David Strauss : > Committed. Did you happen to be on 32-bit? I didn't see any warnings > on my own builds (64-bit Fedora 19). Yes, 32 bit Archlinux ___ systemd-devel mailing list systemd-devel@lists.freedesktop.org http://lists.freedesktop.org/

Re: [systemd-devel] [PATCH] fix compiler warnings

2013-10-31 Thread David Strauss
I did a quick follow-up to make some of them %zu where they're actually unsigned. Can you let me know if that still works for you? ___ systemd-devel mailing list systemd-devel@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/systemd-dev

Re: [systemd-devel] [PATCH] fix compiler warnings

2013-10-31 Thread Ronny Chevalier
Oh sorry my bad, I missed the fact that some of them were unsigned. And yes it works. 2013/10/31 David Strauss : > I did a quick follow-up to make some of them %zu where they're > actually unsigned. Can you let me know if that still works for you? ___ sy

[systemd-devel] [PATCH] sd-login: Fix typo

2013-10-31 Thread Bastien Nocera
s/sessio,/session,/ --- src/systemd/sd-login.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/systemd/sd-login.h b/src/systemd/sd-login.h index c5837f0..961bd88 100644 --- a/src/systemd/sd-login.h +++ b/src/systemd/sd-login.h @@ -101,7 +101,7 @@ int sd_session_is_active(c