Re: [Nmh-workers] Doing a 1.7.1 release

[Ralph Corderoy]

Hi Ken,

> - The fix for the incorrect %(putnumf) truncation

aa034d7e

> - The removal of the removal of sbr/dtimep.c from 'make clean'

babec898

> Others?

Is it from the pool of commits made on master since the commit tagged
1.7 on 1.7-release?  These are the two master commits that bracket 1.7's
tagging in UTC time.

    2017-11-16 00:14:35  8f802697  fmt_new.c: Move interface to own file.
1.7 2017-11-17 15:23:34  ce7dac0f  Update for 1.7 final release.
    2017-11-18 15:35:19  f102d048  Move etc/rmmproc.messageid to fix its 
install permissions.

Having read through those, these look interesting.

    commit babec898bb63d36349aae85a8cbde6b97b274192
    Author: David Levine <address@hidden>
    Date:   Mon Nov 27 20:11:30 2017 -0500

        Removed sbr/dtimep.c from CLEANFILES.
        
        automake puts it in maintainer-clean-generic, which is where it belongs.
        Reversal of commit 1353a95720e4049e5f8357cd009b76c0450ec39f.

     Makefile.am | 4 +---
     1 file changed, 1 insertion(+), 3 deletions(-)

    commit 0e593b90e45687680b8ea9eb6c7643b44abde0fb
    Author: Ralph Corderoy <address@hidden>
    Date:   Mon Nov 27 15:20:44 2017 +0000

        test-mhfixmsg: pwd(1)'s `-P' needed to avoid failure with symlinks.
        
        Steven Winikoff reported to nmh-workers that one of this file's tests
        failed in 1.7.  It attempts to manipulate the output of mhpath(1)
        assuming it starts with the result of `pwd', but symlinks in the latter
        may cause this to fail.  Use POSIX's `-P' option to ensure no symlinks,
        as done in some other tests.

     test/mhfixmsg/test-mhfixmsg | 2 +-
     1 file changed, 1 insertion(+), 1 deletion(-)

    commit f81046da683a14b2265a501f0de00df264a77413
    Author: David Levine <address@hidden>
    Date:   Sun Nov 26 17:27:48 2017 -0500

        Use ISO-2022-KR instead of EBCDIC-US in test-charset, because
        Robert Elz reported that it is installed on his NetBSD host.
        Also, don't run that check without iconv.

     test/mhshow/test-charset | 11 +++++++----
     1 file changed, 7 insertions(+), 4 deletions(-)

    commit 47b86722957cca6057bf5fcd07c9d1f01b4516f8
    Author: Ralph Corderoy <address@hidden>
    Date:   Sun Nov 26 12:09:22 2017 +0000

        Fix tests that assume the backup prefix is a comma.
        
        Steven Winikoff reported to nmh-workers that some tests failed in 1.7
        configured with --with-hash-backup because they assume the backup prefix
        is comma rather than hash.  Alter those tests to use `mhparam sbackup',
        as other tests already do.  PATH is already set to use the mhparam being
        tested.
        
        test-mhfixmsg used find(1) to check for leftover files that globbed
        `mhfix*' or `,mhfix*'.  Changed to just `*mhfix*' as I think any file
        containing that would be unwanted, and it will also spot mhfixmsg
        hard-coding the comma prefix.

     test/mhbuild/test-forw      | 2 +-
     test/mhfixmsg/test-mhfixmsg | 9 +++++----
     test/post/test-rfc6531      | 2 +-
     test/refile/test-refile     | 6 ++++--
     4 files changed, 11 insertions(+), 8 deletions(-)

    commit 523641d2282ce5606a8b83250f9e45f33650edf6
    Author: David Levine <address@hidden>
    Date:   Fri Nov 24 18:41:23 2017 -0500

        Reworked parameter value tests to reflect Ralph's (re-?) discovery
        that GNU iconv elides the ? from charset names.  nmh uses it as a
        replacement character when decoding fails.

     test/mhshow/test-charset | 34 +++++++++++++++++-----------------
     1 file changed, 17 insertions(+), 17 deletions(-)

    commit a034d7e5edab2f0d56a62f6e1fcbbc76f3e5f31b
    Author: Ralph Corderoy <address@hidden>
    Date:   Wed Nov 22 13:44:01 2017 +0000

        Rewrite cpnumber() to fix `%2(msg)' with 123 overflowing to `?23'.
        
        Reported in 1.7 by Ken on nmh-workers yesterday.
        `It seems this was introduced in commit 92128dacf'.
        We now use the C library to format the number, and then manipulate the
        result.  A temporary charstring_t is no longer used.
        Also adds left justification using negative width.

     man/mh-format.man |  7 ++--
     sbr/fmt_scan.c    | 98 
++++++++++++++++++++++++++++++-------------------------
     2 files changed, 59 insertions(+), 46 deletions(-)

    commit f102d048fdd4679534460ac5cb0c610ee228d063
    Author: David Levine <address@hidden>
    Date:   Sat Nov 18 10:35:19 2017 -0500

        Move etc/rmmproc.messageid to fix its install permissions.
        
        And removed duplicate docs/contrib/replaliases to fix its install 
permissions.

     Makefile.am | 3 +--
     1 file changed, 1 insertion(+), 2 deletions(-)

-- 
Cheers, Ralph.
https://plus.google.com/+RalphCorderoy