Re: bug in setuptools

"Sergey Schetinin" <[email protected]>
Newsgroups gmane.comp.python.peak
Message-ID <[email protected]>
I understand where the error is coming from, but the more important
part thing here is that Subversion 1.5.0 checkouts aren't supported.
The .svn/entries format hasn't changed as far as I can tell, but the
version marker in the first byte did, so that's why I was suggesting
the change from a previous email.

I mean the fix for the lacking 'log' object is necessary but not enough.

On Sat, Jun 28, 2008 at 03:02, Pavel Repin <[email protected]> wrote:
> When 'entries_finder' gets confused about the svn working copy .svn
> entries, it tries to log an error by doing: log.warn(....). But
> there's no 'log' object. See a simple hack that fixes it (below).
>
> Does 'entries_finder' simply try to find objects that are under
> version control minus the ones that have pending delete status? If so,
> maybe invoking 'svn status' in a child process and parsing its output
> would be more robust?
>
> Bugfix:
> --- sdist-orig.py       2008-06-25 15:41:39.724048000 -0700
> +++ sdist.py    2008-06-25 15:41:18.249444800 -0700
> @@ -95,7 +95,7 @@
>         for match in entries_pattern.finditer(data):
>             yield joinpath(dirname,unescape(match.group(1)))
>     else:
> -        log.warn("unrecognized .svn/entries format in %s", dirname)
> +        print >> sys.stderr, "unrecognized .svn/entries format in
> '%s'" % dirname
>
>
>  finders = [
>
>>
>>
>> setuptools\command\sdist.py", line98, in entries_finder
>> NameError: global name 'log' is not defined
>>
>> Also, the execution started getting to that line after I upgraded
>> subversion to 1.5.0, so adding support for that would be very welcome
>> as well.
>>
>> Changing line 89 to:
>>     if data[:1] in '89':    # subversion 1.4 or 1.5
>> seems to work.
>>
>> A sample .svn/entries  file from a 1.5 working copy is attached, just in case.
>
> _______________________________________________
> PEAK mailing list
> [email protected]
> http://www.eby-sarna.com/mailman/listinfo/peak
>



-- 
Best Regards,
Sergey Schetinin

http://s3bk.com/ -- S3 Backup
http://word-to-html.com/ -- Word to HTML Converter
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.