<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Thu, Feb 26, 2015 at 6:26 PM, Zbigniew Jędrzejewski-Szmek <span dir="ltr"><<a href="mailto:zbyszek@in.waw.pl" target="_blank">zbyszek@in.waw.pl</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I'm not sure we want this... Can you add some justification? Do they share<br>
code?<br></blockquote><div>I found it confusing what parts each of these handled, and some code sharing is possible, but none is shared currently. This also eliminates one directory. All of these are handled by units:</div><div><br></div><div>sleep.target</div><div>shutdown.target</div><div>etc...</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span class=""><br>
On Fri, Feb 20, 2015 at 02:31:00PM -0800, Shawn Landden wrote:<br>
> ---<br>
>  Makefile.am               |   6 +-<br>
>  src/core/shutdown.c       | 420 -----------------------------------------<br>
>  src/power/Makefile        |  28 +++<br>
</span>This should be a symlink.<br>
<span class=""><br></span></blockquote><div>To make all of these symlinks would be a much larger patch, but I can send such a patch </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">
>  src/power/shutdown.c      | 420 +++++++++++++++++++++++++++++++++++++++++<br>
>  src/power/shutdownd.c     | 461 ++++++++++++++++++++++++++++++++++++++++++++++<br>
>  src/power/sleep.c         | 219 ++++++++++++++++++++++<br>
>  src/shutdownd/Makefile    |   1 -<br>
>  src/shutdownd/shutdownd.c | 461 ----------------------------------------------<br>
>  src/sleep/Makefile        |   1 -<br>
>  src/sleep/sleep.c         | 219 ----------------------<br>
>  10 files changed, 1131 insertions(+), 1105 deletions(-)<br>
>  delete mode 100644 src/core/shutdown.c<br>
>  create mode 100644 src/power/Makefile<br>
>  create mode 100644 src/power/shutdown.c<br>
>  create mode 100644 src/power/shutdownd.c<br>
>  create mode 100644 src/power/sleep.c<br>
>  delete mode 120000 src/shutdownd/Makefile<br>
>  delete mode 100644 src/shutdownd/shutdownd.c<br>
>  delete mode 120000 src/sleep/Makefile<br>
>  delete mode 100644 src/sleep/sleep.c<br>
</span>It's better to use -M for such patches... Make it easier to see what is<br>
hapenning.<br>
<br>
Zbyszek<br>
_______________________________________________<br>
systemd-devel mailing list<br>
<a href="mailto:systemd-devel@lists.freedesktop.org">systemd-devel@lists.freedesktop.org</a><br>
<a href="http://lists.freedesktop.org/mailman/listinfo/systemd-devel" target="_blank">http://lists.freedesktop.org/mailman/listinfo/systemd-devel</a><br>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature">Shawn Landden</div>
</div></div>