Newsgroups: php.internals Path: news.php.net Xref: news.php.net php.internals:37649 Return-Path: Mailing-List: contact internals-help@lists.php.net; run by ezmlm Delivered-To: mailing list internals@lists.php.net Received: (qmail 24073 invoked from network); 14 May 2008 14:05:24 -0000 Received: from unknown (HELO lists.php.net) (127.0.0.1) by localhost with SMTP; 14 May 2008 14:05:24 -0000 Authentication-Results: pb1.pair.com header.from=rquadling@googlemail.com; sender-id=pass; domainkeys=bad Authentication-Results: pb1.pair.com smtp.mail=rquadling@googlemail.com; spf=pass; sender-id=pass Received-SPF: pass (pb1.pair.com: domain googlemail.com designates 64.233.182.189 as permitted sender) DomainKey-Status: bad X-DomainKeys: Ecelerity dk_validate implementing draft-delany-domainkeys-base-01 X-PHP-List-Original-Sender: rquadling@googlemail.com X-Host-Fingerprint: 64.233.182.189 nf-out-0910.google.com Received: from [64.233.182.189] ([64.233.182.189:42149] helo=nf-out-0910.google.com) by pb1.pair.com (ecelerity 2.1.1.9-wez r(12769M)) with ESMTP id 1A/E4-31980-3A1FA284 for ; Wed, 14 May 2008 10:05:24 -0400 Received: by nf-out-0910.google.com with SMTP id b11so1446552nfh.13 for ; Wed, 14 May 2008 07:05:21 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=googlemail.com; s=gamma; h=domainkey-signature:received:received:message-id:date:from:reply-to:to:subject:cc:in-reply-to:mime-version:content-type:content-transfer-encoding:content-disposition:references; bh=LRiC5CHLwcqlsDvKoW7pEnWczUsFjp5U4buKbAa9/t4=; b=vk09bsQq6wCKOyf8vVI9RrszZEeelASQTfUeJGj2SrtWnaedldmEY8r1U1vpa17Mx8DqMiK25t0LbdFY+mWI2XMR/oyG7dbl95IWOdsPV1SPnIq3JXH17A/zkmgt86o+LBXxLp4ag96JN14kMuleHWzz7LiP6Hfs2xHhv26sCws= DomainKey-Signature: a=rsa-sha1; c=nofws; d=googlemail.com; s=gamma; h=message-id:date:from:reply-to:to:subject:cc:in-reply-to:mime-version:content-type:content-transfer-encoding:content-disposition:references; b=kQov9CYjgoHy0bA7emwYAjhyx9qJuck7LyjsQtx7otIE6uPvmWUvW5WoEp69KsdPY1k6a3RSEHMEaBMHjLEPhUFr2wNUd7lZst+MYeRrjOjdukSvTdj7J+/8JAGbiBk4HXgk9A/T7i+YfOMFT+fzzR7vV8PyxtJ7heEnesiFnyw= Received: by 10.125.133.13 with SMTP id k13mr431236mkn.25.1210773921068; Wed, 14 May 2008 07:05:21 -0700 (PDT) Received: by 10.86.60.20 with HTTP; Wed, 14 May 2008 07:05:20 -0700 (PDT) Message-ID: <10845a340805140705s6acd7c71rd49d0485c7cc520c@mail.gmail.com> Date: Wed, 14 May 2008 15:05:20 +0100 Reply-To: RQuadling@GoogleMail.com To: "Gregory Beaver" Cc: "internals Mailing List" In-Reply-To: <482AED62.8050709@chiaraquartet.net> MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Content-Disposition: inline References: <4828BA83.7050900@chiaraquartet.net> <48296276.4070103@daylessday.org> <10845a340805140431u52942409p1d4896c96d8ddb66@mail.gmail.com> <482AED62.8050709@chiaraquartet.net> Subject: Re: [PHP-DEV] [HEADS UP] pecl/phar is now ext/phar From: rquadling@googlemail.com ("Richard Quadling") 2008/5/14 Gregory Beaver : > Richard Quadling wrote: >> 2008/5/13 Antony Dovgal : >> >>> On 13.05.2008 01:45, Gregory Beaver wrote: >>> >>>> Thanks to all who have contributed, particularly Marcus, Steph, Lars, >>>> and the others who chimed in with ideas on the list. >>>> >>> phar_detect_phar_fname_ext() fails if is_complete = 1 and filename contains >>> ".". >>> >>> For example: >>> Breakpoint 1, phar_detect_phar_fname_ext (filename=0x124db80 >>> "/local/qa/5_3.zts/ext/phar/tests/DataArchive.phar", check_length=1, >>> >> >> >> Am I being picky in saying that having a . in a folder containing a >> file should be allowed? And if there is an extension on the filename, >> then it will also have a . >> > This was always the intention, and is already fixed. In fact, opening > URLs like phar://whatever/has.dot/my.phar/internal/file.php already > worked, the only thing broken was instantiating a Phar object with a > path containing "." >> What happens for relative filenames? > > They have worked since phar 2.0.0a1. The only requirement for phar to > detect its file is that you either use a registered alias or that the > filename contain a ".". Executable phar archives must also contain > ".phar" as part of the filename extension. > > In short, the problem with Phar objects crept in only because none of us > had a path in our dev environment containing a "." and didn't think to > add a test case for it until the problem appeared. It's now fixed, and > all tests pass in CVS. > > Greg > Thank you. -- ----- Richard Quadling Zend Certified Engineer : http://zend.com/zce.php?c=ZEND002498&r=213474731 "Standing on the shoulders of some very clever giants!"