Received: from mail.netbsd.org (mail.netbsd.org [199.233.217.200])
	(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits))
	(Client CN "mail.netbsd.org", Issuer "Postmaster NetBSD.org" (verified OK))
	by mollari.NetBSD.org (Postfix) with ESMTPS id 0A0E97A46C
	for <mhonarc+pkgsrc-changes@mail-index.netbsd.org>; Sat,  9 Jul 2016 16:07:38 +0000 (UTC)
Received: by mail.netbsd.org (Postfix, from userid 605)
	id 783C185EA2; Sat,  9 Jul 2016 16:07:37 +0000 (UTC)
Received: from localhost (localhost [127.0.0.1])
	by mail.netbsd.org (Postfix) with ESMTP id 0AA5085E74
	for <pkgsrc-changes@NetBSD.org>; Sat,  9 Jul 2016 16:07:37 +0000 (UTC)
X-Virus-Scanned: amavisd-new at netbsd.org
Received: from mail.netbsd.org ([127.0.0.1])
	by localhost (mail.netbsd.org [127.0.0.1]) (amavisd-new, port 10025)
	with ESMTP id JG2p27yH-8Mo for <pkgsrc-changes@netbsd.org>;
	Sat,  9 Jul 2016 16:07:36 +0000 (UTC)
Received: from cvs.NetBSD.org (ivanova.netbsd.org [199.233.217.197])
	by mail.netbsd.org (Postfix) with ESMTP id 0348684CE9
	for <pkgsrc-changes@NetBSD.org>; Sat,  9 Jul 2016 16:07:35 +0000 (UTC)
Received: by cvs.NetBSD.org (Postfix, from userid 500)
	id E4161FBB5; Sat,  9 Jul 2016 16:07:35 +0000 (UTC)
Content-Transfer-Encoding: 7bit
Content-Type: multipart/mixed; boundary="_----------=_146808045510330"
MIME-Version: 1.0
Date: Sat, 9 Jul 2016 16:07:35 +0000
From: "Roland Illig" <rillig@netbsd.org>
Subject: CVS commit: pkgsrc/doc/guide/files
To: pkgsrc-changes@NetBSD.org
Reply-To: rillig@netbsd.org
X-Mailer: log_accum
Message-Id: <20160709160735.E4161FBB5@cvs.NetBSD.org>
Sender: pkgsrc-changes-owner@NetBSD.org
List-Id: pkgsrc-changes.NetBSD.org
Precedence: bulk

This is a multi-part message in MIME format.

--_----------=_146808045510330
Content-Disposition: inline
Content-Transfer-Encoding: 8bit
Content-Type: text/plain; charset="US-ASCII"

Module Name:	pkgsrc
Committed By:	rillig
Date:		Sat Jul  9 16:07:35 UTC 2016

Modified Files:
	pkgsrc/doc/guide/files: fixes.xml gnome.xml plist.xml

Log Message:
Moved <programlisting> elements to column 1, since they generate an
additional empty line otherwise.


To generate a diff of this commit:
cvs rdiff -u -r1.139 -r1.140 pkgsrc/doc/guide/files/fixes.xml
cvs rdiff -u -r1.9 -r1.10 pkgsrc/doc/guide/files/gnome.xml
cvs rdiff -u -r1.22 -r1.23 pkgsrc/doc/guide/files/plist.xml

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.


--_----------=_146808045510330
Content-Disposition: inline
Content-Length: 15161
Content-Transfer-Encoding: binary
Content-Type: text/x-diff; charset=us-ascii

Modified files:

Index: pkgsrc/doc/guide/files/fixes.xml
diff -u pkgsrc/doc/guide/files/fixes.xml:1.139 pkgsrc/doc/guide/files/fixes.xml:1.140
--- pkgsrc/doc/guide/files/fixes.xml:1.139	Sat Jul  9 15:19:21 2016
+++ pkgsrc/doc/guide/files/fixes.xml	Sat Jul  9 16:07:35 2016
@@ -1,4 +1,4 @@
-<!-- $NetBSD: fixes.xml,v 1.139 2016/07/09 15:19:21 rillig Exp $ -->
+<!-- $NetBSD: fixes.xml,v 1.140 2016/07/09 16:07:35 rillig Exp $ -->
 
 <chapter id="fixes"> <?dbhtml filename="fixes.html"?>
 <title>Making your package work</title>
@@ -76,9 +76,9 @@
     variable to define which stages need interaction. This should be
     done in the package's <filename>Makefile</filename>, e.g.:</para>
 
