Apparently this bug was reported in 2004.

http://sourceforge.net/tracker/?func=detail&aid=981781&group_id=221&atid=100221

I went ahead and posted a patch there for the next version (if
there is one).

-Nate

On Thu, Aug 26, 2010 at 12:01:25AM +0100, Stuart Henderson wrote:
> On 2010/08/25 18:28, Nathan Houghton wrote:
> > As far as I can tell from the OpenBSD web cvs, this port has not
> > changed in a long while (years). I can try contacting the rxvt
> > folks, but the project seems to be pretty dead lately (as far as I
> > can see on the sourceforge page).
> 
> I agree, no substantial changes since 2004. I think this should go
> in. Any ok's?
> 
> 
> Index: Makefile
> ===================================================================
> RCS file: /cvs/ports/x11/rxvt/Makefile,v
> retrieving revision 1.41
> diff -u -p -r1.41 Makefile
> --- Makefile  30 Jul 2010 06:59:00 -0000      1.41
> +++ Makefile  25 Aug 2010 22:58:25 -0000
> @@ -6,9 +6,9 @@ COMMENT-clock=        analog clock for X11 w/ap
>  VER=         2.7.10
>  DISTNAME=    rxvt-${VER}
>  PKGNAME-main=        rxvt-${VER}
> -REVISION-main=       4
> +REVISION-main=       5
>  PKGNAME-clock=       rclock-${VER}
> -REVISION-clock=      0
> +REVISION-clock=      1
>  CATEGORIES=  x11
>  MASTER_SITES=        ${MASTER_SITE_SOURCEFORGE:=rxvt/} \
>               ftp://ftp.fu-berlin.de/unix/X11/terms/rxvt/ \
> @@ -19,7 +19,7 @@ HOMEPAGE=   http://www.rxvt.org/
>  
>  MAINTAINER=  Jason Ish <[email protected]>
>  
> -# GPL
> +# GPLv2+
>  PERMIT_PACKAGE_CDROM=        Yes
>  PERMIT_PACKAGE_FTP=  Yes
>  PERMIT_DISTFILES_CDROM=      Yes
> Index: patches/patch-src_init_c
> ===================================================================
> RCS file: patches/patch-src_init_c
> diff -N patches/patch-src_init_c
> --- /dev/null 1 Jan 1970 00:00:00 -0000
> +++ patches/patch-src_init_c  25 Aug 2010 22:58:25 -0000
> @@ -0,0 +1,26 @@
> +$OpenBSD$
> +
> +avoid a race commonly hit by tiling WMs
> +
> +--- src/init.c.orig  Wed Aug 25 23:52:24 2010
> ++++ src/init.c       Wed Aug 25 23:53:06 2010
> +@@ -1316,6 +1316,9 @@ rxvt_run_command(rxvt_t *r, const char *const
> *argv)
> + /* need to trap SIGURG for SVR4 (Unixware) rlogin */
> + /* signal (SIGURG, SIG_DFL); */
> + 
> ++/* set window size */
> ++    rxvt_tt_winsize(STDIN_FILENO, r->TermWin.ncol, r->TermWin.nrow);
> ++
> + #ifndef __QNX__
> + /* spin off the command interpreter */
> +     switch (r->h->cmd_pid = fork()) {
> +@@ -1398,9 +1401,6 @@ rxvt_run_child(rxvt_t *r, const char *const
> *argv)
> +     }
> + #endif                              /* SRIOCSREDIR */
> +     }
> +-
> +-    /* set window size */
> +-    rxvt_tt_winsize(STDIN_FILENO, r->TermWin.ncol, r->TermWin.nrow);
> + 
> + /* reset signals and spin off the command interpreter */
> +     signal(SIGINT, SIG_DFL);
> Index: pkg/PLIST-clock
> ===================================================================
> RCS file: /cvs/ports/x11/rxvt/pkg/PLIST-clock,v
> retrieving revision 1.2
> diff -u -p -r1.2 PLIST-clock
> --- pkg/PLIST-clock   15 Sep 2004 19:26:31 -0000      1.2
> +++ pkg/PLIST-clock   25 Aug 2010 22:58:25 -0000
> @@ -1,3 +1,3 @@
>  @comment $OpenBSD: PLIST-clock,v 1.2 2004/09/15 19:26:31 espie Exp $
> -bin/rclock
> +...@bin bin/rclock
>  @man man/man1/rclock.1
> Index: pkg/PLIST-main
> ===================================================================
> RCS file: /cvs/ports/x11/rxvt/pkg/PLIST-main,v
> retrieving revision 1.1
> diff -u -p -r1.1 PLIST-main
> --- pkg/PLIST-main    21 Nov 2006 21:04:56 -0000      1.1
> +++ pkg/PLIST-main    25 Aug 2010 22:58:25 -0000
> @@ -1,6 +1,6 @@
>  @comment $OpenBSD: PLIST-main,v 1.1 2006/11/21 21:04:56 espie Exp $
>  @pkgpath x11/rxvt
>  @mode 4755
> -bin/rxvt
> +...@bin bin/rxvt
>  @mode
>  @man man/man1/rxvt.1
> 

Reply via email to