On Mon, Jan 22, 2007 at 08:53:56AM +0100, Giovanni Bechis wrote:
> I have updated tin to 1.8.2 version, I have tested it only on i386,
> a patch is attached.
> With this release the help command do not sigsev anymore.

Well, you could have e-mailed the maintainer first, who (w|sh)ould have
told you that I was already working on an updated port [1].

There are some more things that could be done. For one, it seems the
whole thing is under a BSD license; grep, CHANGES.old, sourceforge.net,
and Debian
<http://packages.debian.org/changelogs/pool/main/t/tin/tin_1.7.8+20050314-1/tin.copyright>
all seem to think so. I was waiting on fgsch@ to answer whether or not
there was a special reason it couldn't be distributed.

My diff is below; I've held off on changing the plugin locations while
waiting for him to respond. I'm in favour of that change, BTW
- but you might want to add a notice that one must change one's
configuration if upgrading from an old version.

Note that the --without-x doesn't actually do anything on my system.

However, if steven is right about this crashing on sparc64 platforms,
that could be a problem.

How do you want to proceed? You're welcome to take it, but I'd like it
to be maintained. If you don't want to, I would be willing to take
MAINTAINER (fgsch@ was okay with that in his last message; I suppose
that hasn't changed, and would extend to you). I use it almost daily,
anyway.

FWIW, this has been working well for over a month on my -current/i386.

Also note that this release fixes several `security holes'; I have not
investigated whether or not they are actually exploitable on OpenBSD,
but it's unlikely to be easy. Still, fixing them doesn't hurt.

                Joachim

[1] I just mailed him - again - not five minutes ago, ironically. He
didn't respond to my last message, about two weeks before.

diff -Nurd /usr/ports/news/tin/Makefile /usr/ports/mystuff/news/tin/Makefile
--- /usr/ports/news/tin/Makefile        Thu Dec  2 20:19:59 2004
+++ /usr/ports/mystuff/news/tin/Makefile        Mon Jan 22 23:26:01 2007
@@ -2,28 +2,32 @@
 
 COMMENT=       'threaded NNTP and spool based UseNet newsreader'
 
-DISTNAME=      tin-1.6.2
+DISTNAME=      tin-${MAJORVERSION}.${PATCHLEVEL}
+MAJORVERSION=  1.8
+PATCHLEVEL=    2
 CATEGORIES=    news
 
-MASTER_SITES=  ftp://ftp.tin.org/pub/news/clients/tin/stable/ \
-               ftp://ftp.funet.fi/pub/unix/news/tin-unoff/stable/ \
-               ftp://ftp.akk.uni-karlsruhe.de/pub/news/clients/tin/stable/ \
-               ftp://ftp.cuhk.edu.hk/pub/packages/news/tin/stable/ \
-               ftp://ftp.sunet.se/pub/news/readers/tin/stable/
+MASTER_SITES=  ftp://ftp.tin.org/pub/news/clients/tin/v${MAJORVERSION}/ \
+               
ftp://ftp.akk.uni-karlsruhe.de/pub/news/clients/tin/v${MAJORVERSION}/ \
+               ftp://ftp.cuhk.edu.hk/pub/packages/news/tin/v${MAJORVERSION}/ \
+               ftp://ftp.sunet.se/pub/news/readers/tin/v${MAJORVERSION}/ \
+               ftp://ftp.funet.fi/pub/unix/news/tin-unoff/
 
 HOMEPAGE=      http://www.tin.org/
 
-MAINTAINER=     Federico G. Schwindt <[EMAIL PROTECTED]>
+MAINTAINER=     Joachim Schipper <[EMAIL PROTECTED]>
 
 MODULES=       devel/gettext
 
-PERMIT_PACKAGE_CDROM=  "no fee"
+# XXX: It is my belief that this software can be redistributed freely.
+PERMIT_PACKAGE_CDROM=  Yes
 PERMIT_PACKAGE_FTP=    Yes
-PERMIT_DISTFILES_CDROM=        "no fee"
+PERMIT_DISTFILES_CDROM=        Yes
 PERMIT_DISTFILES_FTP=  Yes
 WANTLIB=               c ncurses
 
 CONFIGURE_STYLE=       gnu
+# --with-x does not affect the build, but let's be safe.
 CONFIGURE_ARGS+=       --with-ispell=${LOCALBASE}/bin/ispell \
                        --without-metamail \
                        --without-pgp \
@@ -31,23 +35,16 @@
                        --with-gpg=${LOCALBASE}/bin/gpg \
                        --with-screen=ncurses \
                        --with-pcre=${LOCALBASE} \
+                       --with-libdir=${PREFIX}/lib/news \
+                       --with-spooldir=/var/news \
                        --enable-break-long-lines \
-                       --enable-ipv6
+                       --enable-ipv6 \
+                       --without-x
 CONFIGURE_ENV= CPPFLAGS="-I${LOCALBASE}/include" \
                LDFLAGS="-L${LOCALBASE}/lib"
 
 LIB_DEPENDS+=  pcre::devel/pcre \
                utf8::misc/libutf8
-
-FLAVORS=       nntp_only
-FLAVOR?=
-
-.if ${FLAVOR:L:Mnntp_only}
-CONFIGURE_ARGS+=       --enable-nntp-only
-.else
-CONFIGURE_ARGS+=       --with-libdir=${PREFIX}/lib/news \
-                       --with-spooldir=/var/news
-.endif
 
 ALL_TARGET=    build
 
diff -Nurd /usr/ports/news/tin/distinfo /usr/ports/mystuff/news/tin/distinfo
--- /usr/ports/news/tin/distinfo        Wed Jan  5 18:21:52 2005
+++ /usr/ports/mystuff/news/tin/distinfo        Thu Jan  4 23:01:46 2007
@@ -1,4 +1,4 @@
-MD5 (tin-1.6.2.tar.gz) = 8f34747809b6fe5c135694fcbe2c8f46
-RMD160 (tin-1.6.2.tar.gz) = 651a6709f09346c2276866968d1efcafefb5080e
-SHA1 (tin-1.6.2.tar.gz) = f2113583c93105ca518fe4b56dc9d2825e6a15f5
-SIZE (tin-1.6.2.tar.gz) = 1548753
+MD5 (tin-1.8.2.tar.gz) = 397b6e22861d1ed4fc0fdb0e4a9f8876
+RMD160 (tin-1.8.2.tar.gz) = cc72f22f4b85091b8968a63ef8b36568cb484c6f
+SHA1 (tin-1.8.2.tar.gz) = 6a0b4505d9d769cb58dea483fadab8e86a08ec69
+SIZE (tin-1.8.2.tar.gz) = 1996960
diff -Nurd /usr/ports/news/tin/patches/patch-src_Makefile_in 
/usr/ports/mystuff/news/tin/patches/patch-src_Makefile_in
--- /usr/ports/news/tin/patches/patch-src_Makefile_in   Tue Aug 19 18:48:00 2003
+++ /usr/ports/mystuff/news/tin/patches/patch-src_Makefile_in   Mon Jan 22 
23:40:08 2007
@@ -1,6 +1,6 @@
-$OpenBSD: patch-src_Makefile_in,v 1.2 2003/08/19 16:48:00 fgsch Exp $
---- src/Makefile.in.orig       2003-07-24 11:18:45.000000000 -0300
-+++ src/Makefile.in    2003-08-19 13:47:24.000000000 -0300
+$OpenBSD$
+--- src/Makefile.in.orig       Sat Jul 16 21:49:36 2005
++++ src/Makefile.in    Sun Jan  7 18:59:07 2007
 @@ -36,6 +36,7 @@ VERSION = @VERSION@
  
  SHELL         = /bin/sh
@@ -9,7 +9,7 @@
  exec_prefix   = @exec_prefix@
  top_srcdir    = @top_srcdir@
  top_builddir  = ..
-@@ -69,6 +70,7 @@ LIBS         = $(PCRE_LIBS) $(CANLIB) @LIBS@ @I
+@@ -78,6 +79,7 @@ LINK_LIBS    = $(PCRE_LIBS) $(CANLIB) @LIBS
  # Where do you want the binary & manual page installed?
  DESTDIR       = @DESTDIR@
  INS_BINARY_DIR        = @bindir@
@@ -17,7 +17,7 @@
  INS_MANUAL_DIR        = @mandir@/man
  INS_SYSDEF_DIR        = @TIN_DEFAULTS_DIR@
  INS_MANUAL_EXT        = 1
-@@ -318,16 +320,15 @@ help:
+@@ -328,16 +330,15 @@ help :
  $(DESTDIR)$(INS_MANUAL_DIR)$(INS_MANUAL_EXT) \
  $(DESTDIR)$(INS_MANUAL_DIR)5 \
  $(DESTDIR)$(INS_BINARY_DIR) \
@@ -25,8 +25,8 @@
  $(DESTDIR)$(INS_SYSDEF_DIR) :
        $(top_srcdir)/mkdirs.sh $@
  
--install_manpage: $(DESTDIR)$(INS_MANUAL_DIR)$(INS_MANUAL_EXT) 
$(DESTDIR)$(INS_MANUAL_DIR)5
-+install_manpage: $(DESTDIR)$(INS_MANUAL_DIR)$(INS_MANUAL_EXT) 
$(DESTDIR)$(INS_MANUAL_DIR)5 $(DESTDIR)$(INS_DATA_DIR)
+-install_manpage : $(DESTDIR)$(INS_MANUAL_DIR)$(INS_MANUAL_EXT) 
$(DESTDIR)$(INS_MANUAL_DIR)5
++install_manpage : $(DESTDIR)$(INS_MANUAL_DIR)$(INS_MANUAL_EXT) 
$(DESTDIR)$(INS_MANUAL_DIR)5 $(DESTDIR)$(INS_DATA_DIR)
        @$(ECHO) "Installing $(PROJECT) manual pages to 
$(DESTDIR)$(INS_MANUAL_DIR) ..."
        @$(INSTALL) -m 444 $(DOCDIR)/tin.1 
$(DESTDIR)$(INS_MANUAL_DIR)$(INS_MANUAL_EXT)/$(PROJECT).$(INS_MANUAL_EXT)
        @( $(CD) $(DESTDIR)$(INS_MANUAL_DIR)$(INS_MANUAL_EXT) && $(RM) -f 
r$(PROJECT).$(INS_MANUAL_EXT) && $(LN) $(PROJECT).$(INS_MANUAL_EXT) 
r$(PROJECT).$(INS_MANUAL_EXT) )
@@ -34,27 +34,40 @@
 -      @-if test ! -f $(DESTDIR)$(INS_MANUAL_DIR)5/mbox.5 ; then $(INSTALL) -m 
444 $(DOCDIR)/mbox.5 $(DESTDIR)$(INS_MANUAL_DIR)5/mbox.5 ; else $(ECHO) "  ... 
skipping $(DESTDIR)$(INS_MANUAL_DIR)5/mbox.5 - file already exists" ; fi
 -      @-if test ! -f $(DESTDIR)$(INS_MANUAL_DIR)5/mmdf.5 ; then $(INSTALL) -m 
444 $(DOCDIR)/mmdf.5 $(DESTDIR)$(INS_MANUAL_DIR)5/mmdf.5 ; else $(ECHO) "  ... 
skipping $(DESTDIR)$(INS_MANUAL_DIR)5/mmdf.5 - file already exists" ; fi
  
- uninstall_manpage:
+ uninstall_manpage :
        @$(ECHO) "Uninstalling $(PROJECT) manual page in 
$(DESTDIR)$(INS_MANUAL_DIR)$(INS_MANUAL_EXT) ..."
-@@ -349,14 +350,14 @@ uninstall_sysdefs:
+@@ -359,16 +360,16 @@ uninstall_sysdefs :
        @-$(RM) -f $(DESTDIR)$(INS_SYSDEF_DIR)/tin.defaults
        @-$(RM) -f $(DESTDIR)$(INS_SYSDEF_DIR)/mime.types
  
--install_helpers: $(DESTDIR)$(INS_BINARY_DIR) 
$(DESTDIR)$(INS_MANUAL_DIR)$(INS_MANUAL_EXT)
-+install_helpers: $(DESTDIR)$(INS_DATA_DIR)
+-install_helpers : $(DESTDIR)$(INS_BINARY_DIR) 
$(DESTDIR)$(INS_MANUAL_DIR)$(INS_MANUAL_EXT)
++install_helpers : $(DESTDIR)$(INS_DATA_DIR)
        @$(ECHO) "Installing helper applications ..."
 -      @-if test ! -f $(DESTDIR)$(INS_BINARY_DIR)/url_handler.sh ; then 
$(INSTALL) -m 755 $(TOLDIR)/url_handler.sh 
$(DESTDIR)$(INS_BINARY_DIR)/url_handler.sh ; fi
 -      @-if test ! -f $(DESTDIR)$(INS_BINARY_DIR)/metamutt ; then $(INSTALL) 
-m 755 $(TOLDIR)/metamutt $(DESTDIR)$(INS_BINARY_DIR)/metamutt ; fi
 -      @$(INSTALL) -m 755 $(TOLDIR)/w2r.pl $(DESTDIR)$(INS_BINARY_DIR)/w2r.pl
 -      @$(INSTALL) -m 755 $(TOLDIR)/opt-case.pl 
$(DESTDIR)$(INS_BINARY_DIR)/opt-case.pl
+-      @$(INSTALL) -m 755 $(TOLDIR)/tinews.pl 
$(DESTDIR)$(INS_BINARY_DIR)/tinews.pl
 -      @$(INSTALL) -m 444 $(DOCDIR)/w2r.1 
$(DESTDIR)$(INS_MANUAL_DIR)$(INS_MANUAL_EXT)/w2r.pl.$(INS_MANUAL_EXT)
 -      @$(INSTALL) -m 444 $(DOCDIR)/opt-case.1 
$(DESTDIR)$(INS_MANUAL_DIR)$(INS_MANUAL_EXT)/opt-case.pl.$(INS_MANUAL_EXT)
+-      @$(INSTALL) -m 444 $(DOCDIR)/tinews.1 
$(DESTDIR)$(INS_MANUAL_DIR)$(INS_MANUAL_EXT)/tinews.pl.$(INS_MANUAL_EXT)
 +      @-if test ! -f $(DESTDIR)$(INS_DATA_DIR)/url_handler.sh ; then 
$(INSTALL) -m 755 $(TOLDIR)/url_handler.sh 
$(DESTDIR)$(INS_DATA_DIR)/url_handler.sh ; fi
 +      @-if test ! -f $(DESTDIR)$(INS_DATA_DIR)/metamutt ; then $(INSTALL) -m 
755 $(TOLDIR)/metamutt $(DESTDIR)$(INS_DATA_DIR)/metamutt ; fi
 +      @$(INSTALL) -m 755 $(TOLDIR)/w2r.pl $(DESTDIR)$(INS_DATA_DIR)/w2r.pl
 +      @$(INSTALL) -m 755 $(TOLDIR)/opt-case.pl 
$(DESTDIR)$(INS_DATA_DIR)/opt-case.pl
++      @$(INSTALL) -m 755 $(TOLDIR)/tinews.pl 
$(DESTDIR)$(INS_DATA_DIR)/tinews.pl
 +      @$(INSTALL) -m 444 $(DOCDIR)/w2r.1 
$(DESTDIR)$(INS_DATA_DIR)/w2r.pl.$(INS_MANUAL_EXT)
 +      @$(INSTALL) -m 444 $(DOCDIR)/opt-case.1 
$(DESTDIR)$(INS_DATA_DIR)/opt-case.pl.$(INS_MANUAL_EXT)
++      @$(INSTALL) -m 444 $(DOCDIR)/tinews.1 
$(DESTDIR)$(INS_DATA_DIR)/tinews.pl.$(INS_MANUAL_EXT)
  
- uninstall_helpers:
+ uninstall_helpers :
        @$(ECHO) "Removing helper applications ..."
+@@ -474,7 +475,7 @@ makecfg$X :                        $(SRCDIR)/makecfg.c \
+                               $(INCDIR)/tin.h \
+                               ../include/autoconf.h
+ @SHOW_CC@
+-      @[EMAIL PROTECTED](BUILD_CC) $(BUILD_CFLAGS) $(BUILD_CPPFLAGS) 
$(BUILD_LDFLAGS) -o $@ makecfg.c #$(BUILD_LIBS)
++      @[EMAIL PROTECTED](BUILD_CC) $(BUILD_CFLAGS) $(BUILD_CPPFLAGS) 
$(BUILD_LDFLAGS) -o $@ makecfg.c $(BUILD_LIBS)
+ 
+ TIN_DEP       = \
+       Makefile \
diff -Nurd /usr/ports/news/tin/pkg/PLIST /usr/ports/mystuff/news/tin/pkg/PLIST
--- /usr/ports/news/tin/pkg/PLIST       Wed Sep 15 20:21:22 2004
+++ /usr/ports/mystuff/news/tin/pkg/PLIST       Sun Jan  7 19:04:37 2007
@@ -10,12 +10,20 @@
 share/locale/en_GB/LC_MESSAGES/tin.mo
 share/locale/et/LC_MESSAGES/tin.mo
 share/locale/fr/LC_MESSAGES/tin.mo
+share/locale/ru/LC_MESSAGES/tin.mo
+share/locale/rw/
+share/locale/rw/LC_MESSAGES/
+share/locale/rw/LC_MESSAGES/tin.mo
+share/locale/sv/LC_MESSAGES/tin.mo
+share/locale/tr/LC_MESSAGES/tin.mo
 share/tin/
 share/tin/metamutt
 share/tin/mime.types
 share/tin/opt-case.pl
 share/tin/opt-case.pl.1
 share/tin/tin.defaults
+share/tin/tinews.pl
+share/tin/tinews.pl.1
 share/tin/url_handler.sh
 share/tin/w2r.pl
 share/tin/w2r.pl.1

Reply via email to