Modify

Opened 9 years ago

Closed 7 years ago

#4525 closed defect (fixed)

[patch] SvnServePasswordStore looks at wrong place for svnserve.conf file

Reported by: luke@… Owned by: Mikael Relbe
Priority: normal Component: AccountManagerPlugin
Severity: normal Keywords: SvnServePasswordStore
Cc: Ryan J Ollos Trac Release: 0.11

Description

By default the svnserve.conf file lives in /path/to/repos/conf/svnserve.conf. The SvnServePasswordStore looks in /path/to/repos/svnserve.conf

Attachments (1)

4525-svnserve.conf.path-7163.patch (837 bytes) - added by Mikael Relbe 8 years ago.
Fixes this ticket

Download all attachments as: .zip

Change History (7)

Changed 8 years ago by Mikael Relbe

Fixes this ticket

comment:1 Changed 8 years ago by Mikael Relbe

Summary: SvnServePasswordStore does not look at the correct svnserve.conf file[PATCH] SvnServePasswordStore does not look at the correct svnserve.conf file

Added attachment:4525-svnserve.conf.path-7163.patch

A no-brainer -- but this is my very first patch ever supplied to Trac/Trac-Hacks, and was therefore a suitable problem to practice "patching" on.

Please be gentle, guys.

comment:2 in reply to:  1 Changed 7 years ago by Steffen Hoffmann

Cc: Ryan J Ollos added; anonymous removed
Owner: changed from John Hampton to Steffen Hoffmann
Summary: [PATCH] SvnServePasswordStore does not look at the correct svnserve.conf file[patch] SvnServePasswordStore looks at wrong place for svnserve.conf file

Replying to mrelbe:

Added attachment:4525-svnserve.conf.path-7163.patch

A no-brainer -- but this is my very first patch ever supplied to Trac/Trac-Hacks, and was therefore a suitable problem to practice "patching" on.

Oh, never mind. In fact, even a small typo could turn a good product into really bad user experience, so finding and correcting all kind of bugs and glitches is highly appreciated. Receive a 1+ for a ready-made patch.

Please be gentle, guys.

No response for so may months is not nice, indeed. Anyway I'll not only say "sorry" but allow your patch to go into trunk right now. Would you be so kind as to test it yourself? Short after taking over plugin maintenance here I still suffer a real test environment for this special propose. Hereby I award you premium permission to close this ticket, if you're able to test patched code successfully.

You may take a mental note to checkout further changes to AccountManagerPlugin within the next few weeks again. My Mercurial Queue is filling up with patches right now, most of them related to fixes and improvements to authentication/login form, new user registration and more equally important stuff.

comment:3 Changed 7 years ago by Steffen Hoffmann

(In [9251]) AccountManagerPlugin: Correct path in SvnServePasswordStore to look for svnserve.conf, refs #4525.

Thanks to Mikael Relbe for report and suggested fix.

comment:4 Changed 7 years ago by Mikael Relbe

Hello -- this was not yesterday (no worries about time, I'm in the same boat :)

I'm currently not using SVN in this way, so I'll have to dig into my archives to verify this (and I'm about to switch computer and move from WinXP-32 to Win7-64).

Please give me-- umm --a month or so (not kidding).

comment:5 Changed 7 years ago by Steffen Hoffmann

Owner: changed from Steffen Hoffmann to Mikael Relbe

I've checked successfully against existing unittests. And I don't think, that some else is going on a faster lane right now to give us the feedback much earlier. So I'm ok with you taking as much time as you need.

Of course you may give "your" ticket back to me again anytime.

comment:6 Changed 7 years ago by Mikael Relbe

Resolution: fixed
Status: newclosed

I would probably forget this after a month, so I prioritized this...

It works!

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Mikael Relbe.
The resolution will be deleted.

Add Comment


E-mail address and name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.