On Sun, 18.05.14 19:36, Przemek Rudy (pru...@o2.pl) wrote:

> diff --git a/src/core/automount.c b/src/core/automount.c
> index 65e6d6f..d4359b9 100644
> --- a/src/core/automount.c
> +++ b/src/core/automount.c
> @@ -124,7 +124,7 @@ static int automount_add_mount_links(Automount *a) {
>          if (r < 0)
>                  return r;
>  
> -        return unit_require_mounts_for(UNIT(a), parent);
> +        return unit_needs_mounts_for(UNIT(a), parent, true);
>  }
>  

Please make the the last parameter one of type UnitDependency, so that
one can either pass UNIT_WANTS or UNIT_REQUIRES there. THat makes the
calls much more descriptive, as well as more generic.

Otherwise looks pretty good!

Lennart

-- 
Lennart Poettering, Red Hat
_______________________________________________
systemd-devel mailing list
systemd-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/systemd-devel

Reply via email to