diff mbox

[v2,1/2] acpid: replace "echo -e" with printf

Message ID 1400660710-15758-1-git-send-email-maxime.hadjinlian@gmail.com
State Accepted
Commit 6b9bc5fe407c519b048faea22468d59cadec49a1
Headers show

Commit Message

Maxime Hadjinlian May 21, 2014, 8:25 a.m. UTC
printf is POSIX-compliant, echo -e is not.

Signed-off-by: Maxime Hadjinlian <maxime.hadjinlian@gmail.com>
Cc: Thomas De Schampheleire <patrickdepinguin@gmail.com>
---
Changes v1 -> v2:
    - Added trailing \n (Peter Korsgaard)
---
 package/acpid/acpid.mk | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Peter Korsgaard May 21, 2014, 8:42 a.m. UTC | #1
>>>>> "Maxime" == Maxime Hadjinlian <maxime.hadjinlian@gmail.com> writes:

 > printf is POSIX-compliant, echo -e is not.
 > Signed-off-by: Maxime Hadjinlian <maxime.hadjinlian@gmail.com>
 > Cc: Thomas De Schampheleire <patrickdepinguin@gmail.com>
 > ---
 > Changes v1 -> v2:
 >     - Added trailing \n (Peter Korsgaard)

Committed, thanks.
diff mbox

Patch

diff --git a/package/acpid/acpid.mk b/package/acpid/acpid.mk
index 1f4ec31..7a6a478 100644
--- a/package/acpid/acpid.mk
+++ b/package/acpid/acpid.mk
@@ -17,7 +17,7 @@  endef
 
 define ACPID_SET_EVENTS
 	mkdir -p $(TARGET_DIR)/etc/acpi/events
-	/bin/echo -e "event=button[ /]power\naction=/sbin/poweroff" \
+	printf "event=button[ /]power\naction=/sbin/poweroff\n" \
 		>$(TARGET_DIR)/etc/acpi/events/powerbtn
 endef