[Checkins] Re: SVN: lovely.memcached/trunk/buildout.cfg added explicit path to `python-memcached` to enable its installation.

Mon Jul 30 13:30:26 EDT 2007

Tom Lazar wrote:
> On 06.06.2007, at 15:46, Tom Lazar wrote:
> 
>> Log message for revision 76416:
>>   added explicit path to `python-memcached` to enable its installation.
>>
>>
>> Changed:
>>   U   lovely.memcached/trunk/buildout.cfg
>>
>> -=-
>> Modified: lovely.memcached/trunk/buildout.cfg
>> ===================================================================
>> --- lovely.memcached/trunk/buildout.cfg    2007-06-06 13:37:28 UTC (rev 76415)
>> +++ lovely.memcached/trunk/buildout.cfg    2007-06-06 13:46:21 UTC (rev 76416)
>> @@ -1,7 +1,10 @@
>>  [buildout]
>>  develop = .
>>  parts = test
>> +find-links = ftp://ftp.tummy.com/pub/python-memcached/python-memcached-1.34.tar.gz
>> +             http://download.zope.org/distribution/
>>
>> +
>>  [test]
>>  recipe = zc.recipe.testrunner
>>  eggs = lovely.memcached [test] 

Whatever it is you're tryinig to do, hard-wiring a path to a tarball of 
a *specific version* is always a bad idea. Not to mention that this path 
is already obsolete, the tarball has been moved to the old-releases subdir.

find-links is not supposed to contain paths to tarballs, it's suppose to 
provide setuptools with a hint of where to find more links to eggs which 
it can then evaluate. Don't try to outsmart it.

>> probably this should be fixed in the cheeseshop download url?
> 
> sorry, i'm not sure what you mean by this...

If easy_install python-memcached doesn't work, then its CheeseShop page 
(http://cheeseshop.python.org/pypi/memcached/) might not link to the 
right download URL. But as far as I can see, it does, making your patch 
even more obsucre.

>> did you file a bug upstream?
> 
> no, wouldn't know why.

Because at first it seems like you were trying to fix a problem in the 
package's metadata by pointing zc.buildout to the download location 
explicitly. A problem in a package's metadata is a bug, hence the 
suggested bug report.

Perhaps you could explain the rationale behind your patch?

-- 
http://worldcookery.com -- Professional Zope documentation and training