All of lore.kernel.org
 help / color / mirror / Atom feed
From: Armin Kuster <akuster808@gmail.com>
To: openembedded-devel@lists.openembedded.org
Subject: [kirkstone 17/22] s-nail: Set VAL_MTA
Date: Sun, 15 May 2022 06:49:17 -0700	[thread overview]
Message-ID: <eb37ec6ba87d78338c66ea1ec5f76a54891000ec.1652622481.git.akuster808@gmail.com> (raw)
In-Reply-To: <cover.1652622481.git.akuster808@gmail.com>

From: Mingli Yu <mingli.yu@windriver.com>

Some build server may provide sendmail like /usr/lib/sendmail which
may result the VAL_MTA to be /usr/lib/sendmail during do_configure,
but the sendmail is actually /usr/sbin/sendmail on the target.

Fixes:
  # echo "invalide address email" | s-nail -s "test mail" -r test@test.com tester@test.com
  s-nail: Cannot start /usr/lib/sendmail: executable not found (adjust *mta* variable)
  /home/root/dead.letter 10/234
  s-nail: ... message not sent

  # which sendmail
  /usr/sbin/sendmail

Signed-off-by: Mingli Yu <mingli.yu@windriver.com>
Signed-off-by: Khem Raj <raj.khem@gmail.com>
(cherry picked from commit f334a009472136b266c4f72612b549846df20c0e)
Signed-off-by: Armin Kuster <akuster808@gmail.com>
---
 .../files/0001-make.rc-set-VAL_MTA.patch      | 48 +++++++++++++++++++
 .../recipes-extended/s-nail/s-nail_14.9.24.bb |  3 +-
 2 files changed, 50 insertions(+), 1 deletion(-)
 create mode 100644 meta-oe/recipes-extended/s-nail/files/0001-make.rc-set-VAL_MTA.patch