-    <programlisting>
+<programlisting>
 INTERACTIVE_STAGE=      configure install
-    </programlisting>
+</programlisting>
 
     <para>The user can then decide to skip this package by setting the
     <varname>BATCH</varname> variable. Packages that require interaction
@@ -125,22 +125,22 @@ INTERACTIVE_STAGE=      configure instal
     license, e.g. in <filename
     role="pkg">graphics/xv</filename>:</para>
 
-    <programlisting>
+<programlisting>
 LICENSE=        xv-license
-    </programlisting>
+</programlisting>
 
     <para>When trying to build, the user will get a notice that the
     package is covered by a license which has not been placed in the
     <varname>ACCEPTABLE_LICENSES</varname> variable:</para>
 
-    <programlisting>
+<programlisting>
 &cprompt; <userinput>make</userinput>
 ===> xv-3.10anb9 has an unacceptable license: xv-license.
 ===>     To view the license, enter "/usr/bin/make show-license".
 ===>     To indicate acceptance, add this line to your /etc/mk.conf:
 ===>     ACCEPTABLE_LICENSES+=xv-license
 *** Error code 1
-    </programlisting>
+</programlisting>
 
     <para>The license can be viewed with <command>make
     show-license</command>, and if the user so chooses, the line
@@ -148,9 +148,9 @@ LICENSE=        xv-license
     convey to pkgsrc that it should not in the future fail because of
     that license:</para>
 
-    <programlisting>
+<programlisting>
 ACCEPTABLE_LICENSES+=xv-license
-    </programlisting>
+</programlisting>
 
     <para>The use of <varname>LICENSE=shareware</varname>,
     <varname>LICENSE=no-commercial-use</varname>, and similar language
@@ -311,9 +311,9 @@ ACCEPTABLE_LICENSES+=xv-license
     <para>The format for a <varname>BUILD_DEPENDS</varname> and a
     <varname>DEPENDS</varname> definition is:</para>
 
-    <programlisting>
+<programlisting>
 &lt;pre-req-package-name&gt;:../../&lt;category&gt;/&lt;pre-req-package&gt;
-    </programlisting>
+</programlisting>
 
     <para>Please note that the <quote>pre-req-package-name</quote>
     may include any of the wildcard version numbers recognized by
@@ -442,15 +442,15 @@ DEPENDS+=       ImageMagick>=6.0:../../g
     use the same config file, you would set in
     <filename>foo/bar/Makefile</filename>:</para>
 
-    <programlisting>
+<programlisting>
 CONFLICTS=      baz-[0-9]*
-    </programlisting>
+</programlisting>
 
     <para>and in <filename>pkgsrc/foo/baz/Makefile</filename>:</para>
 
-    <programlisting>
+<programlisting>
 CONFLICTS=      bar-[0-9]*
-    </programlisting>
+</programlisting>
 
   </sect2>
 
@@ -543,10 +543,10 @@ CONFLICTS=      bar-[0-9]*
     (2, ...). The <quote>nb</quote> is treated like a
     <quote>.</quote> by the package tools. e.g.</para>
 
-    <programlisting>
+<programlisting>
 DISTNAME=       foo-17.42
 PKGREVISION=    9
-    </programlisting>
+</programlisting>
 
     <para>will result in a <varname>PKGNAME</varname> of
     <quote>foo-17.42nb9</quote>. If you want to use the original
@@ -558,9 +558,9 @@ PKGREVISION=    9
     <varname>PKGREVISION</varname> should be removed, e.g. on a new
     minor release of the above package, things should be like:</para>
 
-    <programlisting>
+<programlisting>
 DISTNAME=       foo-17.43
-    </programlisting>
+</programlisting>
 
     <para><varname>PKGREVISION</varname> should be incremented for any
     non-trivial change in the resulting binary package.  Without a
@@ -605,7 +605,7 @@ DISTNAME=       foo-17.43
     easy-to-use interface for replacing text in files.
     Example:</para>
 
-    <programlisting>
+<programlisting>
 SUBST_CLASSES+=                 fix-paths
 SUBST_STAGE.fix-paths=          pre-configure
 SUBST_MESSAGE.fix-paths=        Fixing absolute paths.
