pkg/48254: Update mail/fdm to 1.7

To: pkg-manager%netbsd.org@localhost,gnats-admin%netbsd.org@localhost,pkgsrc-bugs%netbsd.org@localhost
Subject: pkg/48254: Update mail/fdm to 1.7
From: Leonardo Taccari <iamleot%gmail.com@localhost>
Date: Mon, 30 Sep 2013 18:25:00 +0000 (UTC)
        Note: There was a bad value `' for the field `Class'.
        It was set to the default value of `sw-bug'.

>Number:         48254
>Category:       pkg
>Synopsis:       Update mail/fdm to 1.7
>Confidential:   no
>Severity:       non-critical
>Priority:       low
>Responsible:    pkg-manager
>State:          open
>Class:          sw-bug
>Submitter-Id:   net
>Arrival-Date:   Mon Sep 30 18:25:00 +0000 2013
>Originator:     Leonardo Taccari
>Release:        NetBSD 6.1.2
>Organization:
>Environment:
System: NetBSD :) 6.1.2 NetBSD 6.1.2 (EEEPC900) #0: Mon Sep 30 15:26:39 CEST 
2013 leot@:):/usr/src/sys/arch/i386/compile/EEEPC900 i386
Architecture: i386
Machine: i386
>Description:
        mail/fdm in pkgsrc (1.6) is not the latest stable version (1.7).
>How-To-Repeat:
        $ cd pkgsrc/mail/fdm
        $ make show-var VARNAME=DISTNAME
>Fix:
        Please apply the attached patches.

        I tried to improve the package in various ways:
         * Properly use the SUBST framework (and get rid of patches/patch-aa)
         * options.mk: fdm can be built with "debug" and "pcre" options
           (previously the user was not able to disable the PCRE support)
         * MANUAL is now generated via an awk(1) script. Add a line to
           post-install target accordingly.
         * Add HAS_CONFIGURE. Now fdm use a "configure" that automatically
           generate a config.h and config.mk (depending on ``uname -s''). If an 
OS
           is not supported adding it to ${WRKSRC}/configure is the first thing 
to
           do.
         * Add a comment to patches/patch-fdm.h
         * Various cosmetic changes to Makefile (inspired by
           pkgsrc/doc/Makefile-example, please email me regarding any mistakes
           that I've done!)

        While there I will also attach a possible commit message.
        Thank you in advance!


-------------------8<-----------------8<-----------------8<-------------------
Update mail/fdm to fdm-1.7

pkgsrc changes:
 * add options.mk: now fdm supports "debug" and "pcre" options (previously the
   PCRE support was always included).

Changes:
 * Add mbox tags for messages fetched from a mbox
 * Detect GMail's XYZZY capability for IMAP and use it to try and workaround
   some of their broken behaviour (incorrectly reported message sizes).
 * Print a warning on missing maildirs when fetching from them rather than
   crashing or giving an error. Reported by Frank Terbeck.
 * Introduce a configure script and tidy up build infrastructure.
 * GMail IMAP doesn't correctly set the \Seen flag after UID FETCH BODY[], so
   explicitly set it with STORE when mail is kept. Reported by Patrice Clement.
 * Properly count mails when polling multiple folders on a single IMAP server,
   reported by Claudio M. Alessi.
 * Support user and pass on NNTP, requested by Michael Hamann.
 * Escape . properly when delivering to SMTP.
 * Don't be as strict about format at the end of messages when using IMAP -
   accept additional information as well as FLAGS. Reported by rivo nurges.
-------------------8<-----------------8<-----------------8<-------------------


-------------------8<-----------------8<-----------------8<-------------------
Index: Makefile
===================================================================
RCS file: /cvsroot/pkgsrc/mail/fdm/Makefile,v
retrieving revision 1.4
diff -u -r1.4 Makefile
--- Makefile    6 Feb 2013 23:22:44 -0000       1.4
+++ Makefile    30 Sep 2013 17:48:51 -0000
@@ -1,9 +1,7 @@
 # $NetBSD: Makefile,v 1.4 2013/02/06 23:22:44 jperkin Exp $
 
-DISTNAME=              fdm-1.6
-PKGREVISION=           1
+DISTNAME=              fdm-1.7
 CATEGORIES=            mail
-PKG_REVISION=          1
 MASTER_SITES=          ${MASTER_SITE_SOURCEFORGE:=fdm/}
 
 MAINTAINER=            pkgsrc-users%NetBSD.org@localhost
@@ -11,27 +9,30 @@
 COMMENT=               Fetch or receive mail and deliver it in various ways
 LICENSE=               modified-bsd
 
-INSTALLATION_DIRS+=    ${EGDIR} ${DOCDIR}
-EGDIR=                 ${PREFIX}/share/examples/fdm
 DOCDIR=                        ${PREFIX}/share/doc/fdm
+EGDIR=                 ${PREFIX}/share/examples/fdm
+HAS_CONFIGURE=         yes
+INSTALLATION_DIRS+=    ${DOCDIR} ${EGDIR}
 
 USE_TOOLS+=            pax
-MAKE_ENV+=             PCRE=1
 
-SUBST_CLASSES+=                files
-SUBST_STAGE.files=     post-patch
-SUBST_FILES.files=     Makefile
-SUBST_SED.files=       -e "s|@INSTALL_PROGRAM_DIR@|${INSTALL_PROGRAM_DIR}|"
-SUBST_SED.files+=      -e "s|@INSTALL_PROGRAM@|${INSTALL_PROGRAM}|"
-SUBST_SED.files+=      -e "s|@INSTALL_MAN@|${INSTALL_MAN}|"
-SUBST_MESSAGE.files=   Fixing install invocation in Makefile
+SUBST_CLASSES+=                makefile
+SUBST_STAGE.makefile=  post-patch
+SUBST_MESSAGE.makefile=        Fixing install invocation in Makefile
+SUBST_FILES.makefile=  Makefile
+SUBST_SED.makefile+=   -e '/^FDEBUG=/d'        # Disable debug options
+SUBST_SED.makefile+=   -E -e 's,(INSTALLDIR=).*,\1 ${INSTALL_PROGRAM_DIR:Q},'
+SUBST_SED.makefile+=   -E -e 's,(INSTALLBIN=).*,\1 ${INSTALL_PROGRAM:Q},'
+SUBST_SED.makefile+=   -E -e 's,(INSTALLMAN=).*,\1 ${INSTALL_MAN:Q},'
+
+.include "options.mk"
+.include "../../databases/tdb/buildlink3.mk"
+.include "../../devel/zlib/buildlink3.mk"
+.include "../../security/openssl/buildlink3.mk"
 
 post-install:
        cd ${WRKSRC}/examples && pax -rwpam . ${DESTDIR}${EGDIR}
+       cd ${WRKSRC} && ${MAKE_PROGRAM} -f ${WRKSRC}/tools/dist.mk manual
        ${INSTALL_DATA} ${WRKSRC}/MANUAL ${DESTDIR}${DOCDIR}
 
-.include "../../security/openssl/buildlink3.mk"
-.include "../../devel/zlib/buildlink3.mk"
-.include "../../devel/pcre/buildlink3.mk"
-.include "../../databases/tdb/buildlink3.mk"
 .include "../../mk/bsd.pkg.mk"
Index: PLIST
===================================================================
RCS file: /cvsroot/pkgsrc/mail/fdm/PLIST,v
retrieving revision 1.1.1.1
diff -u -r1.1.1.1 PLIST
--- PLIST       17 Aug 2012 17:45:40 -0000      1.1.1.1
+++ PLIST       30 Sep 2013 17:48:51 -0000
@@ -1,11 +1,10 @@
-@comment $NetBSD: PLIST,v 1.1.1.1 2012/08/17 17:45:40 seb Exp $
+@comment $NetBSD$
 bin/fdm
 man/man1/fdm.1
 man/man5/fdm.conf.5
 share/doc/fdm/MANUAL
 share/examples/fdm/f-terbeck.conf
 share/examples/fdm/g-lando.conf
-share/examples/fdm/n-marriott-old.conf
 share/examples/fdm/n-marriott.conf
 share/examples/fdm/t-ulmer.conf
 share/examples/fdm/w-maier.conf
Index: distinfo
===================================================================
RCS file: /cvsroot/pkgsrc/mail/fdm/distinfo,v
retrieving revision 1.2
diff -u -r1.2 distinfo
--- distinfo    23 Aug 2012 18:04:12 -0000      1.2
+++ distinfo    30 Sep 2013 17:48:51 -0000
@@ -1,7 +1,6 @@
 $NetBSD: distinfo,v 1.2 2012/08/23 18:04:12 marino Exp $
 
-SHA1 (fdm-1.6.tar.gz) = fb8042f2355062848ff142c9604069b6657e6a82
-RMD160 (fdm-1.6.tar.gz) = 7e2c846b97dce20e123b8453732e107f88e0de8f
-Size (fdm-1.6.tar.gz) = 192159 bytes
-SHA1 (patch-aa) = 4284d7fc2e88fc9d42fc861e19a890dd526354d6
-SHA1 (patch-fdm.h) = 03ad0167120e06e76f4f1ef086c918c6a0cccf55
+SHA1 (fdm-1.7.tar.gz) = 5534d1ec1831794e092e0821db70f36b7e1ee961
+RMD160 (fdm-1.7.tar.gz) = 942f0989501645aa7e04bd428510daa309e8fca6
+Size (fdm-1.7.tar.gz) = 199808 bytes
+SHA1 (patch-fdm.h) = 5119757e747908a45ca778fe8d27d7944e37125f
Index: options.mk
===================================================================
RCS file: options.mk
diff -N options.mk
--- /dev/null   1 Jan 1970 00:00:00 -0000
+++ options.mk  30 Sep 2013 17:48:51 -0000
@@ -0,0 +1,22 @@
+# $NetBSD$
+
+PKG_OPTIONS_VAR=       PKG_OPTIONS.fdm
+PKG_SUPPORTED_OPTIONS= debug pcre
+PKG_SUGGESTED_OPTIONS= pcre
+
+.include "../../mk/bsd.options.mk"
+
+#
+# Debug
+#
+.if !empty(PKG_OPTIONS:Mdebug)
+   MAKE_ENV+=  FDEBUG=1
+.endif
+
+#
+# PCRE support
+#
+.if !empty(PKG_OPTIONS:Mpcre)
+   MAKE_ENV+=  PCRE=1
+.  include "../../devel/pcre/buildlink3.mk"
+.endif
Index: patches/patch-aa
===================================================================
RCS file: patches/patch-aa
diff -N patches/patch-aa
--- patches/patch-aa    23 Aug 2012 18:04:12 -0000      1.2
+++ /dev/null   1 Jan 1970 00:00:00 -0000
@@ -1,38 +0,0 @@
-$NetBSD: patch-aa,v 1.2 2012/08/23 18:04:12 marino Exp $
-
-cleanup cpp args dans adjust/fix Darwin build
-
---- Makefile.orig      2008-12-22 16:20:05.000000000 +0000
-+++ Makefile
-@@ -37,7 +37,7 @@ HDRS= fdm.h array.h fetch.h match.h deli
- YACC= yacc -d
- 
- CC?= cc
--INCDIRS+= -I. -I- -I/usr/local/include
-+INCDIRS+= -I. 
- .ifdef PROFILE
- # Don't use ccache
- CC= /usr/bin/gcc
-@@ -70,7 +70,8 @@ LIBS+= -lpcre
- .if ${OS} == "Darwin"
- SRCS+= compat/strtonum.c
- INCDIRS+= -Icompat -I/usr/local/include/openssl
--CFLAGS+= -DNO_STRTONUM -DNO_SETRESUID -DNO_SETRESGID -DNO_SETPROCTITLE
-+CFLAGS+= -DNO_STRTONUM -DNO_SETRESUID -DNO_SETRESGID -DNO_SETPROCTITLE 
-DNO_QUEUE_H -DNO_TREE_H
-+LIBS+=        -lresolv
- .endif
- 
- # NetBSD
-@@ -93,9 +94,9 @@ CFLAGS+= -DNO_STRTONUM
- .endif
- 
- PREFIX?= /usr/local
--INSTALLDIR= install -d
--INSTALLBIN= install -g bin -o root -m 555
--INSTALLMAN= install -g bin -o root -m 444
-+INSTALLDIR= @INSTALL_PROGRAM_DIR@
-+INSTALLBIN= @INSTALL_PROGRAM@
-+INSTALLMAN= @INSTALL_MAN@
- 
- LDFLAGS+= -L/usr/local/lib
- .ifdef PROFILE
Index: patches/patch-fdm.h
===================================================================
RCS file: /cvsroot/pkgsrc/mail/fdm/patches/patch-fdm.h,v
retrieving revision 1.1
diff -u -r1.1 patch-fdm.h
--- patches/patch-fdm.h 23 Aug 2012 18:04:12 -0000      1.1
+++ patches/patch-fdm.h 30 Sep 2013 17:48:51 -0000
@@ -1,5 +1,8 @@
 $NetBSD: patch-fdm.h,v 1.1 2012/08/23 18:04:12 marino Exp $
 
+MAXNAMLEN is not POSIX. For systems that does not define it use NAME_MAX
+macro instead.
+
 --- fdm.h.orig 2012-08-23 17:13:57.000000000 +0000
 +++ fdm.h
 @@ -71,6 +71,10 @@
Prev by Date: Re: pkg/48252: x11/rxvt-unicode invalid assembly macro for sparc
Next by Date: pkg/48253: pkgsrc: devel/ncurses/ does not build on Linux/Ubuntu/x86_64 in conjunction with libtool
Previous by Thread: pkg/48252: x11/rxvt-unicode invalid assembly macro for sparc
Next by Thread: Re: pkg/48254: Update mail/fdm to 1.7
Indexes:
Home | Main Index | Thread Index | Old Index