You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@apr.apache.org by Max Bowsher <ma...@ukf.net> on 2004/07/22 00:56:42 UTC

Re: cvs2svn commit: r1267 - in trunk: . packages/rpm/redhat www

Tobias Ringström wrote:
> kfogel@tigris.org wrote:
>
>> Author: kfogel
>> Date: Wed Jul 21 16:47:40 2004
>> New Revision: 1267
>>
>> Added:
>>   trunk/cvs2svn
>>      - copied, changed from r1266, trunk/cvs2svn.py
>>   trunk/verify-cvs2svn
>>      - copied, changed from r1266, trunk/verify-cvs2svn.py
>> Removed:
>>   trunk/cvs2svn.py
>>   trunk/verify-cvs2svn.py
>>
>>
> This change broke profile-cvs2svn.py because it uses "import cvs2svn".
> Perhaps profile-cvs2svn.py could copy cvs2svn to cvs2svn.py before
> importing it?

Yuck.

I propose that we revert to *.py in the repository and in source tarballs.
My opinion is that it is the install process's job to do "cvs2svn.py" ->
"cvs2svn".

Vote, please.

Max.


Re: cvs2svn commit: r1267 - in trunk: . packages/rpm/redhat www

Posted by "Brian W. Fitzpatrick" <fi...@red-bean.com>.
On Jul 21, 2004, at 5:56 PM, Max Bowsher wrote:

> Tobias Ringström wrote:
>> kfogel@tigris.org wrote:
>>
>>> Author: kfogel
>>> Date: Wed Jul 21 16:47:40 2004
>>> New Revision: 1267
>>>
>>> Added:
>>>   trunk/cvs2svn
>>>      - copied, changed from r1266, trunk/cvs2svn.py
>>>   trunk/verify-cvs2svn
>>>      - copied, changed from r1266, trunk/verify-cvs2svn.py
>>> Removed:
>>>   trunk/cvs2svn.py
>>>   trunk/verify-cvs2svn.py
>>>
>>>
>> This change broke profile-cvs2svn.py because it uses "import cvs2svn".
>> Perhaps profile-cvs2svn.py could copy cvs2svn to cvs2svn.py before
>> importing it?
>
> Yuck.
>
> I propose that we revert to *.py in the repository and in source 
> tarballs.
> My opinion is that it is the install process's job to do "cvs2svn.py" 
> ->
> "cvs2svn".
>
> Vote, please.

-1.  I don't want to have to worry about the documentation and a 
million other things referring to cvs2svn here and cvs2svn.py there 
etc.  How important is it that we be able to import cvs2svn into other 
scripts?  If it's that important, then maybe it's time to break the 
bits that we need to import into some library modules that get 
installed in the site directory (although I'm not terribly crazy about 
that either).

-Fitz