@@ -613,7 +613,7 @@ SUBST_FILES.fix-paths=          src/*.c
 SUBST_FILES.fix-paths+=         scripts/*.sh
 SUBST_SED.fix-paths=            -e 's,"/usr/local,"${PREFIX},g'
 SUBST_SED.fix-paths+=           -e 's,"/var/log,"${VARBASE}/log,g'
-    </programlisting>
+</programlisting>
 
     <para><varname>SUBST_CLASSES</varname> is a list of identifiers
     that are used to identify the different SUBST blocks that are
@@ -685,11 +685,11 @@ SUBST_SED.fix-paths+=           -e 's,"/
     <varname>FETCH_MESSAGE</varname> to a list of lines that are
     displayed to the user before aborting the build. Example:</para>
 
-    <programlisting>
+<programlisting>
 FETCH_MESSAGE=  "Please download the files"
 FETCH_MESSAGE+= "    "${DISTFILES:Q}
 FETCH_MESSAGE+= "manually from "${MASTER_SITES:Q}"."
-    </programlisting>
+</programlisting>
 
   </sect2>
 
@@ -737,7 +737,7 @@ FETCH_MESSAGE+= "manually from "${MASTER
       <para>
         If your distfile URL looks similar to <literal>http://github.com/username/exampleproject/archive/v1.0.zip</literal>, then you are packaging a tagged release.
       </para>
-      <programlisting>
+<programlisting>
 DISTNAME=       exampleproject-1.0
 MASTER_SITES=   ${MASTER_SITE_GITHUB:=username/}
 #GITHUB_PROJECT=        # can be omitted if same as DISTNAME
@@ -750,26 +750,26 @@ EXTRACT_SUFX=   .zip
       <para>
         If your distfile URL looks similar to <literal>http://github.com/example/example/archive/988881adc9fc3655077dc2d4d757d480b5ea0e11.tar.gz</literal>, then you are packaging a specific commit not tied to a release.
       </para>
-      <programlisting>
+<programlisting>
 DISTNAME=       example-1.0
 MASTER_SITES=   ${MASTER_SITE_GITHUB:=example/}
 #GITHUB_PROJECT=        # can be omitted if same as DISTNAME
 GITHUB_TAG=     988881adc9fc3655077dc2d4d757d480b5ea0e11
-      </programlisting>
+</programlisting>
     </sect3>
     <sect3 id="build.fetch.github.release">
       <title>Fetch based on release</title>
       <para>
         If your distfile URL looks similar to <literal>http://github.com/username/exampleproject/releases/download/rel-1.6/offensive-1.6.zip</literal>, then you are packaging a release.
       </para>
-      <programlisting>
+<programlisting>
 DISTNAME=       offensive-1.6
 PKGNAME=        ${DISTNAME:S/offensive/proper/}
 MASTER_SITES=   ${MASTER_SITE_GITHUB:=username/}
 GITHUB_PROJECT= exampleproject
 GITHUB_RELEASE= rel-${PKGVERSION_NOREV} # usually just set this to ${DISTNAME}
 EXTRACT_SUFX=   .zip
-      </programlisting>
+</programlisting>
     </sect3>
   </sect2>
 </sect1>
@@ -1000,7 +1000,7 @@ ${LIBTOOL} --mode=install ${BSD_INSTALL_
 
     <para>For packages that need only autoconf:</para>
 
-    <programlisting>
+<programlisting>
 AUTOCONF_REQD=  2.50            # if default version is not good enough
 USE_TOOLS+=     autoconf        # use "autoconf213" for autoconf-2.13
 ...
@@ -1009,11 +1009,11 @@ pre-configure:
         cd ${WRKSRC} &amp;&amp; autoconf
 
 ...
-    </programlisting>
+</programlisting>
 
     <para>and for packages that need automake and autoconf:</para>
 
-    <programlisting>
+<programlisting>
 AUTOMAKE_REQD=  1.7.1           # if default version is not good enough
 USE_TOOLS+=     automake        # use "automake14" for automake-1.4
 ...
@@ -1023,7 +1023,7 @@ pre-configure:
         aclocal; autoheader; automake -a --foreign -i; autoconf
 
 ...
-    </programlisting>
+</programlisting>
 
     <para>Packages which use GNU Automake will almost certainly
     require GNU Make.</para>
@@ -1337,10 +1337,10 @@ of functions.</para>
     directory at a time. As such, you should call
     <literal>${INSTALL_*_DIR}</literal> like this:</para>
 
-    <programlisting>
+<programlisting>
 ${INSTALL_DATA_DIR} ${PREFIX}/dir1
 ${INSTALL_DATA_DIR} ${PREFIX}/dir2
-    </programlisting>
+</programlisting>
 
     <para>You can also just append <quote><literal>dir1
     dir2</literal></quote> to the
@@ -1408,7 +1408,7 @@ ${INSTALL_DATA_DIR} ${PREFIX}/dir2
     and <varname>SPECIAL_PERMS</varname> is used to install setgid the game
     binary:</para>
 
-    <programlisting>
+<programlisting>
 USE_GAMESGROUP=		yes
 
 BUILD_DEFS+=		VARBASE
@@ -1416,7 +1416,7 @@ BUILD_DEFS+=		VARBASE
 OWN_DIRS_PERMS+=	${VARBASE}/games/moon-buggy ${GAMEDIR_PERMS}
 REQD_FILES_PERMS+=	/dev/null ${VARBASE}/games/moon-buggy/mbscore ${GAMEDATA_PERMS}
 SPECIAL_PERMS+=		${PREFIX}/bin/moon-buggy ${SETGID_GAMES_PERMS}
-    </programlisting>
+</programlisting>
 
     <para>Various <varname>INSTALL_*</varname> variables are also available:
     <varname>INSTALL_GAME</varname> to install setgid game binaries,
@@ -1477,13 +1477,13 @@ SPECIAL_PERMS+=		${PREFIX}/bin/moon-bugg
       following definitions in your <filename>Makefile</filename> (we
     shall use <command>tclsh</command> in this example):</para>
 
-    <programlisting>
+<programlisting>
 REPLACE_INTERPRETER+=   tcl
 REPLACE.tcl.old=        .*/bin/tclsh
 REPLACE.tcl.new=        ${PREFIX}/bin/tclsh
 REPLACE_FILES.tcl=      # list of tcl scripts which need to be fixed,
 # relative to ${WRKSRC}, just as in REPLACE_PERL
