Opened 15 years ago
Closed 14 years ago
#4525 closed defect (fixed)
[patch] SvnServePasswordStore looks at wrong place for svnserve.conf file
| Reported by: | 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)
Change History (7)
Changed 14 years ago by
| Attachment: | 4525-svnserve.conf.path-7163.patch added |
|---|
comment:1 follow-up: 2 Changed 14 years ago by
| 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 Changed 14 years ago by
| 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 14 years ago by
(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 14 years ago by
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 14 years ago by
| 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 14 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | new → closed |
I would probably forget this after a month, so I prioritized this...
It works!
Fixes this ticket