From nobody@FreeBSD.org  Thu Oct 25 15:07:24 2007
Return-Path: <nobody@FreeBSD.org>
Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34])
	by hub.freebsd.org (Postfix) with ESMTP id 1EF1516A417
	for <freebsd-gnats-submit@FreeBSD.org>; Thu, 25 Oct 2007 15:07:24 +0000 (UTC)
	(envelope-from nobody@FreeBSD.org)
Received: from www.freebsd.org (www.freebsd.org [IPv6:2001:4f8:fff6::21])
	by mx1.freebsd.org (Postfix) with ESMTP id 1B98F13C4A3
	for <freebsd-gnats-submit@FreeBSD.org>; Thu, 25 Oct 2007 15:07:24 +0000 (UTC)
	(envelope-from nobody@FreeBSD.org)
Received: from www.freebsd.org (localhost [127.0.0.1])
	by www.freebsd.org (8.14.1/8.14.1) with ESMTP id l9PF6stw029861
	for <freebsd-gnats-submit@FreeBSD.org>; Thu, 25 Oct 2007 15:06:54 GMT
	(envelope-from nobody@www.freebsd.org)
Received: (from nobody@localhost)
	by www.freebsd.org (8.14.1/8.14.1/Submit) id l9PF6smx029860;
	Thu, 25 Oct 2007 15:06:54 GMT
	(envelope-from nobody)
Message-Id: <200710251506.l9PF6smx029860@www.freebsd.org>
Date: Thu, 25 Oct 2007 15:06:54 GMT
From: Pietro Cerutti <gahr@gahr.ch>
To: freebsd-gnats-submit@FreeBSD.org
Subject: [patch] sysutils/screen enable installation of man page
X-Send-Pr-Version: www-3.1
X-GNATS-Notify:

>Number:         117488
>Category:       ports
>Synopsis:       [patch] sysutils/screen enable installation of man page
>Confidential:   no
>Severity:       serious
>Priority:       medium
>Responsible:    cy
>State:          closed
>Quarter:        
>Keywords:       
>Date-Required:  
>Class:          change-request
>Submitter-Id:   current-users
>Arrival-Date:   Thu Oct 25 15:10:02 UTC 2007
>Closed-Date:    Thu Oct 25 18:02:48 UTC 2007
>Last-Modified:  Sat Oct 27 17:50:01 UTC 2007
>Originator:     Pietro Cerutti
>Release:        6.2-STABLE
>Organization:
Bern University of Applied Sciences
>Environment:
FreeBSD gahrtop.localhost 6.2-STABLE FreeBSD 6.2-STABLE #3: Wed Oct 24 08:30:39 CEST 2007     root@gahrtop.localhost:/usr/obj/usr/src/sys/MSI1034  i386

>Description:
sysutils/screen doesn't handle the MAN option correctly: the man page is never installed
>How-To-Repeat:
cd /usr/ports/sysutils/screen
make WITH_MAN=yes install
find /usr/local/man/man1/ -name "screen*"
>Fix:
Apply this patch to the port's Makefile.
Note: a similar problem exists with the INFO option: the INFO doesn't get installed. I'm planning to solve that issue in a future PR, but since INFO pages seem to be much less used than MAN pages, I think this patch deserves higher attention.

--- Makefile.orig       2007-10-25 17:00:50.000000000 +0200 +++ Makefile    2007-10-25 16:58:22.000000000 +0200
@@ -24,11 +24,6 @@
                XTERM_256 "Enable support for 256 colour xterm" OFF

 GNU_CONFIGURE= yes
-.if defined(WITH_MAN)
-MAN1=          screen.1
-.else
-MAKE_ARGS+=    -DNO_MAN
-.endif
 .if defined(WITH_INFO)
 .if !exists(/usr/bin/install-info)
 RUN_DEPENDS+=  install-info:${PORTSDIR}/print/texinfo
@@ -42,6 +37,12 @@

 .include <bsd.port.pre.mk>

+.if defined(WITH_MAN) +MAN1=          screen.1
+.else
+MAKE_ARGS+=    -DNO_MAN
+.endif
+
 .if ${ARCH} == "alpha"
 CFLAGS+=       -O0
 .endif @@ -72,5 +73,8 @@
        @if [ ! -f ${PREFIX}/etc/screenrc ]; then \                 ${CP} ${EXAMPLESDIR}/screenrc ${PREFIX}/etc/screenrc ; \
        fi
+.if defined(WITH_MAN)
+       ${INSTALL_MAN} ${WRKSRC}/doc/screen.1 ${MANPREFIX}/man/man1
+.endif

 .include <bsd.port.post.mk>


>Release-Note:
>Audit-Trail:
Responsible-Changed-From-To: freebsd-ports-bugs->cy 
Responsible-Changed-By: edwin 
Responsible-Changed-When: Thu Oct 25 15:10:12 UTC 2007 
Responsible-Changed-Why:  
Over to maintainer (via the GNATS Auto Assign Tool) 

http://www.freebsd.org/cgi/query-pr.cgi?pr=117488 
State-Changed-From-To: open->analyzed 
State-Changed-By: cy 
State-Changed-When: Thu Oct 25 17:18:35 UTC 2007 
State-Changed-Why:  
The problem is more serious than man pages not being installed. Info 
files are not installed either. Testing a patch locally to resolve this 
and the info issue. 

http://www.freebsd.org/cgi/query-pr.cgi?pr=117488 
State-Changed-From-To: analyzed->closed 
State-Changed-By: cy 
State-Changed-When: Thu Oct 25 18:00:54 UTC 2007 
State-Changed-Why:  
Repaired the Makefile to install man pages (as per this PR) and info 
documents. 

http://www.freebsd.org/cgi/query-pr.cgi?pr=117488 

From: dfilter@FreeBSD.ORG (dfilter service)
To: bug-followup@FreeBSD.org
Cc:  
Subject: Re: ports/117488: commit references a PR
Date: Thu, 25 Oct 2007 18:01:59 +0000 (UTC)

 cy          2007-10-25 18:01:51 UTC
 
   FreeBSD ports repository
 
   Modified files:
     sysutils/screen      Makefile 
   Log:
   Fix (enable) installation of man pages and info documents.
   
   PR:             117488
   
   Revision  Changes    Path
   1.68      +8 -4      ports/sysutils/screen/Makefile
 _______________________________________________
 cvs-all@freebsd.org mailing list
 http://lists.freebsd.org/mailman/listinfo/cvs-all
 To unsubscribe, send any mail to "cvs-all-unsubscribe@freebsd.org"
 

From: Pietro Cerutti <gahr@gahr.ch>
To: cy@FreeBSD.org,  bug-followup@FreeBSD.org
Cc:  
Subject: Re: ports/117488: [patch] sysutils/screen enable installation of
 man page
Date: Fri, 26 Oct 2007 10:08:37 +0200

 This is an OpenPGP/MIME signed message (RFC 2440 and 3156)
 --------------enig045BC6F740992BBB42E9963D
 Content-Type: text/plain; charset=UTF-8
 Content-Transfer-Encoding: quoted-printable
 
 cy@FreeBSD.org wrote:
 > Synopsis: [patch] sysutils/screen enable installation of man page
 >=20
 > State-Changed-From-To: analyzed->closed
 > State-Changed-By: cy
 > State-Changed-When: Thu Oct 25 18:00:54 UTC 2007
 > State-Changed-Why:=20
 > Repaired the Makefile to install man pages (as per this PR) and info
 > documents.
 >=20
 > http://www.freebsd.org/cgi/query-pr.cgi?pr=3D117488
 
 Have you tested it?
 The INFO part doesn't work.
 
 Your solution seems to be logical: enable the .if defined(WITH_INFO)
 part by including <bsd.port.pre.mk> before the directive.
 It was also my first thought. However I cannot get the INFO part to work.=
 
 
 I suggest that you check the output below, and revert to my original
 patch, which solves the MAN part at least, until we get to a solution
 for the INFO part.
 
 Thank you,
 
 
 =3D=3D=3D>  Installing for screen-4.0.3_1
 =3D=3D=3D>   Generating temporary packing list
 =3D=3D=3D>  Checking if sysutils/screen already installed
 =2E/etc/mkinstalldirs /usr/local/bin `sed < config.h -n -e '/define
 SCREENENCODINGS/s/^.*"\([^"]*\)"/\1/p'`
 cd doc ; make  installdirs
 =2E/../etc/mkinstalldirs /usr/local/man/man1
 install  -s -o root -g wheel -m 555 screen /usr/local/bin/screen
 chown root:wheel /usr/local/bin/screen && chmod 4755 /usr/local/bin/scree=
 n
 cp ./utf8encodings/?? `sed < config.h -n -e '/define
 SCREENENCODINGS/s/^.*"\([^"]*\)"/\1/p'`
 cd doc ; make  install
 =2E/../etc/mkinstalldirs /usr/local/man/man1
 install  -o root -g wheel -m 444 ./screen.1 /usr/local/man/man1/screen.1
 if [ -d /usr/lib/terminfo ]; then  PATH=3D"$PATH:/usr/5bin" tic
 =2E/terminfo/screeninfo.src;  chmod 644 /usr/lib/terminfo/s/screen*;  fi
 termcap entry (./terminfo/screencap) should be installed manually.
 You may also want to install ./etc/etcscreenrc in /usr/local/etc/screenrc=
 
 install  -o root -g wheel -m 444
 /usr/ports/sysutils/screen/work/screen-4.0.3/doc/screen.1
 /usr/local/man/man1
 install-info --quiet /usr/local/info/screen.info /usr/local/info/dir
 install-info: No such file or directory for /usr/local/info/screen.info
 *** Error code 1
 
 Stop in /usr/ports/sysutils/screen.
 *** Error code 1
 
 Stop in /usr/ports/sysutils/screen.
 Exit 1
 
 
 --=20
 Pietro Cerutti
 
 PGP Public Key:
 http://gahr.ch/pgp
 
 
 --------------enig045BC6F740992BBB42E9963D
 Content-Type: application/pgp-signature; name="signature.asc"
 Content-Description: OpenPGP digital signature
 Content-Disposition: attachment; filename="signature.asc"
 
 -----BEGIN PGP SIGNATURE-----
 Version: GnuPG v1.4.7 (FreeBSD)
 
 iD8DBQFHIaCJwMJqmJVx944RCnXwAJ9XQsYUJytVKl/XCIPHUeHJyfxLlACfQgSX
 Bg2pVUQxi/tdTiE3yqfcyG8=
 =hBgB
 -----END PGP SIGNATURE-----
 
 --------------enig045BC6F740992BBB42E9963D--

From: Pietro Cerutti <gahr@gahr.ch>
To: cy@FreeBSD.org,  bug-followup@FreeBSD.org
Cc:  
Subject: Re: ports/117488: [patch] sysutils/screen enable installation of
 man page
Date: Fri, 26 Oct 2007 10:48:30 +0200

 This is an OpenPGP/MIME signed message (RFC 2440 and 3156)
 --------------enig14E192FFECFACCFF222FF981
 Content-Type: text/plain; charset=UTF-8
 Content-Transfer-Encoding: quoted-printable
 
 Please apply this patch to the Makefile you committed yesterday (1.68).
 It finally solves the INFO problem..
 
 
 --- Makefile.orig       2007-10-26 10:42:30.000000000 +0200
 +++ Makefile    2007-10-26 10:43:25.000000000 +0200
 @@ -31,16 +31,14 @@
  GNU_CONFIGURE=3D yes
  .if defined(WITH_MAN)
  MAN1=3D          screen.1
 -.else
 -MAKE_ARGS+=3D    -DNO_MAN
 +MAKE_ARGS+=3D    -DWITH_MAN
  .endif
  .if defined(WITH_INFO)
  .if !exists(/usr/bin/install-info)
  RUN_DEPENDS+=3D  install-info:${PORTSDIR}/print/texinfo
  .endif
  INFO=3D          screen
 -.else
 -MAKE_ARGS+=3D    -DNO_INFO
 +MAKE_ARGS+=3D    -DWITH_INFO
  .endif
 
  .if ${ARCH} =3D=3D "alpha"
 
 
 Thanks, regards.
 
 --=20
 Pietro Cerutti
 
 PGP Public Key:
 http://gahr.ch/pgp
 
 
 --------------enig14E192FFECFACCFF222FF981
 Content-Type: application/pgp-signature; name="signature.asc"
 Content-Description: OpenPGP digital signature
 Content-Disposition: attachment; filename="signature.asc"
 
 -----BEGIN PGP SIGNATURE-----
 Version: GnuPG v1.4.7 (FreeBSD)
 
 iD8DBQFHIaniwMJqmJVx944RCn0xAJ0QBdtF5gljTYIO11xj5MKW6oSPWQCeNd0b
 lFUPaVdQ+zsduQNA22AWXyo=
 =SABO
 -----END PGP SIGNATURE-----
 
 --------------enig14E192FFECFACCFF222FF981--

From: Pietro Cerutti <gahr@gahr.ch>
To: cy@FreeBSD.org,  bug-followup@FreeBSD.org
Cc:  
Subject: Re: ports/117488: [patch] sysutils/screen enable installation of
 man page