-    </programlisting>
+</programlisting>
 
     <note><para>Before March 2006, these variables were called
     <varname>_REPLACE.*</varname> and
@@ -1514,9 +1514,9 @@ REPLACE_FILES.tcl=      # list of tcl sc
     (<varname>PERL5_INSTALLVENDORARCH</varname> by default),
     e.g.:</para>
 
-    <programlisting>
+<programlisting>
 PERL5_PACKLIST= auto/Pg/.packlist
-    </programlisting>
+</programlisting>
 
     <para>The perl5 config variables
     <varname>installarchlib</varname>,

Index: pkgsrc/doc/guide/files/gnome.xml
diff -u pkgsrc/doc/guide/files/gnome.xml:1.9 pkgsrc/doc/guide/files/gnome.xml:1.10
--- pkgsrc/doc/guide/files/gnome.xml:1.9	Fri Apr  3 01:52:24 2009
+++ pkgsrc/doc/guide/files/gnome.xml	Sat Jul  9 16:07:35 2016
@@ -1,4 +1,4 @@
-<!-- $NetBSD: gnome.xml,v 1.9 2009/04/03 01:52:24 snj Exp $ -->
+<!-- $NetBSD: gnome.xml,v 1.10 2016/07/09 16:07:35 rillig Exp $ -->
 
 <chapter id="gnome"> <?dbhtml filename="gnome.html"?>
 <title>GNOME packaging and porting</title>
@@ -108,7 +108,7 @@ give you a general idea on the minimum r
     build system.  As a general rule you will need to tell this to your
     package:</para>
 
-    <programlisting>
+<programlisting>
 GNU_CONFIGURE=yes
 USE_LIBTOOL=yes
 USE_TOOLS+=gmake
@@ -119,7 +119,9 @@ USE_TOOLS+=gmake
     <para>If the package uses pkg-config to detect dependencies, add this
     tool to the list of required utilities:</para>
 
-    <programlisting>USE_TOOLS+=pkg-config</programlisting>
+<programlisting>
+USE_TOOLS+=pkg-config
+</programlisting>
 
     <para>Also use <filename role="pkg">pkgtools/verifypc</filename> at
     the end of the build process to ensure that you did not miss to
@@ -142,7 +144,9 @@ USE_TOOLS+=gmake
     report.  For such packages you should disable gtk-doc (unless it is
     the default):</para>
 
-    <programlisting>CONFIGURE_ARGS+=--disable-gtk-doc</programlisting>
+<programlisting>
+CONFIGURE_ARGS+=--disable-gtk-doc
+</programlisting>
 
     <para>The default location of installed HTML files
     (<filename>share/gtk-doc/&lt;package-name&gt;</filename>) is correct
@@ -151,7 +155,10 @@ USE_TOOLS+=gmake
     <command>devhelp</command> will not be able to open them.  You can
     do that with an entry similar to:</para>
 
