[pkgsrc/trunk]: pkgsrc/mail/neomutt Apply upstream patch:

To: pkgsrc-changes-hg%NetBSD.org@localhost
Subject: [pkgsrc/trunk]: pkgsrc/mail/neomutt Apply upstream patch:
From: wiz <wiz%pkgsrc.org@localhost>
Date: Wed, 15 Jan 2020 06:08:04 +0000
details:   https://anonhg.NetBSD.org/pkgsrc/rev/c8906feca917
branches:  trunk
changeset: 353720:c8906feca917
user:      wiz <wiz%pkgsrc.org@localhost>
date:      Mon Oct 10 19:26:31 2016 +0000

description:
Apply upstream patch:

forgotten-attachment: fix empty regex expression

The original regex was of the form "abc(|def)" to check for both "abc"
and "abcdef".  Unfortunately, the regex libraries on BSDs/MacOS don't
like this use of an empty sub-expression.

Expanding the regex to: "(abc|abcdef)" fixes the problem.

https://github.com/neomutt/neomutt/commit/3bc69ca25077b171f1ae9beee484629c5b5c6482

Bump PKGREVISION.

diffstat:

 mail/neomutt/Makefile                                      |   4 +-
 mail/neomutt/distinfo                                      |   7 ++-
 mail/neomutt/patches/patch-doc_manual.xml.head             |  24 ++++++++
 mail/neomutt/patches/patch-doc_muttrc.forgotten-attachment |  23 ++++++++
 mail/neomutt/patches/patch-init.c                          |  39 ++++++++++++++
 mail/neomutt/patches/patch-init.h                          |  23 ++++++++
 mail/neomutt/patches/patch-send.c                          |  23 ++++++++
 7 files changed, 140 insertions(+), 3 deletions(-)

diffs (184 lines):

diff -r 15b1a40f0edf -r c8906feca917 mail/neomutt/Makefile
--- a/mail/neomutt/Makefile     Mon Oct 10 17:13:42 2016 +0000
+++ b/mail/neomutt/Makefile     Mon Oct 10 19:26:31 2016 +0000
@@ -1,7 +1,7 @@
-# $NetBSD: Makefile,v 1.14 2016/10/06 16:30:18 wiz Exp $
+# $NetBSD: Makefile,v 1.15 2016/10/10 19:26:31 wiz Exp $
 
 DISTNAME=              neomutt-20161003
-PKGREVISION=           1
+PKGREVISION=           2
 CATEGORIES=            mail
 MASTER_SITES=          ${MASTER_SITE_GITHUB:=neomutt/}
 GITHUB_TAG=            ${DISTNAME}
diff -r 15b1a40f0edf -r c8906feca917 mail/neomutt/distinfo
--- a/mail/neomutt/distinfo     Mon Oct 10 17:13:42 2016 +0000
+++ b/mail/neomutt/distinfo     Mon Oct 10 19:26:31 2016 +0000
@@ -1,9 +1,14 @@
-$NetBSD: distinfo,v 1.10 2016/10/06 16:27:15 wiz Exp $
+$NetBSD: distinfo,v 1.11 2016/10/10 19:26:31 wiz Exp $
 
 SHA1 (neomutt-20161003.tar.gz) = d0849c7efaa7e099b17137d8f5f90b9a201142f1
 RMD160 (neomutt-20161003.tar.gz) = c69fe3dba35c3341987b6eb21b4bc0a1eb546762
 SHA512 (neomutt-20161003.tar.gz) = f3b42e0bf12038bfdce20db30d2ac4ee60a4ce317894ff035a4180a53d0899936aef6f809c52ef8786eab6b247e98ae244578d00a5dafb44e04f4b3ccc84b592
 Size (neomutt-20161003.tar.gz) = 2656280 bytes
+SHA1 (patch-doc_manual.xml.head) = c569f4d51781ecb5e980e83e6f8cde9cdb97a189
+SHA1 (patch-doc_muttrc.forgotten-attachment) = 185086380ac27b02e8f39e611d8de5461c1af8b0
 SHA1 (patch-hash.c) = 63df8e500d91f0dffb24801b84dc7155539d34a5
 SHA1 (patch-hash.h) = ddc94cfaeead9800f5f902de9f4255803122463d
 SHA1 (patch-headers.c) = 8d985b9dfc717c5c30e9fb314c33b35d61dcaf56
