sysutils/screen -- INFO issue

Pietro Cerutti gahr at
Mon Oct 29 14:46:29 PDT 2007

Dear Cy, dear list

here I am to talk about the INFO issue of sysutils/screen [screen].

To summarize:
screen has an option to [en|dis]able installation of an INFO page.
If the option is set, WITH_INFO gets set accordingly, otherwise the
port's Makefile sets the make argument NO_INFO.
In turn, ${WRKSRC}/doc/Makefile checks for NO_INFO, and if it's not set,
it prepares the INFO page for installation by running
makeinfo --no-split ./screen.texinfo -o

The problem is that make.conf(5) documents the variable NO_INFO under
the section "BUILDING THE WORLD - The following list provides a name and
short description for variables that are used during the world build".

I think here we have a namespace collision problem for the variable NO_INFO.

If NO_INFO is set in make.conf and the WITH_INFO option is enabled while
building screen, ${WRKSRC}/doc/Makefile won't prepare the info page but
the port's Makefile will try to install it anyway. This leads to a "file
not found" build error:

install-info --quiet /usr/local/info/ /usr/local/info/dir
install-info: No such file or directory for /usr/local/info/
*** Error code 1

I suggest that we change ${WRKSRC}/doc/Makefile to check for WITH_INFO
(used within ports) instead of !NO_INFO (used while building the world).
In addition, we should change the port's Makefile to set the make
argument WITH_INFO if the option was chosen instead of setting NO_INFO
it it wasn't.

The same whole issue exists for the MAN page, for which I suggest the
same modifications in the port's Makefile and in ${WRKSRC}/doc/Makefile.

The following patches to the port's Makefile and for
files/ implement the changes discussed above.

--- Makefile.orig	2007-10-29 19:18:41.000000000 +0100
+++ Makefile	2007-10-29 20:15:20.000000000 +0100
@@ -31,16 +31,14 @@
 .if defined(WITH_MAN)
 MAN1=		screen.1
 .if defined(WITH_INFO)
 .if !exists(/usr/bin/install-info)
 RUN_DEPENDS+=	install-info:${PORTSDIR}/print/texinfo
 INFO=		screen

 .if ${ARCH} == "alpha"
--- files/	2007-10-29 19:40:13.000000000 +0100
+++ files/	2007-10-29 20:11:43.000000000 +0100
@@ -4,10 +4,10 @@
  	$(MAKEINFO) $(srcdir)/screen.texinfo -o

  install: installdirs
-+.if !defined(NO_MAN)
++.if defined(WITH_MAN)
  	$(INSTALL_DATA) $(srcdir)/screen.1 $(DESTDIR)$(mandir)/man1/screen.1
-+.if !defined(NO_INFO)
++.if defined(WITH_INFO)
  	-if test -f; then d=.; else d=$(srcdir); fi; \
  	if test -f $$d/; then \
@@ -23,10 +23,10 @@

 -	$(srcdir)/../etc/mkinstalldirs $(DESTDIR)$(mandir)/man1
-+.if !defined(NO_MAN)
++.if defined(WITH_MAN)
 +	$(srcdir)/../etc/mkinstalldirs $(DESTDIR)$(mandir)/man1
-+.if !defined(NO_INFO)
++.if defined(WITH_INFO)
 +	$(srcdir)/../etc/mkinstalldirs $(DESTDIR)$(infodir)

I would greatly appreciate input / clarification / feedback / ideas on
the whole story before filling a PR.

Best Regards,

Pietro Cerutti

PGP Public Key:

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 187 bytes
Desc: OpenPGP digital signature
Url :

More information about the freebsd-ports mailing list