-    <programlisting>CONFIGURE_ARGS+=--with-html-dir=${PREFIX}/share/gtk-doc/...</programlisting>
+<programlisting>
+CONFIGURE_ARGS+=--with-html-dir=${PREFIX}/share/gtk-doc/...
+</programlisting>
+
   </listitem>
 </itemizedlist>
 

Index: pkgsrc/doc/guide/files/plist.xml
diff -u pkgsrc/doc/guide/files/plist.xml:1.22 pkgsrc/doc/guide/files/plist.xml:1.23
--- pkgsrc/doc/guide/files/plist.xml:1.22	Sun Jun  7 03:34:54 2015
+++ pkgsrc/doc/guide/files/plist.xml	Sat Jul  9 16:07:35 2016
@@ -1,4 +1,4 @@
-<!-- $NetBSD: plist.xml,v 1.22 2015/06/07 03:34:54 joerg Exp $ -->
+<!-- $NetBSD: plist.xml,v 1.23 2016/07/09 16:07:35 rillig Exp $ -->
 
 <chapter id="plist">
   <title>PLIST issues</title>
@@ -19,9 +19,9 @@
     <para>Be sure to add a RCS ID line as the first thing in any
     <filename>PLIST</filename> file you write:</para>
 
-    <programlisting>
+<programlisting>
 @comment &#36;NetBSD&#36;
-    </programlisting>
+</programlisting>
   </sect1>
 
   <sect1 id="automatic-plist-generation">
@@ -46,9 +46,9 @@
     <filename>libdata/foo</filename> directory removed from the
     resulting PLIST:</para>
 
-    <programlisting>
+<programlisting>
 PRINT_PLIST_AWK+=       /^libdata\/foo/ { next; }
-    </programlisting>
+</programlisting>
   </sect1>
 
   <sect1 id="plist.misc">
@@ -123,9 +123,9 @@ well as searching the <filename>pkgsrc/m
     following way, similar to <varname>MESSAGE_SUBST</varname> (see <xref
     linkend="components.optional"/>):</para>
 
-    <programlisting>
+<programlisting>
 PLIST_SUBST+=   SOMEVAR="somevalue"
-    </programlisting>
+</programlisting>
 
     <para>This replaces all occurrences of <quote>${SOMEVAR}</quote>
     in the <filename>PLIST</filename> with
@@ -144,22 +144,22 @@ PLIST_SUBST+=   SOMEVAR="somevalue"
     <quote><literal>"@comment "</literal></quote>.
     For example, in <filename>Makefile</filename>:</para>
 
-    <programlisting>
+<programlisting>
 PLIST_VARS+=	foo
 .if <replaceable>condition</replaceable>
 PLIST.foo=	yes
 .else
-    </programlisting>
+</programlisting>
 
     <para>And then in <filename>PLIST</filename>:</para>
-    <programlisting>
+<programlisting>
 @comment &#36;NetBSD&#36;
 bin/bar
 man/man1/bar.1
 ${PLIST.foo}bin/foo
 ${PLIST.foo}man/man1/foo.1
 ${PLIST.foo}share/bar/foo.data
-    </programlisting>
+</programlisting>
 
   </sect1>
 
@@ -230,9 +230,11 @@ ${PLIST.foo}share/bar/foo.data
     terminated (with a semicolon) that will output PLIST entries which
     will be appended to the PLIST</para>
     <para>You can find one example in editors/xemacs:</para>
-    <programlisting>
-      GENERATE_PLIST+=        ${ECHO} bin/${DISTNAME}-`${WRKSRC}/src/xemacs -sd`.dmp ;
-    </programlisting>
+
+<programlisting>
+GENERATE_PLIST+=        ${ECHO} bin/${DISTNAME}-`${WRKSRC}/src/xemacs -sd`.dmp ;
+</programlisting>
+
     <para>which will append something like
     <filename>bin/xemacs-21.4.23-54e8ea71.dmp</filename> to the
     <filename>PLIST</filename>.
@@ -256,9 +258,11 @@ ${PLIST.foo}share/bar/foo.data
     <para>If a package needs an empty directory to work, create
     the directory during installation as usual, and also add an
     entry to the PLIST:
-    <programlisting>
+
+<programlisting>
 @pkgdir path/to/empty/directory
-    </programlisting>
+</programlisting>
+
     or take a look at <varname>MAKE_DIRS</varname> and
     <varname>OWN_DIRS</varname>.
     </para>


--_----------=_146808045510330--