+SHA1 (patch-init.c) = 607da46758dcf4ee85644926f5d33f821b2f6713
+SHA1 (patch-init.h) = f2cfba5e802527d546623dbf01d5f2a8b1552cf3
+SHA1 (patch-send.c) = 16ed40585b04e136d96eb20e12caa2855c856b23
diff -r 15b1a40f0edf -r c8906feca917 mail/neomutt/patches/patch-doc_manual.xml.head
--- /dev/null   Thu Jan 01 00:00:00 1970 +0000
+++ b/mail/neomutt/patches/patch-doc_manual.xml.head    Mon Oct 10 19:26:31 2016 +0000
@@ -0,0 +1,24 @@
+$NetBSD: patch-doc_manual.xml.head,v 1.1 2016/10/10 19:26:31 wiz Exp $
+
+forgotten-attachment: fix empty regex expression
+
+The original regex was of the form "abc(|def)" to check for both "abc"
+and "abcdef".  Unfortunately, the regex libraries on BSDs/MacOS don't
+like this use of an empty sub-expression.
+
+Expanding the regex to: "(abc|abcdef)" fixes the problem.
+
+https://github.com/neomutt/neomutt/commit/3bc69ca25077b171f1ae9beee484629c5b5c6482
+
+
+--- doc/manual.xml.head.orig   2016-10-03 11:27:32.000000000 +0000
++++ doc/manual.xml.head
+@@ -10142,7 +10142,7 @@ set pgp_encrypt_self = &quot;no&quot;
+           <row>
+             <entry><literal>attach_keyword</literal></entry>
+             <entry>regular expression</entry>
+-            <entry><literal>\\&lt;attach(|ed|ments?)\\&gt;</literal></entry>
++            <entry><literal>\\&lt;(attach|attached|attachments?)\\&gt;</literal></entry>
+           </row>
+           <row>
+             <entry><literal>abort_noattach</literal></entry>
diff -r 15b1a40f0edf -r c8906feca917 mail/neomutt/patches/patch-doc_muttrc.forgotten-attachment
--- /dev/null   Thu Jan 01 00:00:00 1970 +0000
+++ b/mail/neomutt/patches/patch-doc_muttrc.forgotten-attachment        Mon Oct 10 19:26:31 2016 +0000
@@ -0,0 +1,23 @@
+$NetBSD: patch-doc_muttrc.forgotten-attachment,v 1.1 2016/10/10 19:26:31 wiz Exp $
+
+forgotten-attachment: fix empty regex expression
+
+The original regex was of the form "abc(|def)" to check for both "abc"
+and "abcdef".  Unfortunately, the regex libraries on BSDs/MacOS don't
+like this use of an empty sub-expression.
+
+Expanding the regex to: "(abc|abcdef)" fixes the problem.
+
+https://github.com/neomutt/neomutt/commit/3bc69ca25077b171f1ae9beee484629c5b5c6482
+
+--- doc/muttrc.forgotten-attachment.orig       2016-10-03 11:27:32.000000000 +0000
++++ doc/muttrc.forgotten-attachment
+@@ -16,7 +16,7 @@ set abort_noattach = no
+ # Search for the following regular expression in the body of the email
+ 
+ # English: attach, attached, attachment, attachments
+-set attach_keyword = "\\<attach(|ed|ments?)\\>"
++set attach_keyword = "\\<(attach|attached|attachments?)\\>"
+ 
+ # Nederlands:
+ # set attach_keyword = "\\<(bijvoegen|bijgevoegd|bijlage|bijlagen)\\>"
diff -r 15b1a40f0edf -r c8906feca917 mail/neomutt/patches/patch-init.c
--- /dev/null   Thu Jan 01 00:00:00 1970 +0000
+++ b/mail/neomutt/patches/patch-init.c Mon Oct 10 19:26:31 2016 +0000
@@ -0,0 +1,39 @@
+$NetBSD: patch-init.c,v 1.1 2016/10/10 19:26:31 wiz Exp $
+
+forgotten-attachment: fix empty regex expression
+
+The original regex was of the form "abc(|def)" to check for both "abc"
+and "abcdef".  Unfortunately, the regex libraries on BSDs/MacOS don't
+like this use of an empty sub-expression.
+
+Expanding the regex to: "(abc|abcdef)" fixes the problem.
+
+https://github.com/neomutt/neomutt/commit/3bc69ca25077b171f1ae9beee484629c5b5c6482
+
+--- init.c.orig        2016-10-03 11:27:32.000000000 +0000
++++ init.c
+@@ -1704,6 +1704,7 @@ static void mutt_restore_default (struct
+ 
+       if (p->init)
+       {
++        int retval;
+         char *s = (char *) p->init;
+ 
+         pp->rx = safe_calloc (1, sizeof (regex_t));
+@@ -1715,10 +1716,15 @@ static void mutt_restore_default (struct
+           s++;
+           pp->not = 1;
+         }
+-        if (REGCOMP (pp->rx, s, flags) != 0)
++        retval = REGCOMP (pp->rx, s, flags);
++        if (retval != 0)
+         {
++          char msgbuf[STRING];
++          regerror (retval, pp->rx, msgbuf, sizeof (msgbuf));
+           fprintf (stderr, _("mutt_restore_default(%s): error in regexp: %s\n"),
+                    p->option, pp->pattern);
++          fprintf (stderr, "%s\n", msgbuf);
++          mutt_sleep (0);
+           FREE (&pp->pattern);
+           FREE (&pp->rx);
+         }
diff -r 15b1a40f0edf -r c8906feca917 mail/neomutt/patches/patch-init.h
--- /dev/null   Thu Jan 01 00:00:00 1970 +0000
+++ b/mail/neomutt/patches/patch-init.h Mon Oct 10 19:26:31 2016 +0000
@@ -0,0 +1,23 @@
+$NetBSD: patch-init.h,v 1.1 2016/10/10 19:26:31 wiz Exp $
+
+forgotten-attachment: fix empty regex expression
+
+The original regex was of the form "abc(|def)" to check for both "abc"
+and "abcdef".  Unfortunately, the regex libraries on BSDs/MacOS don't
+like this use of an empty sub-expression.
+
+Expanding the regex to: "(abc|abcdef)" fixes the problem.
+
+https://github.com/neomutt/neomutt/commit/3bc69ca25077b171f1ae9beee484629c5b5c6482
+
+--- init.h.orig        2016-10-03 11:27:32.000000000 +0000
++++ init.h
+@@ -267,7 +267,7 @@ struct option_t MuttVars[] = {
+   ** .pp
+   ** For an explanation of ``soft-fill'', see the $$index_format documentation.
+   */
+-  { "attach_keyword",  DT_RX,  R_NONE, UL &AttachKeyword, UL "\\<attach(|ed|ments?)\\>" },
++  { "attach_keyword",  DT_RX,  R_NONE, UL &AttachKeyword, UL "\\<(attach|attached|attachments?)\\>" },
+   /*
+   ** .pp
+   ** If $abort_noattach is not set to no, then the body of the message
diff -r 15b1a40f0edf -r c8906feca917 mail/neomutt/patches/patch-send.c
--- /dev/null   Thu Jan 01 00:00:00 1970 +0000
+++ b/mail/neomutt/patches/patch-send.c Mon Oct 10 19:26:31 2016 +0000
@@ -0,0 +1,23 @@
+$NetBSD: patch-send.c,v 1.1 2016/10/10 19:26:31 wiz Exp $
+
+forgotten-attachment: fix empty regex expression
+
+The original regex was of the form "abc(|def)" to check for both "abc"
+and "abcdef".  Unfortunately, the regex libraries on BSDs/MacOS don't
+like this use of an empty sub-expression.
+
+Expanding the regex to: "(abc|abcdef)" fixes the problem.
+
+https://github.com/neomutt/neomutt/commit/3bc69ca25077b171f1ae9beee484629c5b5c6482
+
+--- send.c.orig        2016-10-03 11:27:32.000000000 +0000
++++ send.c
+@@ -1902,7 +1902,7 @@ main_loop:
+     /* if the abort is automatic, print an error message */
+     if (quadoption (OPT_ATTACH) == MUTT_YES)
+     {
+-      mutt_error _("Message contains text matching \"attach_keyword\". Not sending.");
++      mutt_error _("Message contains text matching \"$attach_keyword\". Not sending.");
+     }
+     goto main_loop;
+   }
Prev by Date: [pkgsrc/trunk]: pkgsrc/doc Note update of graphics/ruby-chunky_png package to...
Next by Date: [pkgsrc/trunk]: pkgsrc/devel/boost-headers PLIST update after distfile change
Previous by Thread: [pkgsrc/trunk]: pkgsrc/doc Note update of graphics/ruby-chunky_png package to...
Next by Thread: [pkgsrc/trunk]: pkgsrc/devel/boost-headers PLIST update after distfile change
Indexes:
Home | Main Index | Thread Index | Old Index