CVS commit: pkgsrc/mail/neomutt

To: pkgsrc-changes%NetBSD.org@localhost
Subject: CVS commit: pkgsrc/mail/neomutt
From: "Thomas Klausner" <wiz%netbsd.org@localhost>
Date: Mon, 10 Oct 2016 19:26:31 +0000

Module Name:    pkgsrc
Committed By:   wiz
Date:           Mon Oct 10 19:26:31 UTC 2016

Modified Files:
        pkgsrc/mail/neomutt: Makefile distinfo
Added Files:
        pkgsrc/mail/neomutt/patches: patch-doc_manual.xml.head
            patch-doc_muttrc.forgotten-attachment patch-init.c patch-init.h
            patch-send.c

Log Message:
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.


To generate a diff of this commit:
cvs rdiff -u -r1.14 -r1.15 pkgsrc/mail/neomutt/Makefile
cvs rdiff -u -r1.10 -r1.11 pkgsrc/mail/neomutt/distinfo
cvs rdiff -u -r0 -r1.1 pkgsrc/mail/neomutt/patches/patch-doc_manual.xml.head \
    pkgsrc/mail/neomutt/patches/patch-doc_muttrc.forgotten-attachment \
    pkgsrc/mail/neomutt/patches/patch-init.c \
    pkgsrc/mail/neomutt/patches/patch-init.h \
    pkgsrc/mail/neomutt/patches/patch-send.c

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.

Modified files:

Index: pkgsrc/mail/neomutt/Makefile
diff -u pkgsrc/mail/neomutt/Makefile:1.14 pkgsrc/mail/neomutt/Makefile:1.15
--- pkgsrc/mail/neomutt/Makefile:1.14   Thu Oct  6 16:30:18 2016
+++ pkgsrc/mail/neomutt/Makefile        Mon Oct 10 19:26:31 2016
@@ -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}

Index: pkgsrc/mail/neomutt/distinfo
diff -u pkgsrc/mail/neomutt/distinfo:1.10 pkgsrc/mail/neomutt/distinfo:1.11
--- pkgsrc/mail/neomutt/distinfo:1.10   Thu Oct  6 16:27:15 2016
+++ pkgsrc/mail/neomutt/distinfo        Mon Oct 10 19:26:31 2016
@@ -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

Added files:

Index: pkgsrc/mail/neomutt/patches/patch-doc_manual.xml.head
diff -u /dev/null pkgsrc/mail/neomutt/patches/patch-doc_manual.xml.head:1.1
--- /dev/null   Mon Oct 10 19:26:31 2016
+++ pkgsrc/mail/neomutt/patches/patch-doc_manual.xml.head       Mon Oct 10 19:26:31 2016
@@ -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>
Index: pkgsrc/mail/neomutt/patches/patch-doc_muttrc.forgotten-attachment
diff -u /dev/null pkgsrc/mail/neomutt/patches/patch-doc_muttrc.forgotten-attachment:1.1
--- /dev/null   Mon Oct 10 19:26:31 2016
+++ pkgsrc/mail/neomutt/patches/patch-doc_muttrc.forgotten-attachment   Mon Oct 10 19:26:31 2016
@@ -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)\\>"
Index: pkgsrc/mail/neomutt/patches/patch-init.c
diff -u /dev/null pkgsrc/mail/neomutt/patches/patch-init.c:1.1
--- /dev/null   Mon Oct 10 19:26:31 2016
+++ pkgsrc/mail/neomutt/patches/patch-init.c    Mon Oct 10 19:26:31 2016
@@ -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);
+         }
Index: pkgsrc/mail/neomutt/patches/patch-init.h
diff -u /dev/null pkgsrc/mail/neomutt/patches/patch-init.h:1.1
--- /dev/null   Mon Oct 10 19:26:31 2016
+++ pkgsrc/mail/neomutt/patches/patch-init.h    Mon Oct 10 19:26:31 2016
@@ -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
Index: pkgsrc/mail/neomutt/patches/patch-send.c
diff -u /dev/null pkgsrc/mail/neomutt/patches/patch-send.c:1.1
--- /dev/null   Mon Oct 10 19:26:31 2016
+++ pkgsrc/mail/neomutt/patches/patch-send.c    Mon Oct 10 19:26:31 2016
@@ -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: CVS commit: pkgsrc/multimedia/x265
Next by Date: CVS commit: pkgsrc/doc
Previous by Thread: CVS commit: pkgsrc/multimedia/x265
Next by Thread: CVS commit: pkgsrc/doc
Indexes:

Home | Main Index | Thread Index | Old Index