X-Recipient: archive-cygwin AT delorie DOT com
X-SWARE-Spam-Status: No, hits=-5.2 required=5.0	tests=AWL,BAYES_00,DKIM_SIGNED,DKIM_VALID,FREEMAIL_FROM,KHOP_RCVD_TRUST,KHOP_THREADED,RCVD_IN_DNSWL_LOW,RCVD_IN_HOSTKARMA_YE,TW_RX,TW_YG
X-Spam-Check-By: sourceware.org
Message-ID: <5033F431.5040402@users.sourceforge.net>
Date: Tue, 21 Aug 2012 15:48:49 -0500
From: "Yaakov (Cygwin/X)" <yselkowitz AT users DOT sourceforge DOT net>
User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:14.0) Gecko/20120713 Thunderbird/14.0
MIME-Version: 1.0
To: cygwin AT cygwin DOT com
Subject: Re: Oddity with cygport, debuginfo generation, and rxvt-unicode
References: <5033C360 DOT 1060007 AT cwilson DOT fastmail DOT fm>
In-Reply-To: <5033C360.1060007@cwilson.fastmail.fm>
Content-Type: text/plain; charset=ISO-8859-1; format=flowed
Content-Transfer-Encoding: 7bit
Mailing-List: contact cygwin-help AT cygwin DOT com; run by ezmlm
List-Id: <cygwin.cygwin.com>
List-Subscribe: <mailto:cygwin-subscribe AT cygwin DOT com>
List-Archive: <http://sourceware.org/ml/cygwin/>
List-Post: <mailto:cygwin AT cygwin DOT com>
List-Help: <mailto:cygwin-help AT cygwin DOT com>, <http://sourceware.org/ml/#faqs>
Sender: cygwin-owner AT cygwin DOT com
Mail-Followup-To: cygwin AT cygwin DOT com
Delivered-To: mailing list cygwin AT cygwin DOT com

On 2012-08-21 12:20, Charles Wilson wrote:
> While rebuilding rxvt-unicode, I saw the following error during 'cygport
> install':
>
> Preparing debuginfo source files:
> realpath: unknown option -- p
> Try `realpath --help' for more information.
> 	80 files
> Preparing executables for UAC:
>
>
> A little debugging showed that realpath is being invoked during
> src_postinstall here:
>
> line 978:do
>                   # make sure that paths with '../' don't leave srcdir
>                   case $(realpath -m ${dbgsrc}) in
>                           ${dbgsrcdir}/*) ;;
>                           *)  continue ;;
>                   esac
>
>                   srcf=${dbgsrc#${dbgsrcdir}/}

FWIW, readlink is now used in git instead of realpath, as the latter is 
not available in Fedora 16 or EL6, but the result should be the same.

> I see that this line is hit many times, but here is the relevant sequence:
>
> ...
> realpath -m <iom_perl.xs
> realpath -m -pe
> realpath: unknown option -- p
> Try `realpath --help' for more information.
> realpath -m s/IOM_MODULE/urxvt/g,s/IOM_CLASS/urxvt/g
> realpath -m |
> realpath -m /usr/src/debug/rxvt-unicode-X-9.07-2/src/../libev/ev++.h
> ...
>
> Although only the second of these lines triggers the error message, each
> of the first four is problematic.
>
> This looks like a bug in gcc/binutils to me; debug src paths shouldn't
> include pipe line shell commands, should they?

Not AFAIK.  That is very odd, but I am able to duplicate it with your 
current package.

> In the meantime, I think a fix like this for cygport would be appropriate:

Could you confirm that just this change fixes this:

-          case $(realpath -m ${dbgsrc}) in
+          case $(readlink -m -- "${dbgsrc}") in

> ...although I'm not sure if this might cause a problem if a filename
> actually contains a shell redirection metacharacter (and the originating
> project is always careful to appropriate quote that filename).

And when was the last time you saw that?


Yaakov


--
Problem reports:       http://cygwin.com/problems.html
FAQ:                   http://cygwin.com/faq/
Documentation:         http://cygwin.com/docs.html
Unsubscribe info:      http://cygwin.com/ml/#unsubscribe-simple