[systemd-devel] [RFC] load-fragment: use unquote_first_word in config_parse_exec

Andrei Borzenkov arvidjaar at gmail.com
Wed Jun 3 20:45:45 PDT 2015


В Wed, 3 Jun 2015 20:34:53 -0700
Filipe Brandenburger <filbranden at google.com> пишет:

> 
> >> - We do something different for empty string (clear the command list)
> >>   than we do for just whitespace. This is pre-existing. Maybe we need to
> >>   fix that? I put a comment on that case, that branch is triggered both
> >>   in the "just whitespace" case as well as right after a semicolon
> >>   command separator.
> >
> > Not sure I understand what you mean. Do you suggest that
> >
> > ExecStart=/usr/bin/foo ; ; /usr/bin/bar
> >
> > should discard /usr/bin/foo? Or that it does it already?
> 
> I added a test case to cover two semicolons in a row. With the new
> code it breaks as it thinks ";" is a new command name, I think that's
> appropriate enough, let me know if you think otherwise.
> 

I cannot think where this would be useful, so I guess it is OK as long
as error is clear enough.


More information about the systemd-devel mailing list