Date: Fri, 26 Oct 2007 10:55:04 +0200

 This is an OpenPGP/MIME signed message (RFC 2440 and 3156)
 --------------enig9B1EF4A280AD1C13765DAA48
 Content-Type: text/plain; charset=UTF-8
 Content-Transfer-Encoding: quoted-printable
 
 Pietro Cerutti wrote:
 > Please apply this patch to the Makefile you committed yesterday (1.68).=
 
 > It finally solves the INFO problem..
 >=20
 >=20
 > --- Makefile.orig       2007-10-26 10:42:30.000000000 +0200
 > +++ Makefile    2007-10-26 10:43:25.000000000 +0200
 > @@ -31,16 +31,14 @@
 >  GNU_CONFIGURE=3D yes
 >  .if defined(WITH_MAN)
 >  MAN1=3D          screen.1
 > -.else
 > -MAKE_ARGS+=3D    -DNO_MAN
 > +MAKE_ARGS+=3D    -DWITH_MAN
 >  .endif
 >  .if defined(WITH_INFO)
 >  .if !exists(/usr/bin/install-info)
 >  RUN_DEPENDS+=3D  install-info:${PORTSDIR}/print/texinfo
 >  .endif
 >  INFO=3D          screen
 > -.else
 > -MAKE_ARGS+=3D    -DNO_INFO
 > +MAKE_ARGS+=3D    -DWITH_INFO
 >  .endif
 >=20
 >  .if ${ARCH} =3D=3D "alpha"
 >=20
 >=20
 > Thanks, regards.
 >=20
 
 
 Actually, the installation of the MAN page is taken care of in
 doc/Makefile. We can safely remove the explicit installation from the
 port's Makefile.
 
 The following is the actual patch to be applied against yesterday's 1.68:=
 
 
 --- Makefile.orig       2007-10-26 10:42:30.000000000 +0200
 +++ Makefile    2007-10-26 10:52:19.000000000 +0200
 @@ -31,16 +31,14 @@
  GNU_CONFIGURE=3D yes
  .if defined(WITH_MAN)
  MAN1=3D          screen.1
 -.else
 -MAKE_ARGS+=3D    -DNO_MAN
 +MAKE_ARGS+=3D    -DWITH_MAN
  .endif
  .if defined(WITH_INFO)
  .if !exists(/usr/bin/install-info)
  RUN_DEPENDS+=3D  install-info:${PORTSDIR}/print/texinfo
  .endif
  INFO=3D          screen
 -.else
 -MAKE_ARGS+=3D    -DNO_INFO
 +MAKE_ARGS+=3D    -DWITH_INFO
  .endif
 
  .if ${ARCH} =3D=3D "alpha"
 @@ -73,8 +71,5 @@
         @if [ ! -f ${PREFIX}/etc/screenrc ]; then \
                 ${CP} ${EXAMPLESDIR}/screenrc ${PREFIX}/etc/screenrc ; \
         fi
 -.if defined(WITH_MAN)
 -       ${INSTALL_MAN} ${WRKSRC}/doc/screen.1 ${MANPREFIX}/man/man1
 -.endif
 
  .include <bsd.port.post.mk>
 
 --=20
 Pietro Cerutti
 
 PGP Public Key:
 http://gahr.ch/pgp
 
 
 --------------enig9B1EF4A280AD1C13765DAA48
 Content-Type: application/pgp-signature; name="signature.asc"
 Content-Description: OpenPGP digital signature
 Content-Disposition: attachment; filename="signature.asc"
 
 -----BEGIN PGP SIGNATURE-----
 Version: GnuPG v1.4.7 (FreeBSD)
 
 iD8DBQFHIattwMJqmJVx944RCkaFAKCjbZzttzNE4lTKGvNo2UCtCuCQGQCfe3cG
 RyoxjY4HfX4EpXSHrsYp6DM=
 =XJ4w
 -----END PGP SIGNATURE-----
 
 --------------enig9B1EF4A280AD1C13765DAA48--

From: Cy Schubert <Cy.Schubert@komquats.com>
To: Pietro Cerutti <gahr@gahr.ch>
Cc: bug-followup@FreeBSD.org
Subject: Re: ports/117488: [patch] sysutils/screen enable installation of man
 page
Date: Fri, 26 Oct 2007 09:10:29 -0700

 In message <4721A9DE.5010404@gahr.ch>, Pietro Cerutti writes:
 > This is an OpenPGP/MIME signed message (RFC 2440 and 3156)
 > --------------enig14E192FFECFACCFF222FF981
 > Content-Type: text/plain; charset=UTF-8
 > Content-Transfer-Encoding: quoted-printable
 > 
 > Please apply this patch to the Makefile you committed yesterday (1.68).
 > It finally solves the INFO problem..
 > 
 > 
 > --- Makefile.orig       2007-10-26 10:42:30.000000000 +0200
 > +++ Makefile    2007-10-26 10:43:25.000000000 +0200
 > @@ -31,16 +31,14 @@
 >  GNU_CONFIGURE=3D yes
 >  .if defined(WITH_MAN)
 >  MAN1=3D          screen.1
 > -.else
 > -MAKE_ARGS+=    -DNO_MAN
 > +MAKE_ARGS+=    -DWITH_MAN
 >  .endif
 >  .if defined(WITH_INFO)
 >  .if !exists(/usr/bin/install-info)
 >  RUN_DEPENDS+=3D  install-info:${PORTSDIR}/print/texinfo
 >  .endif
 >  INFO=3D          screen
 > -.else
 > -MAKE_ARGS+=    -DNO_INFO
 > +MAKE_ARGS+=    -DWITH_INFO
 >  .endif
 > 
 >  .if ${ARCH} == "alpha"
 
 Your patch is incorrect. NO_MAN and NO_INFO are used when WITH_MAN and WITH_INFO are not respectively defined.
 
 
 -- 
 Cheers,
 Cy Schubert <Cy.Schubert@komquats.com>
 FreeBSD UNIX:  <cy@FreeBSD.org>   Web:  http://www.FreeBSD.org
 
 			e**(i*pi)+1=0
 
 

From: Cy Schubert <Cy.Schubert@komquats.com>
To: Pietro Cerutti <gahr@gahr.ch>
Cc: Cy Schubert <Cy.Schubert@komquats.com>,
 bug-followup <bug-followup@freebsd.org>
Subject: Re: ports/117488: [patch] sysutils/screen enable installation of man
 page
Date: Fri, 26 Oct 2007 09:28:02 -0700

 In message <4722143A.5000007@gahr.ch>, Pietro Cerutti writes:
 > This is an OpenPGP/MIME signed message (RFC 2440 and 3156)
 > --------------enigC8EB130EC3F13747B4605930
 > Content-Type: text/plain; charset=ISO-8859-1
 > Content-Transfer-Encoding: quoted-printable
 > 
 > Cy Schubert wrote:
 > > In message <4721A9DE.5010404@gahr.ch>, Pietro Cerutti writes:
 > >> This is an OpenPGP/MIME signed message (RFC 2440 and 3156)
 > >> --------------enig14E192FFECFACCFF222FF981
 > >> Content-Type: text/plain; charset=3DUTF-8
 > >> Content-Transfer-Encoding: quoted-printable
 > >>
 > >> Please apply this patch to the Makefile you committed yesterday (1.68)=
 > =2E
 > >> It finally solves the INFO problem..
 > >>
 > >>
 > >> --- Makefile.orig       2007-10-26 10:42:30.000000000 +0200
 > >> +++ Makefile    2007-10-26 10:43:25.000000000 +0200
 > >> @@ -31,16 +31,14 @@
 > >>  GNU_CONFIGURE=3D3D yes
 > >>  .if defined(WITH_MAN)
 > >>  MAN1=3D3D          screen.1
 > >> -.else
 > >> -MAKE_ARGS+=3D    -DNO_MAN
 > >> +MAKE_ARGS+=3D    -DWITH_MAN
 > >>  .endif
 > >>  .if defined(WITH_INFO)
 > >>  .if !exists(/usr/bin/install-info)
 > >>  RUN_DEPENDS+=3D3D  install-info:${PORTSDIR}/print/texinfo
 > >>  .endif
 > >>  INFO=3D3D          screen
 > >> -.else
 > >> -MAKE_ARGS+=3D    -DNO_INFO
 > >> +MAKE_ARGS+=3D    -DWITH_INFO
 > >>  .endif
 > >>
 > >>  .if ${ARCH} =3D=3D "alpha"
 > >=20
 > > Your patch is incorrect. NO_MAN and NO_INFO are used when WITH_MAN and =
 > WITH_INFO are not respectively defined.
 > >=20
 > 
 > The Makefile in doc/ doesn't care about NO_INFO, but checks for
 > WITH_INFO. That's why, without the patch above, the INFO page doesn't
 > get installed.
 > I recognize that it's a dirty hack, do you have a better solution?
 > 
 
 cwsys$ pkg_info -L screen-4.0.3_1
 Information for screen-4.0.3_1:
 
 Files:
 /usr/local/man/man1/screen.1.gz
 /usr/local/bin/screen
 /usr/local/share/screen/utf8encodings/01
 /usr/local/share/screen/utf8encodings/02
 /usr/local/share/screen/utf8encodings/03
 /usr/local/share/screen/utf8encodings/04
 /usr/local/share/screen/utf8encodings/18
 /usr/local/share/screen/utf8encodings/19
 /usr/local/share/screen/utf8encodings/a1
 /usr/local/share/screen/utf8encodings/bf
 /usr/local/share/screen/utf8encodings/c2
 /usr/local/share/screen/utf8encodings/c3
 /usr/local/share/screen/utf8encodings/c4
 /usr/local/share/screen/utf8encodings/c6
 /usr/local/share/screen/utf8encodings/c7
 /usr/local/share/screen/utf8encodings/c8
 /usr/local/share/screen/utf8encodings/cc
 /usr/local/share/screen/utf8encodings/cd
 /usr/local/share/screen/utf8encodings/d6
 /usr/local/share/examples/screen/screenrc
 /usr/local/info/screen.info
 
 cwsys$ ll /usr/local/man/man1/screen.1.gz /usr/local/info/screen.info
 -r--r--r--  1 root  wheel  225811 Oct 26 09:21 /usr/local/info/screen.info
 -r--r--r--  1 root  wheel   49066 Oct 26 09:21 /usr/local/man/man1/screen.1.gz
 cwsys$ 
 
 No installation problems on my production server. Screen options should be similar to the following. Are yours the same?
 
 cwsys$ cat /var/db/ports/screen/options 
 # This file is auto-generated by 'make config'.
 # No user-servicable parts inside!
 # Options for screen-4.0.3
 _OPTIONS_READ=screen-4.0.3
 WITH_XTERM_256=true
 WITH_NETHACK=true
 WITHOUT_CJK=true
 WITH_MAN=true
 WITH_INFO=true
 cwsys$ 
 
 
 -- 
 Cheers,
 Cy Schubert <Cy.Schubert@komquats.com>
 FreeBSD UNIX:  <cy@FreeBSD.org>   Web:  http://www.FreeBSD.org
 
 			e**(i*pi)+1=0
 
 

From: Pietro Cerutti <gahr@gahr.ch>
To: bug-followup@FreeBSD.org
Cc:  
Subject: Re: ports/117488: [patch] sysutils/screen enable installation of
 man page