diff --git a/meta-oe/recipes-extended/s-nail/files/0001-make.rc-set-VAL_MTA.patch b/meta-oe/recipes-extended/s-nail/files/0001-make.rc-set-VAL_MTA.patch
new file mode 100644
index 0000000000..f4decda141
--- /dev/null
+++ b/meta-oe/recipes-extended/s-nail/files/0001-make.rc-set-VAL_MTA.patch
@@ -0,0 +1,48 @@
+From da8c7a027390de7828d4272a3c8bcdbffbac25ad Mon Sep 17 00:00:00 2001
+From: Mingli Yu <mingli.yu@windriver.com>
+Date: Thu, 5 May 2022 16:57:48 +0800
+Subject: [PATCH] make.rc: set VAL_MTA
+
+Some build server may provide sendmail like /usr/lib/sendmail which
+may result the VAL_MTA to be /usr/lib/sendmail, but the sendmail
+is actually /usr/sbin/sendmail on the target.
+
+Fixes:
+  # echo "invalide address email" | s-nail -s "test mail" -r test@test.com tester@test.com
+  s-nail: Cannot start /usr/lib/sendmail: executable not found (adjust *mta* variable)
+  /home/root/dead.letter 10/234
+  s-nail: ... message not sent
+
+  # which sendmail
+  /usr/sbin/sendmail
+
+Upstream-Status: Inappropriate[oe-specific]
+
+Signed-off-by: Mingli Yu <mingli.yu@windriver.com>
+---
+ make.rc | 9 +--------
+ 1 file changed, 1 insertion(+), 8 deletions(-)
+
+diff --git a/make.rc b/make.rc
+index 98bd968..45ac0d4 100644
+--- a/make.rc
++++ b/make.rc
+@@ -96,14 +96,7 @@ VAL_MAIL=`\
+ 
+ # Path to the local MTA (Mail-Transfer-Agent).
+ # MTA aliases (aliases(5)) are optionally supported via OPT_MTA_ALIASES.
+-VAL_MTA=`\
+-   if [ -x /usr/bin/sendmail ]; then \
+-      echo /usr/bin/sendmail;\
+-   elif [ -x /usr/lib/sendmail ]; then \
+-      echo /usr/lib/sendmail;\
+-   else \
+-      echo /usr/sbin/sendmail;\
+-   fi`
++VAL_MTA=/usr/sbin/sendmail
+ 
+ # Today a lot of systems no longer use sendmail(1), but a different MTA.
+ # To ensure compatibility with sendmail(1), a system called
+-- 
+2.25.1
+
diff --git a/meta-oe/recipes-extended/s-nail/s-nail_14.9.24.bb b/meta-oe/recipes-extended/s-nail/s-nail_14.9.24.bb
index bc514b577a..539f9c3c82 100644
--- a/meta-oe/recipes-extended/s-nail/s-nail_14.9.24.bb
+++ b/meta-oe/recipes-extended/s-nail/s-nail_14.9.24.bb
@@ -6,7 +6,8 @@ LICENSE = "ISC & BSD-3-Clause & BSD-4-Clause"
 LIC_FILES_CHKSUM = "file://COPYING;md5=87266591c81260f10221f1f38872d023"
 
 SRC_URI = "https://ftp.sdaoden.eu/${BP}.tar.xz \
-           file://make-errors.patch"
+           file://make-errors.patch \
+           file://0001-make.rc-set-VAL_MTA.patch"
 SRC_URI[sha256sum] = "2714d6b8fb2af3b363fc7c79b76d058753716345d1b6ebcd8870ecd0e4f7ef8c"
 
 DEPENDS = "coreutils-native"
-- 
2.25.1



  parent reply	other threads:[~2022-05-15 13:49 UTC|newest]

Thread overview: 21+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-05-15 13:49 [kirkstone 00/22] Patch review Armin Kuster
2022-05-15 13:49 ` [kirkstone 01/22] meta-oe-image: fix build depends Armin Kuster
2022-05-15 13:49 ` [kirkstone 02/22] frr: inherit autotools-brokensep instead of autotools Armin Kuster
2022-05-15 13:49 ` [kirkstone 03/22] conntrack-tools: Fix missing capability Armin Kuster
2022-05-15 13:49 ` [kirkstone 04/22] ufw: Fix packaging errors found with ppc64 Armin Kuster
2022-05-15 13:49 ` [kirkstone 05/22] libcereal: Enable for glibc/ppc Armin Kuster
2022-05-15 13:49 ` [kirkstone 06/22] mimic: Use special rateconv.c license Armin Kuster
2022-05-15 13:49 ` [kirkstone 07/22] makedumpfile: Use right TARGET for ppc32 Armin Kuster
2022-05-15 13:49 ` [kirkstone 10/22] python3-astroid: upgrade 2.11.2 -> 2.11.3 Armin Kuster
2022-05-15 13:49 ` [kirkstone 11/22] meta-python-image: Fix build depends Armin Kuster
2022-05-15 13:49 ` [kirkstone 12/22] python3-wxgtk4: Needs x11 for sip module Armin Kuster
2022-05-15 13:49 ` [kirkstone 13/22] meta-gnome: fix layer depends Armin Kuster
2022-05-15 13:49 ` [kirkstone 14/22] networkmanager: fix parallel build failure Armin Kuster
2022-05-15 13:49 ` [kirkstone 15/22] openldap: Remove libgcrypt dependency Armin Kuster
2022-05-15 13:49 ` [kirkstone 16/22] openldap: Upgrade 2.5.9 -> 2.5.12 Armin Kuster
2022-05-15 13:49 ` Armin Kuster [this message]
2022-05-15 13:49 ` [kirkstone 18/22] devmem2: reinstate previous patches, removed by mistake Armin Kuster
2022-05-15 13:49 ` [kirkstone 19/22] devmem2: add support for different page sizes Armin Kuster
2022-05-15 13:49 ` [kirkstone 20/22] devmem2: update SRC_URI according to redirect Armin Kuster
2022-05-15 13:49 ` [kirkstone 21/22] vboxguestdrivers: upgrade 6.1.32 -> 6.1.34 Armin Kuster
2022-05-15 13:49 ` [kirkstone 22/22] minidlna: fix obsolete license warning Armin Kuster

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=eb37ec6ba87d78338c66ea1ec5f76a54891000ec.1652622481.git.akuster808@gmail.com \
    --to=akuster808@gmail.com \
    --cc=openembedded-devel@lists.openembedded.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.