[Patches] [ python-Patches-1774414 ] Make it possible to use SVK to develop Python

SourceForge.net noreply at sourceforge.net
Wed Aug 22 21:48:34 CEST 2007


Patches item #1774414, was opened at 2007-08-14 21:00
Message generated for change (Comment added) made by collinwinter
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=305470&aid=1774414&group_id=5470

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: Core (C code)
Group: Python 3000
>Status: Closed
Resolution: Accepted
Priority: 5
Private: No
Submitted By: Collin Winter (collinwinter)
Assigned to: Collin Winter (collinwinter)
Summary: Make it possible to use SVK to develop Python

Initial Comment:
I want to use SVK instead of SVN when developing Python on my laptop. Currently, the parsing of HEADURL in Python/sysmodule.c:svnversion_init() kills the interpreter at start-up because the HEADURL for SVK looks like /py3k/local/Python/sysmodule.c.

The attached patch causes svnversion_init() to use "unknown" for the branch name instead of raising a fatal error when it encounters a mis-formatted HEADURL.

----------------------------------------------------------------------

>Comment By: Collin Winter (collinwinter)
Date: 2007-08-22 15:48

Message:
Logged In: YES 
user_id=1344176
Originator: YES

Committed as r57283.

----------------------------------------------------------------------

Comment By: Neal Norwitz (nnorwitz)
Date: 2007-08-16 01:57

Message:
Logged In: YES 
user_id=33168
Originator: NO

This is fine.  Note the indentation is screwed up.  It looks like some
mixing of tabs and spaces.  It would be good to add a comment near the
copying of "unknown" to mention about supporting svk and possibly other SCM
systems.

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=305470&aid=1774414&group_id=5470


More information about the Patches mailing list