From owner-freebsd-gnome@FreeBSD.ORG  Thu Jul  1 05:30:51 2010
Return-Path: <owner-freebsd-gnome@FreeBSD.ORG>
Delivered-To: gnome@FreeBSD.org
Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34])
	by hub.freebsd.org (Postfix) with ESMTP id E974F106566C
	for <gnome@FreeBSD.org>; Thu,  1 Jul 2010 05:30:51 +0000 (UTC)
	(envelope-from truckman@FreeBSD.org)
Received: from gw.catspoiler.org (adsl-75-1-14-242.dsl.scrm01.sbcglobal.net
	[75.1.14.242]) by mx1.freebsd.org (Postfix) with ESMTP id B95778FC12
	for <gnome@FreeBSD.org>; Thu,  1 Jul 2010 05:30:51 +0000 (UTC)
Received: from FreeBSD.org (mousie.catspoiler.org [192.168.101.2])
	by gw.catspoiler.org (8.13.3/8.13.3) with ESMTP id o615UghH058792;
	Wed, 30 Jun 2010 22:30:46 -0700 (PDT)
	(envelope-from truckman@FreeBSD.org)
Message-Id: <201007010530.o615UghH058792@gw.catspoiler.org>
Date: Wed, 30 Jun 2010 22:30:42 -0700 (PDT)
From: Don Lewis <truckman@FreeBSD.org>
To: mezz7@cox.net
In-Reply-To: <op.vet2okvm9aq2h7@localhost>
MIME-Version: 1.0
Content-Type: TEXT/plain; charset=us-ascii
Cc: gnome@FreeBSD.org
Subject: Re: ports/148102: make install can fail for ports-mgmt/packagekit
 [patch]
X-BeenThere: freebsd-gnome@freebsd.org
X-Mailman-Version: 2.1.5
Precedence: list
List-Id: GNOME for FreeBSD -- porting and maintaining
	<freebsd-gnome.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/listinfo/freebsd-gnome>,
	<mailto:freebsd-gnome-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/freebsd-gnome>
List-Post: <mailto:freebsd-gnome@freebsd.org>
List-Help: <mailto:freebsd-gnome-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/freebsd-gnome>,
	<mailto:freebsd-gnome-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Thu, 01 Jul 2010 05:30:52 -0000

On 24 Jun, Jeremy Messenger wrote:
> On Thu, 24 Jun 2010 17:38:36 -0500, Don Lewis <truckman@freebsd.org> wrote:

>> make.out:xsltproc  
>> http://docbook.sourceforge.net/release/xsl/current/manpages/docbook.xsl  
>> pkcon.xml &> /dev/null
>> make.out:xsltproc  
>> http://docbook.sourceforge.net/release/xsl/current/manpages/docbook.xsl  
>> pkmon.xml &> /dev/null
>> make.out:xsltproc  
>> http://docbook.sourceforge.net/release/xsl/current/manpages/docbook.xsl  
>> pkgenpack.xml &> /dev/null
>> make.out:xsltproc  
>> http://docbook.sourceforge.net/release/xsl/current/manpages/docbook.xsl  
>> pk-debuginfo-install.xml &> /dev/null
>>
>> The others instances are in contrib/device-rebind/ and client/, though
>> the former doesn't appear to get executed.
> 
> Quick way would be add in the post-patch something looks like this:
> 
> 	@${FIND} ${WRKSRC} -name Makefile.in | ${XARGS} ${REINPLACE_CMD} -e \
> 		's|.xml [$$]? &> /dev|.xml $$? > /dev|g'
> 
> It's untest, I will test it this weekend and see if it works.

It should be possible to trigger the bug by unplugging your internet
connection during "make build".  xsltproc won't be able to download the
stylesheet, but make won't detect the failure because xsltproc is
running in the background.  During the subsequent "make install", make
will detect that the man page is missing and will run xsltproc again.
Because xsltproc is being run in the background, its output might not
get created before make tries to install it, causing the install to
fail.

It seems rather unfriendly and unreliable to repeatedly download the
stylesheet for each man page.  It should probably be done once as a
separate port that is listed as a build dependency.  This probably
affects some huge number of ports ...