[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: Two problems w/ subversion

From: Karl Fogel <kfogel_at_collab.net>
Date: 2001-07-11 03:04:08 CEST

Thanks for the patch, Kapil! Mind if I request you repost with a
couple of minor frobs?:

   1. Run diff -u (or diff -c if you prefer), so the patch is more

   2. There's no need to leave the old code commented out -- just
      remove it entirely. We'll have the diff context right there if
      we want to see the old code, anyway.

   3. Write a log message; even a brief one helps a reader get an
      overview of the patch.

Thanks, hope this isn't too much of an imposition on your time...


kapil thangavelu <kthangavelu@earthlink.net> writes:
> On Tuesday 10 July 2001 15:14, Karl Fogel wrote:
> >>Thanks for the summary, Trent.
> >>
> >>Some history here:
> >>
> >>Subversion does require Python 2.0 for a reason: there's some code in
> >>the test suite that uses a feature that wasn't available in 1.5.2. I
> >>frankly don't remember exactly what the issue was, I just remember
> >>that Mike Sussman or Ben discovered that something didn't work with
> >>pre-2.0 Pythons, did a little research, and found out that we'd made
> >>use of a feature new in 2.0. In other words, 2.0 turned out to be
> >>backward-compatible but not forward-compatible with 1.5.2... as is
> >>usually the case with languages, I suppose.
> one more thing that python 2 adds is list comprehensions and some other
> syntatic sugar for function calling, which apparently is the problem in the
> test suites. it only affects one file actions.py and it looks easy enough to
> fix.
> i'm attaching the diff of the changes to make it work with python 1.5.2
> diff actions.py oldactions.py > actions.diff
> cheers
> kapil

To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Sat Oct 21 14:36:33 2006

This is an archived mail posted to the Subversion Dev mailing list.