Date: Fri, 26 Oct 2007 18:45:50 +0200

 This is an OpenPGP/MIME signed message (RFC 2440 and 3156)
 --------------enig45F7F02B8760C11E5BEF1D0A
 Content-Type: multipart/mixed;
  boundary="------------080301040802070400050107"
 
 This is a multi-part message in MIME format.
 --------------080301040802070400050107
 Content-Type: text/plain; charset=ISO-8859-1
 Content-Transfer-Encoding: quoted-printable
 
 Cy Schubert wrote:
 > In message <4722143A.5000007@gahr.ch>, Pietro Cerutti writes:
 >> This is an OpenPGP/MIME signed message (RFC 2440 and 3156)
 >> --------------enigC8EB130EC3F13747B4605930
 >> Content-Type: text/plain; charset=3DISO-8859-1
 >> Content-Transfer-Encoding: quoted-printable
 >>
 >> Cy Schubert wrote:
 >>> In message <4721A9DE.5010404@gahr.ch>, Pietro Cerutti writes:
 >>>> This is an OpenPGP/MIME signed message (RFC 2440 and 3156)
 >>>> --------------enig14E192FFECFACCFF222FF981
 >>>> Content-Type: text/plain; charset=3D3DUTF-8
 >>>> Content-Transfer-Encoding: quoted-printable
 >>>>
 >>>> Please apply this patch to the Makefile you committed yesterday (1.6=
 8)=3D
 >> =3D2E
 >>>> It finally solves the INFO problem..
 >>>>
 >>>>
 >>>> --- Makefile.orig       2007-10-26 10:42:30.000000000 +0200
 >>>> +++ Makefile    2007-10-26 10:43:25.000000000 +0200
 >>>> @@ -31,16 +31,14 @@
 >>>>  GNU_CONFIGURE=3D3D3D yes
 >>>>  .if defined(WITH_MAN)
 >>>>  MAN1=3D3D3D          screen.1
 >>>> -.else
 >>>> -MAKE_ARGS+=3D3D    -DNO_MAN
 >>>> +MAKE_ARGS+=3D3D    -DWITH_MAN
 >>>>  .endif
 >>>>  .if defined(WITH_INFO)
 >>>>  .if !exists(/usr/bin/install-info)
 >>>>  RUN_DEPENDS+=3D3D3D  install-info:${PORTSDIR}/print/texinfo
 >>>>  .endif
 >>>>  INFO=3D3D3D          screen
 >>>> -.else
 >>>> -MAKE_ARGS+=3D3D    -DNO_INFO
 >>>> +MAKE_ARGS+=3D3D    -DWITH_INFO
 >>>>  .endif
 >>>>
 >>>>  .if ${ARCH} =3D3D=3D3D "alpha"
 >>> =3D20
 >>> Your patch is incorrect. NO_MAN and NO_INFO are used when WITH_MAN an=
 d =3D
 >> WITH_INFO are not respectively defined.
 >>> =3D20
 >> The Makefile in doc/ doesn't care about NO_INFO, but checks for
 >> WITH_INFO. That's why, without the patch above, the INFO page doesn't
 >> get installed.
 >> I recognize that it's a dirty hack, do you have a better solution?
 >>
 >=20
 > cwsys$ pkg_info -L screen-4.0.3_1
 > Information for screen-4.0.3_1:
 >=20
 > Files:
 > /usr/local/man/man1/screen.1.gz
 > /usr/local/bin/screen
 > /usr/local/share/screen/utf8encodings/01
 > /usr/local/share/screen/utf8encodings/02
 > /usr/local/share/screen/utf8encodings/03
 > /usr/local/share/screen/utf8encodings/04
 > /usr/local/share/screen/utf8encodings/18
 > /usr/local/share/screen/utf8encodings/19
 > /usr/local/share/screen/utf8encodings/a1
 > /usr/local/share/screen/utf8encodings/bf
 > /usr/local/share/screen/utf8encodings/c2
 > /usr/local/share/screen/utf8encodings/c3
 > /usr/local/share/screen/utf8encodings/c4
 > /usr/local/share/screen/utf8encodings/c6
 > /usr/local/share/screen/utf8encodings/c7
 > /usr/local/share/screen/utf8encodings/c8
 > /usr/local/share/screen/utf8encodings/cc
 > /usr/local/share/screen/utf8encodings/cd
 > /usr/local/share/screen/utf8encodings/d6
 > /usr/local/share/examples/screen/screenrc
 > /usr/local/info/screen.info
 >=20
 > cwsys$ ll /usr/local/man/man1/screen.1.gz /usr/local/info/screen.info
 > -r--r--r--  1 root  wheel  225811 Oct 26 09:21 /usr/local/info/screen.i=
 nfo
 > -r--r--r--  1 root  wheel   49066 Oct 26 09:21 /usr/local/man/man1/scre=
 en.1.gz
 > cwsys$=20
 >=20
 > No installation problems on my production server. Screen options should=
  be similar to the following. Are yours the same?
 >=20
 > cwsys$ cat /var/db/ports/screen/options=20
 > # This file is auto-generated by 'make config'.
 > # No user-servicable parts inside!
 > # Options for screen-4.0.3
 > _OPTIONS_READ=3Dscreen-4.0.3
 > WITH_XTERM_256=3Dtrue
 > WITH_NETHACK=3Dtrue
 > WITHOUT_CJK=3Dtrue
 > WITH_MAN=3Dtrue
 > WITH_INFO=3Dtrue
 > cwsys$=20
 >=20
 >=20
 
 Nope...
 See output below.. in the meanwhile, I suggest this one against your 1.69=
 :
 
 --- Makefile    2007-10-26 18:29:38.000000000 +0200
 +++ Makefile.mine       2007-10-26 18:30:41.000000000 +0200
 @@ -31,6 +31,7 @@
  GNU_CONFIGURE=3D yes
  .if defined(WITH_MAN)
  MAN1=3D          screen.1
 +MAKE_ARGS+=3D -DWITH_MAN
  .else
  MAKE_ARGS+=3D    -DNO_MAN
  .endif
 @@ -39,6 +40,7 @@
  RUN_DEPENDS+=3D  install-info:${PORTSDIR}/print/texinfo
  .endif
  INFO=3D          screen
 +MAKE_ARGS+=3D -DWITH_INFO
  .else
  MAKE_ARGS+=3D    -DNO_INFO
  .endif
 
 
 Building with your 1.69...
 
 > cat /var/db/ports/screen/options
 # This file is auto-generated by 'make config'.
 # No user-servicable parts inside!
 # Options for screen-4.0.3_1
 _OPTIONS_READ=3Dscreen-4.0.3_1
 WITH_CJK=3Dtrue
 WITH_INFO=3Dtrue
 WITH_MAN=3Dtrue
 WITH_NETHACK=3Dtrue
 WITH_XTERM_256=3Dtrue
 
 > sudo make install
 =3D=3D=3D>  Found saved configuration for screen-4.0.3_1
 =3D=3D=3D>  Extracting for screen-4.0.3_1
 =3D> MD5 Checksum OK for screen-4.0.3.tar.gz.
 =3D> SHA256 Checksum OK for screen-4.0.3.tar.gz.
 =3D=3D=3D>  Patching for screen-4.0.3_1
 =3D=3D=3D>  Applying extra patch /usr/ports/sysutils/screen/files/opt-cjk=
 width
 =3D=3D=3D>  Applying FreeBSD patches for screen-4.0.3_1
 =3D=3D=3D>  Configuring for screen-4.0.3_1
 configure: WARNING: you should use --build, --host, --target
 this is screen version 4.0.3
 checking for i386-portbld-freebsd6.2-gcc... cc
 checking for C compiler default output... a.out
 checking whether the C compiler works... yes
 checking whether we are cross compiling... no
 checking for suffix of executables...
 checking for suffix of object files... o
 checking whether we are using the GNU C compiler... yes
 checking whether cc accepts -g... yes
 checking for cc option to accept ANSI C... none needed
 checking how to run the C preprocessor... cc -E
 checking for egrep... grep -E
 checking whether cc needs -traditional... no
 checking for library containing strerror... none required
 checking for gawk... gawk
 checking for a BSD-compatible install... /usr/bin/install -c -o root -g
 wheel
 configure: checking for buggy tools...
 checking if a system-wide socket dir should be used... yes
 checking for the socket dir... (eff_uid ? "/tmp/uscreens" : "/tmp/screens=
 ")
 configure: checking for MIPS...
 configure: checking for Ultrix...
 configure: checking for butterfly...
 configure: checking for POSIX.1...
 - you have a POSIX system
 configure: checking for System V...
 configure: checking for sequent/ptx...
 configure: checking SVR4...
 checking for ANSI C header files... no
 checking for sys/types.h... yes
 checking for sys/stat.h... yes
 checking for stdlib.h... yes
 checking for string.h... yes
 checking for memory.h... yes
 checking for strings.h... yes
 checking for inttypes.h... yes
 checking for stdint.h... yes
 checking for unistd.h... yes
 configure: checking for Solaris 2.x...
 configure: checking BSD job jontrol...
 - you have jobcontrol
 configure: checking setreuid...
 configure: checking seteuid...
 configure: checking select...
 configure: checking fifos...
 - your fifos are usable
 configure: checking for broken fifo implementation...
 - your implementation is ok
 configure: checking sockets...
 - your sockets are usable
 configure: checking socket implementation...
 - you are normal
 - both sockets and fifos usable. let's take fifos.
 configure: checking select return value...
 - select is ok
 configure: checking for tgetent...
 configure: checking libtermcap...
 - you use the termcap database
 configure: checking ospeed...
 configure: checking for /dev/ptc...
 configure: checking for SVR4 ptys...
 checking for getpt... no
 checking for openpty... no
 checking for openpty in -lutil... yes
 configure: checking for ptyranges...
 configure: checking default tty permissions/group...
 - ptys are world accessable
 configure: checking getutent...
 configure: checking getutent with -lgen...
 configure: checking ut_host...
 checking utempter.h usability... no
 checking utempter.h presence... no
 checking for utempter.h... no
 configure: checking for libutil(s)...
 configure: checking getloadavg...
 assuming posix signal definition
 configure: checking for crypt and sec libraries...
 configure: checking crypt...
 configure: checking IRIX sun library...
 configure: checking syslog...
 configure: checking wait union...
 configure: checking for termio or termios...
 configure: checking getspnam...
 configure: checking getttyent...
 configure: checking fdwalk...
 configure: checking whether memcpy/memmove/bcopy handles overlapping
 arguments...
 checking long file names... yes
 checking for vsprintf... yes
 checking for dirent.h that defines DIR... yes
 checking for library containing opendir... none required
 checking for setenv... yes
 checking for nl_langinfo(CODESET)... yes
 checking for library containing gethostname... none required
 checking for rename... yes
 checking for fchmod... yes
 checking for fchown... yes
 checking for strerror... yes
 checking for lstat... yes
 checking for _exit... yes
 checking for utimes... yes
 checking for vsnprintf... yes
 checking for getcwd... yes
 checking for setlocale... yes
 checking for strftime... yes
 checking for the global screenrc file... "/usr/local/etc/screenrc"
 checking for the utf8-encodings location...
 "/usr/local/share/screen/utf8encodings"
 configure: creating ./config.status
 config.status: creating Makefile
 config.status: creating doc/Makefile
 config.status: creating config.h
 config.status: executing default commands
 
 Now please check the pathnames in the Makefile and in the user
 configuration section in config.h.
 Then type 'make' to make screen. Good luck.
 
 =3D=3D=3D>  Building for screen-4.0.3_1
 CPP=3D"cc -E " srcdir=3D. sh ./osdef.sh
 AWK=3Dgawk CC=3D"cc -O2 -pipe -ggdb -march=3Dpentiumpro -DCOLORS256" srcd=
 ir=3D.
 sh ./comm.sh
 AWK=3Dgawk srcdir=3D. sh ./term.sh
 cc -c -I. -I.    -O2 -pipe -ggdb -march=3Dpentiumpro -DCOLORS256 screen.c=
 
 cc -c -I. -I.    -O2 -pipe -ggdb -march=3Dpentiumpro -DCOLORS256 ansi.c
 cc -c -I. -I.    -O2 -pipe -ggdb -march=3Dpentiumpro -DCOLORS256 fileio.c=
 
 cc -c -I. -I.    -O2 -pipe -ggdb -march=3Dpentiumpro -DCOLORS256 mark.c
 cc -c -I. -I.    -O2 -pipe -ggdb -march=3Dpentiumpro -DCOLORS256 misc.c
 cc -c -I. -I.    -O2 -pipe -ggdb -march=3Dpentiumpro -DCOLORS256 resize.c=
 
 cc -c -I. -I.    -O2 -pipe -ggdb -march=3Dpentiumpro -DCOLORS256 socket.c=
 
 cc -c -I. -I.    -O2 -pipe -ggdb -march=3Dpentiumpro -DCOLORS256 search.c=
 
 sh ./tty.sh tty.c
 cc -c -I. -I.    -O2 -pipe -ggdb -march=3Dpentiumpro -DCOLORS256 tty.c
 cc -c -I. -I.    -O2 -pipe -ggdb -march=3Dpentiumpro -DCOLORS256 term.c
 cc -c -I. -I.    -O2 -pipe -ggdb -march=3Dpentiumpro -DCOLORS256 window.c=
 
 cc -c -I. -I.    -O2 -pipe -ggdb -march=3Dpentiumpro -DCOLORS256 utmp.c
 cc -c -I. -I.    -O2 -pipe -ggdb -march=3Dpentiumpro -DCOLORS256 loadav.c=
 
 cc -c -I. -I.    -O2 -pipe -ggdb -march=3Dpentiumpro -DCOLORS256 putenv.c=
 
 cc -c -I. -I.    -O2 -pipe -ggdb -march=3Dpentiumpro -DCOLORS256 help.c
 cc -c -I. -I.    -O2 -pipe -ggdb -march=3Dpentiumpro -DCOLORS256 termcap.=
 c
 cc -c -I. -I.    -O2 -pipe -ggdb -march=3Dpentiumpro -DCOLORS256 input.c
 cc -c -I. -I.    -O2 -pipe -ggdb -march=3Dpentiumpro -DCOLORS256 attacher=
 =2Ec
 cc -c -I. -I.    -O2 -pipe -ggdb -march=3Dpentiumpro -DCOLORS256 pty.c
 cc -c -I. -I.    -O2 -pipe -ggdb -march=3Dpentiumpro -DCOLORS256 process.=
 c
 cc -c -I. -I.    -O2 -pipe -ggdb -march=3Dpentiumpro -DCOLORS256 display.=
 c
 cc -c -I. -I.    -O2 -pipe -ggdb -march=3Dpentiumpro -DCOLORS256 comm.c
 cc -c -I. -I.    -O2 -pipe -ggdb -march=3Dpentiumpro -DCOLORS256 kmapdef.=
 c
 cc -c -I. -I.    -O2 -pipe -ggdb -march=3Dpentiumpro -DCOLORS256 acls.c
 cc -c -I. -I.    -O2 -pipe -ggdb -march=3Dpentiumpro -DCOLORS256 braille.=
 c
 cc -c -I. -I.    -O2 -pipe -ggdb -march=3Dpentiumpro -DCOLORS256 braille_=
 tsi.c
 cc -c -I. -I.    -O2 -pipe -ggdb -march=3Dpentiumpro -DCOLORS256 logfile.=
 c
 cc -c -I. -I.    -O2 -pipe -ggdb -march=3Dpentiumpro -DCOLORS256 layer.c
 cc -c -I. -I.    -O2 -pipe -ggdb -march=3Dpentiumpro -DCOLORS256 sched.c
 cc -c -I. -I.    -O2 -pipe -ggdb -march=3Dpentiumpro -DCOLORS256 teln.c
 cc -c -I. -I.    -O2 -pipe -ggdb -march=3Dpentiumpro -DCOLORS256 nethack.=
 c
 cc -c -I. -I.    -O2 -pipe -ggdb -march=3Dpentiumpro -DCOLORS256 encoding=
 =2Ec
 cc  -o screen screen.o ansi.o fileio.o mark.o misc.o resize.o socket.o
 search.o tty.o term.o window.o utmp.o loadav.o putenv.o help.o
 termcap.o input.o attacher.o pty.o process.o display.o comm.o  kmapdef.o
 acls.o braille.o braille_tsi.o logfile.o layer.o  sched.o teln.o
 nethack.o encoding.o -ltermcap  -lutil -lutil -lcrypt
 =3D=3D=3D>  Installing for screen-4.0.3_1
 =3D=3D=3D>   Generating temporary packing list
 =3D=3D=3D>  Checking if sysutils/screen already installed
 =2E/etc/mkinstalldirs /usr/local/bin `sed < config.h -n -e '/define
 SCREENENCODINGS/s/^.*"\([^"]*\)"/\1/p'`
 cd doc ; make  installdirs
 install  -s -o root -g wheel -m 555 screen /usr/local/bin/screen
 chown root:wheel /usr/local/bin/screen && chmod 4755 /usr/local/bin/scree=
 n
 cp ./utf8encodings/?? `sed < config.h -n -e '/define
 SCREENENCODINGS/s/^.*"\([^"]*\)"/\1/p'`
 cd doc ; make  install
 if [ -d /usr/lib/terminfo ]; then  PATH=3D"$PATH:/usr/5bin" tic
 =2E/terminfo/screeninfo.src;  chmod 644 /usr/lib/terminfo/s/screen*;  fi
 termcap entry (./terminfo/screencap) should be installed manually.
 You may also want to install ./etc/etcscreenrc in /usr/local/etc/screenrc=
 
 install-info --quiet /usr/local/info/screen.info /usr/local/info/dir
 install-info: No such file or directory for /usr/local/info/screen.info
 *** Error code 1
 
 Stop in /usr/ports/sysutils/screen.
 *** Error code 1
 
 Stop in /usr/ports/sysutils/screen.
 Exit 1
 
 --=20
 Pietro Cerutti
 
 PGP Public Key:
 http://gahr.ch/pgp
 
 --------------080301040802070400050107
 Content-Type: application/pgp-signature;
  name="signature.asc"
 Content-Transfer-Encoding: base64
 Content-Disposition: inline;
  filename="signature.asc"
 
 LS0tLS1CRUdJTiBQR1AgU0lHTkFUVVJFLS0tLS0KVmVyc2lvbjogR251UEcgdjEuNC43IChG
 cmVlQlNEKQoKaUQ4REJRRkhJaGVUd01KcW1KVng5NDRSQ3NYVkFLQ3paeGh4a2NENllvUUhO
 TExoL05oWUxVZHJSZ0NnbGY1SQpXNDVGZC9vSE9OUzltaW1vWWI3RENkND0KPVlIYm0KLS0t
 LS1FTkQgUEdQIFNJR05BVFVSRS0tLS0tCgo=
 --------------080301040802070400050107--
 
 --------------enig45F7F02B8760C11E5BEF1D0A
 Content-Type: application/pgp-signature; name="signature.asc"
 Content-Description: OpenPGP digital signature
 Content-Disposition: attachment; filename="signature.asc"
 
 -----BEGIN PGP SIGNATURE-----
 Version: GnuPG v1.4.7 (FreeBSD)
 
 iD8DBQFHIhnCwMJqmJVx944RCrjuAJ9j8M78BCgnWa5CiOjwdFsp5XvWxgCgnb3N
 tINrRRim683NAUsYLErw9AM=
 =UxnC
 -----END PGP SIGNATURE-----
 
 --------------enig45F7F02B8760C11E5BEF1D0A--

From: Pietro Cerutti <gahr@gahr.ch>
To: Cy Schubert <Cy.Schubert@komquats.com>
Cc: bug-followup <bug-followup@freebsd.org>
Subject: Re: ports/117488: [patch] sysutils/screen enable installation of
 man page
Date: Fri, 26 Oct 2007 18:58:28 +0200

 This is an OpenPGP/MIME signed message (RFC 2440 and 3156)
 --------------enig95E7D086A9750AB5393B7EBE
 Content-Type: text/plain; charset=ISO-8859-1
 Content-Transfer-Encoding: quoted-printable
 
 Solved.
 Actually, ehm, there was a NO_INFO=3D hanging around in my make.conf...
 Makefile 1.69 is ok.
 
 Sorry for loosing your time...
 
 --=20
 Pietro Cerutti
 
 PGP Public Key:
 http://gahr.ch/pgp
 
 
 --------------enig95E7D086A9750AB5393B7EBE
 Content-Type: application/pgp-signature; name="signature.asc"
 Content-Description: OpenPGP digital signature
 Content-Disposition: attachment; filename="signature.asc"
 
 -----BEGIN PGP SIGNATURE-----
 Version: GnuPG v1.4.7 (FreeBSD)
 
 iD8DBQFHIhy5wMJqmJVx944RCjmJAJ0dVREJTsVNLWJJRaGcmskPIvIzjgCgqxdH
 NPd25MmbLOFl5EFDEG/kemY=
 =gjBg
 -----END PGP SIGNATURE-----
 
 --------------enig95E7D086A9750AB5393B7EBE--

From: Cy Schubert <Cy.Schubert@komquats.com>
To: Pietro Cerutti <gahr@gahr.ch>
Cc: bug-followup <bug-followup@freebsd.org>
Subject: Re: ports/117488: [patch] sysutils/screen enable installation of man
 page
Date: Fri, 26 Oct 2007 10:00:48 -0700

 In message <4722178F.3060502@gahr.ch>, Pietro Cerutti writes:
 > This is an OpenPGP/MIME signed message (RFC 2440 and 3156)
 > --------------enig388F62288FF62D67758D2F64
 > Content-Type: text/plain; charset=ISO-8859-1
 > Content-Transfer-Encoding: quoted-printable
 > 
 > Cy Schubert wrote:
 > > In message <4722143A.5000007@gahr.ch>, Pietro Cerutti writes:
 > >> This is an OpenPGP/MIME signed message (RFC 2440 and 3156)
 > >> --------------enigC8EB130EC3F13747B4605930
 > >> Content-Type: text/plain; charset=3DISO-8859-1
 > >> Content-Transfer-Encoding: quoted-printable
 > >>
 > >> Cy Schubert wrote:
 > >>> In message <4721A9DE.5010404@gahr.ch>, Pietro Cerutti writes:
 > >>>> This is an OpenPGP/MIME signed message (RFC 2440 and 3156)
 > >>>> --------------enig14E192FFECFACCFF222FF981
 > >>>> Content-Type: text/plain; charset=3D3DUTF-8
 > >>>> Content-Transfer-Encoding: quoted-printable
 > >>>>
 > >>>> Please apply this patch to the Makefile you committed yesterday (1.6=
 > 8)=3D
 > >> =3D2E
 > >>>> It finally solves the INFO problem..
 > >>>>
 > >>>>
 > >>>> --- Makefile.orig       2007-10-26 10:42:30.000000000 +0200
 > >>>> +++ Makefile    2007-10-26 10:43:25.000000000 +0200
 > >>>> @@ -31,16 +31,14 @@
 > >>>>  GNU_CONFIGURE=3D3D3D yes
 > >>>>  .if defined(WITH_MAN)
 > >>>>  MAN1=3D3D3D          screen.1
 > >>>> -.else
 > >>>> -MAKE_ARGS+=3D3D    -DNO_MAN
 > >>>> +MAKE_ARGS+=3D3D    -DWITH_MAN
 > >>>>  .endif
 > >>>>  .if defined(WITH_INFO)
 > >>>>  .if !exists(/usr/bin/install-info)
 > >>>>  RUN_DEPENDS+=3D3D3D  install-info:${PORTSDIR}/print/texinfo
 > >>>>  .endif
 > >>>>  INFO=3D3D3D          screen
 > >>>> -.else
 > >>>> -MAKE_ARGS+=3D3D    -DNO_INFO
 > >>>> +MAKE_ARGS+=3D3D    -DWITH_INFO
 > >>>>  .endif
 > >>>>
 > >>>>  .if ${ARCH} =3D3D=3D3D "alpha"
 > >>> =3D20
 > >>> Your patch is incorrect. NO_MAN and NO_INFO are used when WITH_MAN an=
 > d =3D
 > >> WITH_INFO are not respectively defined.
 > >>> =3D20
 > >> The Makefile in doc/ doesn't care about NO_INFO, but checks for
 > >> WITH_INFO. That's why, without the patch above, the INFO page doesn't
 > >> get installed.
 > >> I recognize that it's a dirty hack, do you have a better solution?
 > >>
 > >=20
 > > cwsys$ pkg_info -L screen-4.0.3_1
 > > Information for screen-4.0.3_1:
 > >=20
 > > Files:
 > > /usr/local/man/man1/screen.1.gz
 > > /usr/local/bin/screen
 > > /usr/local/share/screen/utf8encodings/01
 > > /usr/local/share/screen/utf8encodings/02
 > > /usr/local/share/screen/utf8encodings/03
 > > /usr/local/share/screen/utf8encodings/04
 > > /usr/local/share/screen/utf8encodings/18
 > > /usr/local/share/screen/utf8encodings/19
 > > /usr/local/share/screen/utf8encodings/a1
 > > /usr/local/share/screen/utf8encodings/bf
 > > /usr/local/share/screen/utf8encodings/c2
 > > /usr/local/share/screen/utf8encodings/c3
 > > /usr/local/share/screen/utf8encodings/c4
 > > /usr/local/share/screen/utf8encodings/c6
 > > /usr/local/share/screen/utf8encodings/c7
 > > /usr/local/share/screen/utf8encodings/c8
 > > /usr/local/share/screen/utf8encodings/cc
 > > /usr/local/share/screen/utf8encodings/cd
 > > /usr/local/share/screen/utf8encodings/d6
 > > /usr/local/share/examples/screen/screenrc
 > > /usr/local/info/screen.info
 > >=20
 > > cwsys$ ll /usr/local/man/man1/screen.1.gz /usr/local/info/screen.info
 > > -r--r--r--  1 root  wheel  225811 Oct 26 09:21 /usr/local/info/screen.i=
 > nfo
 > > -r--r--r--  1 root  wheel   49066 Oct 26 09:21 /usr/local/man/man1/scre=
 > en.1.gz
 > > cwsys$=20
 > >=20
 > > No installation problems on my production server. Screen options should=
 >  be similar to the following. Are yours the same?
 > >=20
 > > cwsys$ cat /var/db/ports/screen/options=20
 > > # This file is auto-generated by 'make config'.
 > > # No user-servicable parts inside!
 > > # Options for screen-4.0.3
 > > _OPTIONS_READ=3Dscreen-4.0.3
 > > WITH_XTERM_256=3Dtrue
 > > WITH_NETHACK=3Dtrue
 > > WITHOUT_CJK=3Dtrue
 > > WITH_MAN=3Dtrue
 > > WITH_INFO=3Dtrue
 > > cwsys$=20
 > >=20
 > >=20
 > 
 > Nope...
 > See output below.. in the meanwhile, I suggest this one against your 1.69=
 > :
 > 
 > --- Makefile    2007-10-26 18:29:38.000000000 +0200
 > +++ Makefile.mine       2007-10-26 18:30:41.000000000 +0200
 > @@ -31,6 +31,7 @@
 >  GNU_CONFIGURE=3D yes
 >  .if defined(WITH_MAN)
 >  MAN1=3D          screen.1
 > +MAKE_ARGS+=3D -DWITH_MAN
 >  .else
 >  MAKE_ARGS+=3D    -DNO_MAN
 >  .endif
 > @@ -39,6 +40,7 @@
 >  RUN_DEPENDS+=3D  install-info:${PORTSDIR}/print/texinfo
 >  .endif
 >  INFO=3D          screen
 > +MAKE_ARGS+=3D -DWITH_INFO
 >  .else
 >  MAKE_ARGS+=3D    -DNO_INFO
 >  .endif
 > 
 > 
 > Building with your 1.69...
 > 
 > > cat /var/db/ports/screen/options
 > # This file is auto-generated by 'make config'.
 > # No user-servicable parts inside!
 > # Options for screen-4.0.3_1
 > _OPTIONS_READ=3Dscreen-4.0.3_1
 > WITH_CJK=3Dtrue
 > WITH_INFO=3Dtrue
 > WITH_MAN=3Dtrue
 > WITH_NETHACK=3Dtrue
 > WITH_XTERM_256=3Dtrue
 > 
 > > sudo make install
 > =3D=3D=3D>  Found saved configuration for screen-4.0.3_1
 > =3D=3D=3D>  Extracting for screen-4.0.3_1
 > =3D> MD5 Checksum OK for screen-4.0.3.tar.gz.
 > =3D> SHA256 Checksum OK for screen-4.0.3.tar.gz.
 > =3D=3D=3D>  Patching for screen-4.0.3_1
 > =3D=3D=3D>  Applying extra patch /usr/ports/sysutils/screen/files/opt-cjk=
 > width
 > =3D=3D=3D>  Applying FreeBSD patches for screen-4.0.3_1
 > =3D=3D=3D>  Configuring for screen-4.0.3_1
 > configure: WARNING: you should use --build, --host, --target
 > this is screen version 4.0.3
 > checking for i386-portbld-freebsd6.2-gcc... cc
 > checking for C compiler default output... a.out
 > checking whether the C compiler works... yes
 > checking whether we are cross compiling... no
 > checking for suffix of executables...
 > checking for suffix of object files... o
 > checking whether we are using the GNU C compiler... yes
 > checking whether cc accepts -g... yes
 > checking for cc option to accept ANSI C... none needed
 > checking how to run the C preprocessor... cc -E
 > checking for egrep... grep -E
 > checking whether cc needs -traditional... no
 > checking for library containing strerror... none required
 > checking for gawk... gawk
 > checking for a BSD-compatible install... /usr/bin/install -c -o root -g
 > wheel
 > configure: checking for buggy tools...
 > checking if a system-wide socket dir should be used... yes
 > checking for the socket dir... (eff_uid ? "/tmp/uscreens" : "/tmp/screens=
 > ")
 > configure: checking for MIPS...
 > configure: checking for Ultrix...
 > configure: checking for butterfly...
 > configure: checking for POSIX.1...
 > - you have a POSIX system
 > configure: checking for System V...
 > configure: checking for sequent/ptx...
 > configure: checking SVR4...
 > checking for ANSI C header files... no
 > checking for sys/types.h... yes
 > checking for sys/stat.h... yes
 > checking for stdlib.h... yes
 > checking for string.h... yes
 > checking for memory.h... yes
 > checking for strings.h... yes
 > checking for inttypes.h... yes
 > checking for stdint.h... yes
 > checking for unistd.h... yes
 > configure: checking for Solaris 2.x...
 > configure: checking BSD job jontrol...
 > - you have jobcontrol
 > configure: checking setreuid...
 > configure: checking seteuid...
 > configure: checking select...
 > configure: checking fifos...
 > - your fifos are usable
 > configure: checking for broken fifo implementation...
 > - your implementation is ok
 > configure: checking sockets...
 > - your sockets are usable
 > configure: checking socket implementation...
 > - you are normal
 > - both sockets and fifos usable. let's take fifos.
 > configure: checking select return value...
 > - select is ok
 > configure: checking for tgetent...
 > configure: checking libtermcap...
 > - you use the termcap database
 > configure: checking ospeed...
 > configure: checking for /dev/ptc...
 > configure: checking for SVR4 ptys...
 > checking for getpt... no
 > checking for openpty... no
 > checking for openpty in -lutil... yes
 > configure: checking for ptyranges...
 > configure: checking default tty permissions/group...
 > - ptys are world accessable
 > configure: checking getutent...
 > configure: checking getutent with -lgen...
 > configure: checking ut_host...
 > checking utempter.h usability... no
 > checking utempter.h presence... no
 > checking for utempter.h... no
 > configure: checking for libutil(s)...
 > configure: checking getloadavg...
 > assuming posix signal definition
 > configure: checking for crypt and sec libraries...
 > configure: checking crypt...
 > configure: checking IRIX sun library...
 > configure: checking syslog...
 > configure: checking wait union...
 > configure: checking for termio or termios...
 > configure: checking getspnam...
 > configure: checking getttyent...
 > configure: checking fdwalk...
 > configure: checking whether memcpy/memmove/bcopy handles overlapping
 > arguments...
 > checking long file names... yes
 > checking for vsprintf... yes
 > checking for dirent.h that defines DIR... yes
 > checking for library containing opendir... none required
 > checking for setenv... yes
 > checking for nl_langinfo(CODESET)... yes
 > checking for library containing gethostname... none required
 > checking for rename... yes
 > checking for fchmod... yes
 > checking for fchown... yes
 > checking for strerror... yes
 > checking for lstat... yes
 > checking for _exit... yes
 > checking for utimes... yes
 > checking for vsnprintf... yes
 > checking for getcwd... yes
 > checking for setlocale... yes
 > checking for strftime... yes
 > checking for the global screenrc file... "/usr/local/etc/screenrc"
 > checking for the utf8-encodings location...
 > "/usr/local/share/screen/utf8encodings"
 > configure: creating ./config.status
 > config.status: creating Makefile
 > config.status: creating doc/Makefile
 > config.status: creating config.h
 > config.status: executing default commands
 > 
 > Now please check the pathnames in the Makefile and in the user
 > configuration section in config.h.
 > Then type 'make' to make screen. Good luck.
 > 
 > =3D=3D=3D>  Building for screen-4.0.3_1
 > CPP=3D"cc -E " srcdir=3D. sh ./osdef.sh
 > AWK=3Dgawk CC=3D"cc -O2 -pipe -ggdb -march=3Dpentiumpro -DCOLORS256" srcd=
 > ir=3D.
 > sh ./comm.sh
 > AWK=3Dgawk srcdir=3D. sh ./term.sh
 > cc -c -I. -I.    -O2 -pipe -ggdb -march=3Dpentiumpro -DCOLORS256 screen.c=
 > 
 > cc -c -I. -I.    -O2 -pipe -ggdb -march=3Dpentiumpro -DCOLORS256 ansi.c
 > cc -c -I. -I.    -O2 -pipe -ggdb -march=3Dpentiumpro -DCOLORS256 fileio.c=
 > 
 > cc -c -I. -I.    -O2 -pipe -ggdb -march=3Dpentiumpro -DCOLORS256 mark.c
 > cc -c -I. -I.    -O2 -pipe -ggdb -march=3Dpentiumpro -DCOLORS256 misc.c
 > cc -c -I. -I.    -O2 -pipe -ggdb -march=3Dpentiumpro -DCOLORS256 resize.c=
 > 
 > cc -c -I. -I.    -O2 -pipe -ggdb -march=3Dpentiumpro -DCOLORS256 socket.c=
 > 
 > cc -c -I. -I.    -O2 -pipe -ggdb -march=3Dpentiumpro -DCOLORS256 search.c=
 > 
 > sh ./tty.sh tty.c
 > cc -c -I. -I.    -O2 -pipe -ggdb -march=3Dpentiumpro -DCOLORS256 tty.c
 > cc -c -I. -I.    -O2 -pipe -ggdb -march=3Dpentiumpro -DCOLORS256 term.c
 > cc -c -I. -I.    -O2 -pipe -ggdb -march=3Dpentiumpro -DCOLORS256 window.c=
 > 
 > cc -c -I. -I.    -O2 -pipe -ggdb -march=3Dpentiumpro -DCOLORS256 utmp.c
 > cc -c -I. -I.    -O2 -pipe -ggdb -march=3Dpentiumpro -DCOLORS256 loadav.c=
 > 
 > cc -c -I. -I.    -O2 -pipe -ggdb -march=3Dpentiumpro -DCOLORS256 putenv.c=
 > 
 > cc -c -I. -I.    -O2 -pipe -ggdb -march=3Dpentiumpro -DCOLORS256 help.c
 > cc -c -I. -I.    -O2 -pipe -ggdb -march=3Dpentiumpro -DCOLORS256 termcap.=
 > c
 > cc -c -I. -I.    -O2 -pipe -ggdb -march=3Dpentiumpro -DCOLORS256 input.c
 > cc -c -I. -I.    -O2 -pipe -ggdb -march=3Dpentiumpro -DCOLORS256 attacher=
 > =2Ec
 > cc -c -I. -I.    -O2 -pipe -ggdb -march=3Dpentiumpro -DCOLORS256 pty.c
 > cc -c -I. -I.    -O2 -pipe -ggdb -march=3Dpentiumpro -DCOLORS256 process.=
 > c
 > cc -c -I. -I.    -O2 -pipe -ggdb -march=3Dpentiumpro -DCOLORS256 display.=
 > c
 > cc -c -I. -I.    -O2 -pipe -ggdb -march=3Dpentiumpro -DCOLORS256 comm.c
 > cc -c -I. -I.    -O2 -pipe -ggdb -march=3Dpentiumpro -DCOLORS256 kmapdef.=
 > c
 > cc -c -I. -I.    -O2 -pipe -ggdb -march=3Dpentiumpro -DCOLORS256 acls.c
 > cc -c -I. -I.    -O2 -pipe -ggdb -march=3Dpentiumpro -DCOLORS256 braille.=
 > c
 > cc -c -I. -I.    -O2 -pipe -ggdb -march=3Dpentiumpro -DCOLORS256 braille_=
 > tsi.c
 > cc -c -I. -I.    -O2 -pipe -ggdb -march=3Dpentiumpro -DCOLORS256 logfile.=
 > c
 > cc -c -I. -I.    -O2 -pipe -ggdb -march=3Dpentiumpro -DCOLORS256 layer.c
 > cc -c -I. -I.    -O2 -pipe -ggdb -march=3Dpentiumpro -DCOLORS256 sched.c
 > cc -c -I. -I.    -O2 -pipe -ggdb -march=3Dpentiumpro -DCOLORS256 teln.c
 > cc -c -I. -I.    -O2 -pipe -ggdb -march=3Dpentiumpro -DCOLORS256 nethack.=
 > c
 > cc -c -I. -I.    -O2 -pipe -ggdb -march=3Dpentiumpro -DCOLORS256 encoding=
 > =2Ec
 > cc  -o screen screen.o ansi.o fileio.o mark.o misc.o resize.o socket.o
 > search.o tty.o term.o window.o utmp.o loadav.o putenv.o help.o
 > termcap.o input.o attacher.o pty.o process.o display.o comm.o  kmapdef.o
 > acls.o braille.o braille_tsi.o logfile.o layer.o  sched.o teln.o
 > nethack.o encoding.o -ltermcap  -lutil -lutil -lcrypt
 > =3D=3D=3D>  Installing for screen-4.0.3_1
 > =3D=3D=3D>   Generating temporary packing list
 > =3D=3D=3D>  Checking if sysutils/screen already installed
 > =2E/etc/mkinstalldirs /usr/local/bin `sed < config.h -n -e '/define
 > SCREENENCODINGS/s/^.*"\([^"]*\)"/\1/p'`
 > cd doc ; make  installdirs
 > install  -s -o root -g wheel -m 555 screen /usr/local/bin/screen
 > chown root:wheel /usr/local/bin/screen && chmod 4755 /usr/local/bin/scree=
 > n
 > cp ./utf8encodings/?? `sed < config.h -n -e '/define
 > SCREENENCODINGS/s/^.*"\([^"]*\)"/\1/p'`
 > cd doc ; make  install
 > if [ -d /usr/lib/terminfo ]; then  PATH=3D"$PATH:/usr/5bin" tic
 > =2E/terminfo/screeninfo.src;  chmod 644 /usr/lib/terminfo/s/screen*;  fi
 > termcap entry (./terminfo/screencap) should be installed manually.
 > You may also want to install ./etc/etcscreenrc in /usr/local/etc/screenrc=
 > 
 > install-info --quiet /usr/local/info/screen.info /usr/local/info/dir
 > install-info: No such file or directory for /usr/local/info/screen.info
 > *** Error code 1
 > 
 > Stop in /usr/ports/sysutils/screen.
 > *** Error code 1
 > 
 > Stop in /usr/ports/sysutils/screen.
 > Exit 1
 
 It's no rocket science to see that there's a problem here. The question is why do I not see the same problem on my systems? Searching the working directory for for WITH_MAN and WITH_INFO results in nothing matched:
 
 cwsys# find /export/wrkdir/usr/ports/sysutils/screen/work/screen-4.0.3 -type f | xargs egrep 'WITH_MAN|WITH_INFO'
 cwsys# 
 
 NO_INFO and NO_MAN does match:
 
 cwsys# find /export/wrkdir/usr/ports/sysutils/screen/work/screen-4.0.3 -type f | xargs egrep 'NO_MAN|NO_INFO'
 /export/wrkdir/usr/ports/sysutils/screen/work/screen-4.0.3/doc/Makefile.in:.if !defined(NO_MAN)
 /export/wrkdir/usr/ports/sysutils/screen/work/screen-4.0.3/doc/Makefile.in:.if !defined(NO_INFO)
 /export/wrkdir/usr/ports/sysutils/screen/work/screen-4.0.3/doc/Makefile.in:.if !defined(NO_MAN)
 /export/wrkdir/usr/ports/sysutils/screen/work/screen-4.0.3/doc/Makefile.in:.if !defined(NO_INFO)
 /export/wrkdir/usr/ports/sysutils/screen/work/screen-4.0.3/doc/Makefile:.if !defined(NO_MAN)
 /export/wrkdir/usr/ports/sysutils/screen/work/screen-4.0.3/doc/Makefile:.if !defined(NO_INFO)
 /export/wrkdir/usr/ports/sysutils/screen/work/screen-4.0.3/doc/Makefile:.if !defined(NO_MAN)
 /export/wrkdir/usr/ports/sysutils/screen/work/screen-4.0.3/doc/Makefile:.if !defined(NO_INFO)
 cwsys# 
 
 The question that needs to be answered is why does your site require WITH_MAN and WITH_INFO to be defined in MAKE_ARGS to work? I will test the install out on my testbed in a pristine environment. What version of FreeBSD are you using? I'm able to boot anything from 5.5 to the latest 8.0-CURRENT on it. Unless you tell me otherwise I will assume you are using 6.2-STABLE.
 
 
 -- 
 Cheers,
 Cy Schubert <Cy.Schubert@komquats.com>
 FreeBSD UNIX:  <cy@FreeBSD.org>   Web:  http://www.FreeBSD.org
 
 			e**(i*pi)+1=0
 
 
 

From: Cy Schubert <Cy.Schubert@komquats.com>
To: Pietro Cerutti <gahr@gahr.ch>
Cc: bug-followup <bug-followup@freebsd.org>
Subject: Re: ports/117488: [patch] sysutils/screen enable installation of man
 page
Date: Fri, 26 Oct 2007 10:04:10 -0700

 In message <47221CB4.5050701@gahr.ch>, Pietro Cerutti writes:
 > 
 > Solved.
 > Actually, ehm, there was a NO_INFO= hanging around in my make.conf...
 > Makefile 1.69 is ok.
 > 
 > Sorry for loosing your time...
 
 No problem.
 
 
 -- 
 Cheers,
 Cy Schubert <Cy.Schubert@komquats.com>
 FreeBSD UNIX:  <cy@FreeBSD.org>   Web:  http://www.FreeBSD.org
 
 			e**(i*pi)+1=0
 
 

From: Pietro Cerutti <gahr@gahr.ch>
To: Cy Schubert <Cy.Schubert@komquats.com>
Cc: bug-followup <bug-followup@freebsd.org>
Subject: Re: ports/117488: [patch] sysutils/screen enable installation of
 man page
Date: Fri, 26 Oct 2007 19:05:59 +0200

 This is an OpenPGP/MIME signed message (RFC 2440 and 3156)
 --------------enig87BCA7960B39C5BDEBE3F851
 Content-Type: text/plain; charset=ISO-8859-1
 Content-Transfer-Encoding: quoted-printable
 
 Cy Schubert wrote:
 > In message <4722178F.3060502@gahr.ch>, Pietro Cerutti writes:
 >> This is an OpenPGP/MIME signed message (RFC 2440 and 3156)
 >> --------------enig388F62288FF62D67758D2F64
 >> Content-Type: text/plain; charset=3DISO-8859-1
 >> Content-Transfer-Encoding: quoted-printable
 >>
 >> Cy Schubert wrote:
 >>> In message <4722143A.5000007@gahr.ch>, Pietro Cerutti writes:
 >>>> This is an OpenPGP/MIME signed message (RFC 2440 and 3156)
 >>>> --------------enigC8EB130EC3F13747B4605930
 >>>> Content-Type: text/plain; charset=3D3DISO-8859-1
 >>>> Content-Transfer-Encoding: quoted-printable
 >>>>
 >>>> Cy Schubert wrote:
 >>>>> In message <4721A9DE.5010404@gahr.ch>, Pietro Cerutti writes:
 >>>>>> This is an OpenPGP/MIME signed message (RFC 2440 and 3156)
 >>>>>> --------------enig14E192FFECFACCFF222FF981
 >>>>>> Content-Type: text/plain; charset=3D3D3DUTF-8
 >>>>>> Content-Transfer-Encoding: quoted-printable
 >>>>>>
 >>>>>> Please apply this patch to the Makefile you committed yesterday (1=
 =2E6=3D
 >> 8)=3D3D
 >>>> =3D3D2E
 >>>>>> It finally solves the INFO problem..
 >>>>>>
 >>>>>>
 >>>>>> --- Makefile.orig       2007-10-26 10:42:30.000000000 +0200
 >>>>>> +++ Makefile    2007-10-26 10:43:25.000000000 +0200
 >>>>>> @@ -31,16 +31,14 @@
 >>>>>>  GNU_CONFIGURE=3D3D3D3D yes
 >>>>>>  .if defined(WITH_MAN)
 >>>>>>  MAN1=3D3D3D3D          screen.1
 >>>>>> -.else
 >>>>>> -MAKE_ARGS+=3D3D3D    -DNO_MAN
 >>>>>> +MAKE_ARGS+=3D3D3D    -DWITH_MAN
 >>>>>>  .endif
 >>>>>>  .if defined(WITH_INFO)
 >>>>>>  .if !exists(/usr/bin/install-info)
 >>>>>>  RUN_DEPENDS+=3D3D3D3D  install-info:${PORTSDIR}/print/texinfo
 >>>>>>  .endif
 >>>>>>  INFO=3D3D3D3D          screen
 >>>>>> -.else
 >>>>>> -MAKE_ARGS+=3D3D3D    -DNO_INFO
 >>>>>> +MAKE_ARGS+=3D3D3D    -DWITH_INFO
 >>>>>>  .endif
 >>>>>>
 >>>>>>  .if ${ARCH} =3D3D3D=3D3D3D "alpha"
 >>>>> =3D3D20
 >>>>> Your patch is incorrect. NO_MAN and NO_INFO are used when WITH_MAN =
 an=3D
 >> d =3D3D
 >>>> WITH_INFO are not respectively defined.
 >>>>> =3D3D20
 >>>> The Makefile in doc/ doesn't care about NO_INFO, but checks for
 >>>> WITH_INFO. That's why, without the patch above, the INFO page doesn'=
 t
 >>>> get installed.
 >>>> I recognize that it's a dirty hack, do you have a better solution?
 >>>>
 >>> =3D20
 >>> cwsys$ pkg_info -L screen-4.0.3_1
 >>> Information for screen-4.0.3_1:
 >>> =3D20
 >>> Files:
 >>> /usr/local/man/man1/screen.1.gz
 >>> /usr/local/bin/screen
 >>> /usr/local/share/screen/utf8encodings/01
 >>> /usr/local/share/screen/utf8encodings/02
 >>> /usr/local/share/screen/utf8encodings/03
 >>> /usr/local/share/screen/utf8encodings/04
 >>> /usr/local/share/screen/utf8encodings/18
 >>> /usr/local/share/screen/utf8encodings/19
 >>> /usr/local/share/screen/utf8encodings/a1
 >>> /usr/local/share/screen/utf8encodings/bf
 >>> /usr/local/share/screen/utf8encodings/c2
 >>> /usr/local/share/screen/utf8encodings/c3
 >>> /usr/local/share/screen/utf8encodings/c4
 >>> /usr/local/share/screen/utf8encodings/c6
 >>> /usr/local/share/screen/utf8encodings/c7
 >>> /usr/local/share/screen/utf8encodings/c8
 >>> /usr/local/share/screen/utf8encodings/cc
 >>> /usr/local/share/screen/utf8encodings/cd
 >>> /usr/local/share/screen/utf8encodings/d6
 >>> /usr/local/share/examples/screen/screenrc
 >>> /usr/local/info/screen.info
 >>> =3D20
 >>> cwsys$ ll /usr/local/man/man1/screen.1.gz /usr/local/info/screen.info=
 
 >>> -r--r--r--  1 root  wheel  225811 Oct 26 09:21 /usr/local/info/screen=
 =2Ei=3D
 >> nfo
 >>> -r--r--r--  1 root  wheel   49066 Oct 26 09:21 /usr/local/man/man1/sc=
 re=3D
 >> en.1.gz
 >>> cwsys$=3D20
 >>> =3D20
 >>> No installation problems on my production server. Screen options shou=
 ld=3D
 >>  be similar to the following. Are yours the same?
 >>> =3D20
 >>> cwsys$ cat /var/db/ports/screen/options=3D20
 >>> # This file is auto-generated by 'make config'.
 >>> # No user-servicable parts inside!
 >>> # Options for screen-4.0.3
 >>> _OPTIONS_READ=3D3Dscreen-4.0.3
 >>> WITH_XTERM_256=3D3Dtrue
 >>> WITH_NETHACK=3D3Dtrue
 >>> WITHOUT_CJK=3D3Dtrue
 >>> WITH_MAN=3D3Dtrue
 >>> WITH_INFO=3D3Dtrue
 >>> cwsys$=3D20
 >>> =3D20
 >>> =3D20
 >> Nope...
 >> See output below.. in the meanwhile, I suggest this one against your 1=
 =2E69=3D
 >> :
 >>
 >> --- Makefile    2007-10-26 18:29:38.000000000 +0200
 >> +++ Makefile.mine       2007-10-26 18:30:41.000000000 +0200
 >> @@ -31,6 +31,7 @@
 >>  GNU_CONFIGURE=3D3D yes
 >>  .if defined(WITH_MAN)
 >>  MAN1=3D3D          screen.1
 >> +MAKE_ARGS+=3D3D -DWITH_MAN
 >>  .else
 >>  MAKE_ARGS+=3D3D    -DNO_MAN
 >>  .endif
 >> @@ -39,6 +40,7 @@
 >>  RUN_DEPENDS+=3D3D  install-info:${PORTSDIR}/print/texinfo
 >>  .endif
 >>  INFO=3D3D          screen
 >> +MAKE_ARGS+=3D3D -DWITH_INFO
 >>  .else
 >>  MAKE_ARGS+=3D3D    -DNO_INFO
 >>  .endif
 >>
 >>
 >> Building with your 1.69...
 >>
 >>> cat /var/db/ports/screen/options
 >> # This file is auto-generated by 'make config'.
 >> # No user-servicable parts inside!
 >> # Options for screen-4.0.3_1
 >> _OPTIONS_READ=3D3Dscreen-4.0.3_1
 >> WITH_CJK=3D3Dtrue
 >> WITH_INFO=3D3Dtrue
 >> WITH_MAN=3D3Dtrue
 >> WITH_NETHACK=3D3Dtrue
 >> WITH_XTERM_256=3D3Dtrue
 >>
 >>> sudo make install
 >> =3D3D=3D3D=3D3D>  Found saved configuration for screen-4.0.3_1
 >> =3D3D=3D3D=3D3D>  Extracting for screen-4.0.3_1
 >> =3D3D> MD5 Checksum OK for screen-4.0.3.tar.gz.
 >> =3D3D> SHA256 Checksum OK for screen-4.0.3.tar.gz.
 >> =3D3D=3D3D=3D3D>  Patching for screen-4.0.3_1
 >> =3D3D=3D3D=3D3D>  Applying extra patch /usr/ports/sysutils/screen/file=
 s/opt-cjk=3D
 >> width
 >> =3D3D=3D3D=3D3D>  Applying FreeBSD patches for screen-4.0.3_1
 >> =3D3D=3D3D=3D3D>  Configuring for screen-4.0.3_1
 >> configure: WARNING: you should use --build, --host, --target
 >> this is screen version 4.0.3
 >> checking for i386-portbld-freebsd6.2-gcc... cc
 >> checking for C compiler default output... a.out
 >> checking whether the C compiler works... yes
 >> checking whether we are cross compiling... no
 >> checking for suffix of executables...
 >> checking for suffix of object files... o
 >> checking whether we are using the GNU C compiler... yes
 >> checking whether cc accepts -g... yes
 >> checking for cc option to accept ANSI C... none needed
 >> checking how to run the C preprocessor... cc -E
 >> checking for egrep... grep -E
 >> checking whether cc needs -traditional... no
 >> checking for library containing strerror... none required
 >> checking for gawk... gawk
 >> checking for a BSD-compatible install... /usr/bin/install -c -o root -=
 g
 >> wheel
 >> configure: checking for buggy tools...
 >> checking if a system-wide socket dir should be used... yes
 >> checking for the socket dir... (eff_uid ? "/tmp/uscreens" : "/tmp/scre=
 ens=3D
 >> ")
 >> configure: checking for MIPS...
 >> configure: checking for Ultrix...
 >> configure: checking for butterfly...
 >> configure: checking for POSIX.1...
 >> - you have a POSIX system
 >> configure: checking for System V...
 >> configure: checking for sequent/ptx...
 >> configure: checking SVR4...
 >> checking for ANSI C header files... no
 >> checking for sys/types.h... yes
 >> checking for sys/stat.h... yes
 >> checking for stdlib.h... yes
 >> checking for string.h... yes
 >> checking for memory.h... yes
 >> checking for strings.h... yes
 >> checking for inttypes.h... yes
 >> checking for stdint.h... yes
 >> checking for unistd.h... yes
 >> configure: checking for Solaris 2.x...
 >> configure: checking BSD job jontrol...
 >> - you have jobcontrol
 >> configure: checking setreuid...
 >> configure: checking seteuid...
 >> configure: checking select...
 >> configure: checking fifos...
 >> - your fifos are usable
 >> configure: checking for broken fifo implementation...
 >> - your implementation is ok
 >> configure: checking sockets...
 >> - your sockets are usable
 >> configure: checking socket implementation...
 >> - you are normal
 >> - both sockets and fifos usable. let's take fifos.
 >> configure: checking select return value...
 >> - select is ok
 >> configure: checking for tgetent...
 >> configure: checking libtermcap...
 >> - you use the termcap database
 >> configure: checking ospeed...
 >> configure: checking for /dev/ptc...
 >> configure: checking for SVR4 ptys...
 >> checking for getpt... no
 >> checking for openpty... no
 >> checking for openpty in -lutil... yes
 >> configure: checking for ptyranges...
 >> configure: checking default tty permissions/group...
 >> - ptys are world accessable
 >> configure: checking getutent...
 >> configure: checking getutent with -lgen...
 >> configure: checking ut_host...
 >> checking utempter.h usability... no
 >> checking utempter.h presence... no
 >> checking for utempter.h... no
 >> configure: checking for libutil(s)...
 >> configure: checking getloadavg...
 >> assuming posix signal definition
 >> configure: checking for crypt and sec libraries...
 >> configure: checking crypt...
 >> configure: checking IRIX sun library...
 >> configure: checking syslog...
 >> configure: checking wait union...
 >> configure: checking for termio or termios...
 >> configure: checking getspnam...
 >> configure: checking getttyent...
 >> configure: checking fdwalk...
 >> configure: checking whether memcpy/memmove/bcopy handles overlapping
 >> arguments...
 >> checking long file names... yes
 >> checking for vsprintf... yes
 >> checking for dirent.h that defines DIR... yes
 >> checking for library containing opendir... none required
 >> checking for setenv... yes
 >> checking for nl_langinfo(CODESET)... yes
 >> checking for library containing gethostname... none required
 >> checking for rename... yes
 >> checking for fchmod... yes
 >> checking for fchown... yes
 >> checking for strerror... yes
 >> checking for lstat... yes
 >> checking for _exit... yes
 >> checking for utimes... yes
 >> checking for vsnprintf... yes
 >> checking for getcwd... yes
 >> checking for setlocale... yes
 >> checking for strftime... yes
 >> checking for the global screenrc file... "/usr/local/etc/screenrc"
 >> checking for the utf8-encodings location...
 >> "/usr/local/share/screen/utf8encodings"
 >> configure: creating ./config.status
 >> config.status: creating Makefile
 >> config.status: creating doc/Makefile
 >> config.status: creating config.h
 >> config.status: executing default commands
 >>
 >> Now please check the pathnames in the Makefile and in the user
 >> configuration section in config.h.
 >> Then type 'make' to make screen. Good luck.
 >>
 >> =3D3D=3D3D=3D3D>  Building for screen-4.0.3_1
 >> CPP=3D3D"cc -E " srcdir=3D3D. sh ./osdef.sh
 >> AWK=3D3Dgawk CC=3D3D"cc -O2 -pipe -ggdb -march=3D3Dpentiumpro -DCOLORS=
 256" srcd=3D
 >> ir=3D3D.
 >> sh ./comm.sh
 >> AWK=3D3Dgawk srcdir=3D3D. sh ./term.sh
 >> cc -c -I. -I.    -O2 -pipe -ggdb -march=3D3Dpentiumpro -DCOLORS256 scr=
 een.c=3D
 >>
 >> cc -c -I. -I.    -O2 -pipe -ggdb -march=3D3Dpentiumpro -DCOLORS256 ans=
 i.c
 >> cc -c -I. -I.    -O2 -pipe -ggdb -march=3D3Dpentiumpro -DCOLORS256 fil=
 eio.c=3D
 >>
 >> cc -c -I. -I.    -O2 -pipe -ggdb -march=3D3Dpentiumpro -DCOLORS256 mar=
 k.c
 >> cc -c -I. -I.    -O2 -pipe -ggdb -march=3D3Dpentiumpro -DCOLORS256 mis=
 c.c
 >> cc -c -I. -I.    -O2 -pipe -ggdb -march=3D3Dpentiumpro -DCOLORS256 res=
 ize.c=3D
 >>
 >> cc -c -I. -I.    -O2 -pipe -ggdb -march=3D3Dpentiumpro -DCOLORS256 soc=
 ket.c=3D
 >>
 >> cc -c -I. -I.    -O2 -pipe -ggdb -march=3D3Dpentiumpro -DCOLORS256 sea=
 rch.c=3D
 >>
 >> sh ./tty.sh tty.c
 >> cc -c -I. -I.    -O2 -pipe -ggdb -march=3D3Dpentiumpro -DCOLORS256 tty=
 =2Ec
 >> cc -c -I. -I.    -O2 -pipe -ggdb -march=3D3Dpentiumpro -DCOLORS256 ter=
 m.c
 >> cc -c -I. -I.    -O2 -pipe -ggdb -march=3D3Dpentiumpro -DCOLORS256 win=
 dow.c=3D
 >>
 >> cc -c -I. -I.    -O2 -pipe -ggdb -march=3D3Dpentiumpro -DCOLORS256 utm=
 p.c
 >> cc -c -I. -I.    -O2 -pipe -ggdb -march=3D3Dpentiumpro -DCOLORS256 loa=
 dav.c=3D
 >>
 >> cc -c -I. -I.    -O2 -pipe -ggdb -march=3D3Dpentiumpro -DCOLORS256 put=
 env.c=3D
 >>
 >> cc -c -I. -I.    -O2 -pipe -ggdb -march=3D3Dpentiumpro -DCOLORS256 hel=
 p.c
 >> cc -c -I. -I.    -O2 -pipe -ggdb -march=3D3Dpentiumpro -DCOLORS256 ter=
 mcap.=3D
 >> c
 >> cc -c -I. -I.    -O2 -pipe -ggdb -march=3D3Dpentiumpro -DCOLORS256 inp=
 ut.c
 >> cc -c -I. -I.    -O2 -pipe -ggdb -march=3D3Dpentiumpro -DCOLORS256 att=
 acher=3D
 >> =3D2Ec
 >> cc -c -I. -I.    -O2 -pipe -ggdb -march=3D3Dpentiumpro -DCOLORS256 pty=
 =2Ec
 >> cc -c -I. -I.    -O2 -pipe -ggdb -march=3D3Dpentiumpro -DCOLORS256 pro=
 cess.=3D
 >> c
 >> cc -c -I. -I.    -O2 -pipe -ggdb -march=3D3Dpentiumpro -DCOLORS256 dis=
 play.=3D
 >> c
 >> cc -c -I. -I.    -O2 -pipe -ggdb -march=3D3Dpentiumpro -DCOLORS256 com=
 m.c
 >> cc -c -I. -I.    -O2 -pipe -ggdb -march=3D3Dpentiumpro -DCOLORS256 kma=
 pdef.=3D
 >> c
 >> cc -c -I. -I.    -O2 -pipe -ggdb -march=3D3Dpentiumpro -DCOLORS256 acl=
 s.c
 >> cc -c -I. -I.    -O2 -pipe -ggdb -march=3D3Dpentiumpro -DCOLORS256 bra=
 ille.=3D
 >> c
 >> cc -c -I. -I.    -O2 -pipe -ggdb -march=3D3Dpentiumpro -DCOLORS256 bra=
 ille_=3D
 >> tsi.c
 >> cc -c -I. -I.    -O2 -pipe -ggdb -march=3D3Dpentiumpro -DCOLORS256 log=
 file.=3D
 >> c
 >> cc -c -I. -I.    -O2 -pipe -ggdb -march=3D3Dpentiumpro -DCOLORS256 lay=
 er.c
 >> cc -c -I. -I.    -O2 -pipe -ggdb -march=3D3Dpentiumpro -DCOLORS256 sch=
 ed.c
 >> cc -c -I. -I.    -O2 -pipe -ggdb -march=3D3Dpentiumpro -DCOLORS256 tel=
 n.c
 >> cc -c -I. -I.    -O2 -pipe -ggdb -march=3D3Dpentiumpro -DCOLORS256 net=
 hack.=3D
 >> c
 >> cc -c -I. -I.    -O2 -pipe -ggdb -march=3D3Dpentiumpro -DCOLORS256 enc=
 oding=3D
 >> =3D2Ec
 >> cc  -o screen screen.o ansi.o fileio.o mark.o misc.o resize.o socket.o=
 
 >> search.o tty.o term.o window.o utmp.o loadav.o putenv.o help.o
 >> termcap.o input.o attacher.o pty.o process.o display.o comm.o  kmapdef=
 =2Eo
 >> acls.o braille.o braille_tsi.o logfile.o layer.o  sched.o teln.o
 >> nethack.o encoding.o -ltermcap  -lutil -lutil -lcrypt
 >> =3D3D=3D3D=3D3D>  Installing for screen-4.0.3_1
 >> =3D3D=3D3D=3D3D>   Generating temporary packing list
 >> =3D3D=3D3D=3D3D>  Checking if sysutils/screen already installed
 >> =3D2E/etc/mkinstalldirs /usr/local/bin `sed < config.h -n -e '/define
 >> SCREENENCODINGS/s/^.*"\([^"]*\)"/\1/p'`
 >> cd doc ; make  installdirs
 >> install  -s -o root -g wheel -m 555 screen /usr/local/bin/screen
 >> chown root:wheel /usr/local/bin/screen && chmod 4755 /usr/local/bin/sc=
 ree=3D
 >> n
 >> cp ./utf8encodings/?? `sed < config.h -n -e '/define
 >> SCREENENCODINGS/s/^.*"\([^"]*\)"/\1/p'`
 >> cd doc ; make  install
 >> if [ -d /usr/lib/terminfo ]; then  PATH=3D3D"$PATH:/usr/5bin" tic
 >> =3D2E/terminfo/screeninfo.src;  chmod 644 /usr/lib/terminfo/s/screen*;=
   fi
 >> termcap entry (./terminfo/screencap) should be installed manually.
 >> You may also want to install ./etc/etcscreenrc in /usr/local/etc/scree=
 nrc=3D
 >>
 >> install-info --quiet /usr/local/info/screen.info /usr/local/info/dir
 >> install-info: No such file or directory for /usr/local/info/screen.inf=
 o
 >> *** Error code 1
 >>
 >> Stop in /usr/ports/sysutils/screen.
 >> *** Error code 1
 >>
 >> Stop in /usr/ports/sysutils/screen.
 >> Exit 1
 >=20
 > It's no rocket science to see that there's a problem here. The question=
  is why do I not see the same problem on my systems? Searching the workin=
 g directory for for WITH_MAN and WITH_INFO results in nothing matched:
 >=20
 > cwsys# find /export/wrkdir/usr/ports/sysutils/screen/work/screen-4.0.3 =
 -type f | xargs egrep 'WITH_MAN|WITH_INFO'
 > cwsys#=20
 >=20
 > NO_INFO and NO_MAN does match:
 >=20
 > cwsys# find /export/wrkdir/usr/ports/sysutils/screen/work/screen-4.0.3 =
 -type f | xargs egrep 'NO_MAN|NO_INFO'
 > /export/wrkdir/usr/ports/sysutils/screen/work/screen-4.0.3/doc/Makefile=
 =2Ein:.if !defined(NO_MAN)
 > /export/wrkdir/usr/ports/sysutils/screen/work/screen-4.0.3/doc/Makefile=
 =2Ein:.if !defined(NO_INFO)
 > /export/wrkdir/usr/ports/sysutils/screen/work/screen-4.0.3/doc/Makefile=
 =2Ein:.if !defined(NO_MAN)
 > /export/wrkdir/usr/ports/sysutils/screen/work/screen-4.0.3/doc/Makefile=
 =2Ein:.if !defined(NO_INFO)
 > /export/wrkdir/usr/ports/sysutils/screen/work/screen-4.0.3/doc/Makefile=
 :.if !defined(NO_MAN)
 > /export/wrkdir/usr/ports/sysutils/screen/work/screen-4.0.3/doc/Makefile=
 :.if !defined(NO_INFO)
 > /export/wrkdir/usr/ports/sysutils/screen/work/screen-4.0.3/doc/Makefile=
 :.if !defined(NO_MAN)
 > /export/wrkdir/usr/ports/sysutils/screen/work/screen-4.0.3/doc/Makefile=
 :.if !defined(NO_INFO)
 > cwsys#=20
 >=20
 > The question that needs to be answered is why does your site require WI=
 TH_MAN and WITH_INFO to be defined in MAKE_ARGS to work? I will test the =
 install out on my testbed in a pristine environment. What version of Free=
 BSD are you using? I'm able to boot anything from 5.5 to the latest 8.0-C=
 URRENT on it. Unless you tell me otherwise I will assume you are using 6.=
 2-STABLE.
 >=20
 >=20
 
 See my last e-mail. It was my fault, I had NO_INFO in make.conf.
 However, in this situation, the port doesn't handle that directive
 properly. Instead of not installing the INFO page, it aborts the
 installation.
 
 I have to leave for the week-end now. I'll keep investigating next week,
 unless I get some fix from you!
 
 In the meanwhile, have a nice week-end. Bye.
 
 --=20
 Pietro Cerutti
 
 PGP Public Key:
 http://gahr.ch/pgp
 
 
 --------------enig87BCA7960B39C5BDEBE3F851
 Content-Type: application/pgp-signature; name="signature.asc"
 Content-Description: OpenPGP digital signature
 Content-Disposition: attachment; filename="signature.asc"
 
 -----BEGIN PGP SIGNATURE-----
 Version: GnuPG v1.4.7 (FreeBSD)
 
 iD8DBQFHIh57wMJqmJVx944RCoElAJ9qHtcYlwBJ9v3eNK382x9AyoNkQgCfSElB
 N9xuhH4ZCBnGtfBYrt4wwBY=
 =ba73
 -----END PGP SIGNATURE-----
 
 --------------enig87BCA7960B39C5BDEBE3F851--

From: Cy Schubert <Cy.Schubert@komquats.com>
To: undisclosed-recipients: ;
Cc:  
Subject: Re: ports/117488: [patch] sysutils/screen enable installation of man
 page
Date: Fri, 26 Oct 2007 10:21:14 -0700

 ------- Blind-Carbon-Copy
 
 X-Mailer: exmh version 2.7.2 01/07/2005 with nmh-1.2
 Reply-to: Cy Schubert <Cy.Schubert@komquats.com>
 From: Cy Schubert <Cy.Schubert@komquats.com>
 X-os: FreeBSD
 X-Sender: cy@cwsent.com
 X-URL: http://www.komquats.com/
 To: Pietro Cerutti <gahr@gahr.ch>
 cc: Cy Schubert <Cy.Schubert@komquats.com>
 cc: freebsd-ports@freebsd.org
 Subject: Re: ports/117488: [patch] sysutils/screen enable installation of 
  man page
 In-Reply-To: Message from Pietro Cerutti <gahr@gahr.ch> 
    of "Fri, 26 Oct 2007 19:05:59 +0200." <47221E77.6030305@gahr.ch> 
 Mime-Version: 1.0
 Content-Type: text/plain; charset=us-ascii
 Date: Fri, 26 Oct 2007 10:21:14 -0700
 Sender: cy@cwsys
 
 In message <47221E77.6030305@gahr.ch>, Pietro Cerutti writes: 
 > See my last e-mail. It was my fault, I had NO_INFO in make.conf.
 > However, in this situation, the port doesn't handle that directive
 > properly. Instead of not installing the INFO page, it aborts the
 > installation.
 
 I consider this specific issue resolved.
 
 As far as ports detecting if options have been set in make.conf, that is a 
 larger issue than with just one port. The ports infrastructure relies on 
 settings in make.conf that do not negatively interfere with the 
 infrastructure or ports build process. Options address this issue though if 
 a person wishes to set a variable in make.conf for an application outside 
 of the ports infrastructure which could have a negative impact on a build, 
 it is possible for ports to unset any possible make or environment variable 
 that would be used during a build to avoid this situation.
 
 I'm CCing this to FreeBSD-ports@ to generate some discussion about the 
 issue in general and BCC to GNATS to maintain a record of this in the 
 database.
 
 
 - -- 
 Cheers,
 Cy Schubert <Cy.Schubert@komquats.com>
 FreeBSD UNIX:  <cy@FreeBSD.org>   Web:  http://www.FreeBSD.org
 
 			e**(i*pi)+1=0
 
 
 
 ------- End of Blind-Carbon-Copy

From: Cy Schubert <Cy.Schubert@komquats.com>
To: Pietro Cerutti <gahr@gahr.ch>
Cc: cy@FreeBSD.org, bug-followup@FreeBSD.org
Subject: Re: ports/117488: [patch] sysutils/screen enable installation of man
 page
Date: Fri, 26 Oct 2007 13:10:34 -0700

 In message <4721A085.7010101@gahr.ch>, Pietro Cerutti writes:
 > This is an OpenPGP/MIME signed message (RFC 2440 and 3156)
 > --------------enig045BC6F740992BBB42E9963D
 > Content-Type: text/plain; charset=UTF-8
 > Content-Transfer-Encoding: quoted-printable
 > 
 > cy@FreeBSD.org wrote:
 > > Synopsis: [patch] sysutils/screen enable installation of man page
 > >=20
 > > State-Changed-From-To: analyzed->closed
 > > State-Changed-By: cy
 > > State-Changed-When: Thu Oct 25 18:00:54 UTC 2007
 > > State-Changed-Why:=20
 > > Repaired the Makefile to install man pages (as per this PR) and info
 > > documents.
 > >=20
 > > http://www.freebsd.org/cgi/query-pr.cgi?pr=3D117488
 > 
 > Have you tested it?
 > The INFO part doesn't work.
 > 
 > Your solution seems to be logical: enable the .if defined(WITH_INFO)
 > part by including <bsd.port.pre.mk> before the directive.
 > It was also my first thought. However I cannot get the INFO part to work.=
 > 
 > 
 > I suggest that you check the output below, and revert to my original
 > patch, which solves the MAN part at least, until we get to a solution
 > for the INFO part.
 > 
 > Thank you,
 > 
 > 
 > =3D=3D=3D>  Installing for screen-4.0.3_1
 > =3D=3D=3D>   Generating temporary packing list
 > =3D=3D=3D>  Checking if sysutils/screen already installed
 > =2E/etc/mkinstalldirs /usr/local/bin `sed < config.h -n -e '/define
 > SCREENENCODINGS/s/^.*"\([^"]*\)"/\1/p'`
 > cd doc ; make  installdirs
 > =2E/../etc/mkinstalldirs /usr/local/man/man1
 > install  -s -o root -g wheel -m 555 screen /usr/local/bin/screen
 chown root:wheel /usr/local/bin/screen && chmod 4755 /usr/local/bin/scree=
 > n
 > cp ./utf8encodings/?? `sed < config.h -n -e '/define
 > SCREENENCODINGS/s/^.*"\([^"]*\)"/\1/p'`
 > cd doc ; make  install
 > =2E/../etc/mkinstalldirs /usr/local/man/man1
 > install  -o root -g wheel -m 444 ./screen.1 /usr/local/man/man1/screen.1
 > if [ -d /usr/lib/terminfo ]; then  PATH=3D"$PATH:/usr/5bin" tic
 > =2E/terminfo/screeninfo.src;  chmod 644 /usr/lib/terminfo/s/screen*;  fi
 > termcap entry (./terminfo/screencap) should be installed manually.
 > You may also want to install ./etc/etcscreenrc in /usr/local/etc/screenrc=
 > 
 > install  -o root -g wheel -m 444
 > /usr/ports/sysutils/screen/work/screen-4.0.3/doc/screen.1
 > /usr/local/man/man1
 > install-info --quiet /usr/local/info/screen.info /usr/local/info/dir
 > install-info: No such file or directory for /usr/local/info/screen.info
 > *** Error code 1
 > 
 > Stop in /usr/ports/sysutils/screen.
 > *** Error code 1
 > 
 > Stop in /usr/ports/sysutils/screen.
 > Exit 1
 
 I'm confused. Your reply time stamped 10:04:10 PDT says this was resolved due to an extraneous NO_INFO= in your make.conf.
 
 
 -- 
 Cheers,
 Cy Schubert <Cy.Schubert@komquats.com>
 FreeBSD UNIX:  <cy@FreeBSD.org>   Web:  http://www.FreeBSD.org
 
 			e**(i*pi)+1=0
 
 

From: Pietro Cerutti <gahr@gahr.ch>
To: Cy Schubert <Cy.Schubert@komquats.com>
Cc: cy@FreeBSD.org,  bug-followup@FreeBSD.org
Subject: Re: ports/117488: [patch] sysutils/screen enable installation of
 man page
Date: Sat, 27 Oct 2007 19:44:13 +0200

 Cy Schubert wrote:
 > In message <4721A085.7010101@gahr.ch>, Pietro Cerutti writes:
 >> This is an OpenPGP/MIME signed message (RFC 2440 and 3156)
 >> --------------enig045BC6F740992BBB42E9963D
 >> Content-Type: text/plain; charset=UTF-8
 >> Content-Transfer-Encoding: quoted-printable
 >>
 >> cy@FreeBSD.org wrote:
 >>> Synopsis: [patch] sysutils/screen enable installation of man page
 >>> =20
 >>> State-Changed-From-To: analyzed->closed
 >>> State-Changed-By: cy
 >>> State-Changed-When: Thu Oct 25 18:00:54 UTC 2007
 >>> State-Changed-Why:=20
 >>> Repaired the Makefile to install man pages (as per this PR) and info
 >>> documents.
 >>> =20
 >>> http://www.freebsd.org/cgi/query-pr.cgi?pr=3D117488
 >> Have you tested it?
 >> The INFO part doesn't work.
 >>
 >> Your solution seems to be logical: enable the .if defined(WITH_INFO)
 >> part by including <bsd.port.pre.mk> before the directive.
 >> It was also my first thought. However I cannot get the INFO part to work.=
 >>
 >>
 >> I suggest that you check the output below, and revert to my original
 >> patch, which solves the MAN part at least, until we get to a solution
 >> for the INFO part.
 >>
 >> Thank you,
 >>
 >>
 >> =3D=3D=3D>  Installing for screen-4.0.3_1
 >> =3D=3D=3D>   Generating temporary packing list
 >> =3D=3D=3D>  Checking if sysutils/screen already installed
 >> =2E/etc/mkinstalldirs /usr/local/bin `sed < config.h -n -e '/define
 >> SCREENENCODINGS/s/^.*"\([^"]*\)"/\1/p'`
 >> cd doc ; make  installdirs
 >> =2E/../etc/mkinstalldirs /usr/local/man/man1
 >> install  -s -o root -g wheel -m 555 screen /usr/local/bin/screen
 > chown root:wheel /usr/local/bin/screen && chmod 4755 /usr/local/bin/scree=
 >> n
 >> cp ./utf8encodings/?? `sed < config.h -n -e '/define
 >> SCREENENCODINGS/s/^.*"\([^"]*\)"/\1/p'`
 >> cd doc ; make  install
 >> =2E/../etc/mkinstalldirs /usr/local/man/man1
 >> install  -o root -g wheel -m 444 ./screen.1 /usr/local/man/man1/screen.1
 >> if [ -d /usr/lib/terminfo ]; then  PATH=3D"$PATH:/usr/5bin" tic
 >> =2E/terminfo/screeninfo.src;  chmod 644 /usr/lib/terminfo/s/screen*;  fi
 >> termcap entry (./terminfo/screencap) should be installed manually.
 >> You may also want to install ./etc/etcscreenrc in /usr/local/etc/screenrc=
 >>
 >> install  -o root -g wheel -m 444
 >> /usr/ports/sysutils/screen/work/screen-4.0.3/doc/screen.1
 >> /usr/local/man/man1
 >> install-info --quiet /usr/local/info/screen.info /usr/local/info/dir
 >> install-info: No such file or directory for /usr/local/info/screen.info
 >> *** Error code 1
 >>
 >> Stop in /usr/ports/sysutils/screen.
 >> *** Error code 1
 >>
 >> Stop in /usr/ports/sysutils/screen.
 >> Exit 1
 > 
 > I'm confused. Your reply time stamped 10:04:10 PDT says this was resolved due to an extraneous NO_INFO= in your make.conf.
 
 You are right. I haven't been clear enough lately.
 This specific issue is to be considered solved with Makefile version 1.69.
 
 However, there is another issue which - as you said - doesn't really 
 belong here. This other issue is about the screen port, and perhaps 
 other as well (I haven't tested, maybe the whole ports infrastructure 
 suffers from it), doesn't handle make.conf options properly.
 
 In this specific case, if I have a NO_INFO= line in make.conf and try to 
 build screen with the INFO option enabled, the build fails.
 Instead, a general policy should be chosen, either a) the port doesn't 
 install the INFO file because make.conf says so or b) the port installs 
 the INFO file because the user says so.
 
 I don't see c) "the port fails to build because make.conf and options 
 don't agree" as an option.
 
 Anyway, I would close this thread here to avoid spamming on GNATS.
 
 If you're interested in discussing this issue further, I suggest that we 
 open a thread somewhere else (freebsd-ports@). I'll be available by Monday.
 
 Best Regards,
 
 
 
>Unformatted:
 >> no output <<
 
