Roundup Tracker - Issues

Issue 1476792

classification
subversion integration
Type: Severity: normal
Components: None Versions:
process
Status: closed fixed
:
: : richard
Priority: normal :

Created on 2006-04-26 10:35 by anonymous, last changed 2006-04-28 02:56 by richard.

Messages
msg2242 Author: [hidden] (anonymous) Date: 2006-04-26 10:35
on my first attempt, back in 2005, to get the 
subversion integration to work, I ran into problems. 
With help from Hernán Martínez Foffani on the mailing 
list they could be solved.
Richard Jones then suggested to file a bug, but that 
got lost as far as I can tell.

Last week I tried again and, abiding to the 
installation and configuration instructions, I ran into 
the same problems.

So here is what Hernán reported:

> The error you mentioned is due to the fact that
> you may have missed to add an item to the class
> svn_repo. You need one item per repository where
> one of them must have "C:/svn/repos/test" in the
> path field (it has to be exactly the same thing
> that SVN pass to th script.)

mentioning this in the installation description would 
help a lot.


> Additionally, there's a minor bug in
> notify-roundup.py (the one that has the link in
> the wiki) You may fix it yourself.  Swap the
> line 160 db = tracker.open(username)
> by moving it just after
>    # tell Roundup
>    tracker = roundup.instance.open(tracker_home)

I'm not realy sure about the scope of this change, but 
at least it did no harm in my setup.

Lorenz
msg2243 Author: [hidden] (anonymous) Date: 2006-04-26 10:36
Logged In: NO 

hmpf, forgot my email adress

lorenznlATyahooDOTcom
msg2244 Author: [hidden] (anonymous) Date: 2006-04-26 10:40
Logged In: NO 

hmpf, forgot my email adress

lorenznlATyahooDOTcom
msg2245 Author: [hidden] (anonymous) Date: 2006-04-26 10:41
Logged In: NO 

fotgot my email adress: lorenznlATyahooDOTcom
msg2246 Author: [hidden] (richard) Date: 2006-04-27 04:16
Logged In: YES 
user_id=6405

Dupe of patch 1468772
msg2247 Author: [hidden] (anonymous) Date: 2006-04-27 07:35
Logged In: NO 

patch 1468772 does not seem to address the two points I 
reported. As far as I can see, it only fixes a formating 
issue (that Hernán also mentioned in his post), but not the 
switched lines in notify-roundup.py nor the configuration 
hint I'm wishing for.

Is this line switch not neccessary, or is it addressed 
elsewhere? In this case your "dupe of ..." comment lead to a 
 missunderstanding on my side.

Well, reading the description of the internals of the 
subversion integration might have given the neccessary hint 
about the svn_repo class, but as I'm not familiar with 
python, I did not pay attention to that part of the 
documentation.
I would propose to mention that point in the "provided 
files" section in the explanation to svn-tracker-X.Y.zip, so 
that we poor nonPythons too don't miss it 8-)


Lorenz

msg2248 Author: [hidden] (richard) Date: 2006-04-28 02:56
Logged In: YES 
user_id=6405

Sorry, was referring to the wrong patch.

I've included a note about svn_repo.
History
Date User Action Args
2006-04-26 10:35:24anonymouscreate