[systemd-devel] [PATCH 3/8] power: refactor the three power management binaries to src/power

Zbigniew Jędrzejewski-Szmek zbyszek at in.waw.pl
Thu Feb 26 18:26:12 PST 2015


I'm not sure we want this... Can you add some justification? Do they share
code?

On Fri, Feb 20, 2015 at 02:31:00PM -0800, Shawn Landden wrote:
> ---
>  Makefile.am               |   6 +-
>  src/core/shutdown.c       | 420 -----------------------------------------
>  src/power/Makefile        |  28 +++
This should be a symlink.

>  src/power/shutdown.c      | 420 +++++++++++++++++++++++++++++++++++++++++
>  src/power/shutdownd.c     | 461 ++++++++++++++++++++++++++++++++++++++++++++++
>  src/power/sleep.c         | 219 ++++++++++++++++++++++
>  src/shutdownd/Makefile    |   1 -
>  src/shutdownd/shutdownd.c | 461 ----------------------------------------------
>  src/sleep/Makefile        |   1 -
>  src/sleep/sleep.c         | 219 ----------------------
>  10 files changed, 1131 insertions(+), 1105 deletions(-)
>  delete mode 100644 src/core/shutdown.c
>  create mode 100644 src/power/Makefile
>  create mode 100644 src/power/shutdown.c
>  create mode 100644 src/power/shutdownd.c
>  create mode 100644 src/power/sleep.c
>  delete mode 120000 src/shutdownd/Makefile
>  delete mode 100644 src/shutdownd/shutdownd.c
>  delete mode 120000 src/sleep/Makefile
>  delete mode 100644 src/sleep/sleep.c
It's better to use -M for such patches... Make it easier to see what is
hapenning.

Zbyszek


More information about the systemd-devel mailing list