Re: CVS Suite WinCVS patches

Arthur,

I will make more detailed review sometime next week, but just a few 
things to ask:
> * strings are stored in the registry as strings 

Sounds troublesome. People can not switch between our build and your 
build freely/

> (so the installer can preset them)

Can't the installer just run .reg file with preset values or 
something? It surely sounds like a (albeit small) can of worms for a 
little benefit to me...

> by default the CVSNT repository finder is used rather 
> than the WinCVS CVSROOT wizard

I already commented on that one earlier so I will just stress that I 
still hope the missing keywords will be re-introduced to "cvs info" 
output so that people can use CVSROOT wizard as intended. I will not 
back down on that, please take it seriously.

> * "finds" cvs using the registry entries used by the suite
> * various documentation and aesthetics

I am not seeing that in the patch - can you elaborate how it's done? 
If I install CVSSuite can I then install cvsgui-donwloaded WinCvs and 
the old registry will be still working? We are using 
"HKLM\\Software\\CVS\\PServer\\InstallPath" right now. If that is not 
set people will face problems.

Best Regards,
Jerzy

--- In [email protected], "Arthur 
Barrett" <arthur.barrett@...> wrote:
>
> 
> Just to be clear, the complete diffs for the changes between the 
"CVS
> Suite" version of WinCVS and the current HEAD of the repository is
> available from the link in the FAQ:
>  http://www.march-hare.com/cvspro/faq/faq1.asp#1k
> 
> I've just uploaded a new context diff that builds against the 
repository
> as of a few hours ago.  The main mods are:
> * strings are stored in the registry as strings (so the installer 
can
> preset them)
> * by default the CVSNT repository finder is used rather than the 
WinCVS
> CVSROOT wizard
> * built on VS.NET 2003 (and associated mods)
> * "finds" cvs using the registry entries used by the suite
> * various documentation and aesthetics
> 
> If the WinCVS/CVSGUI team want to include any/all of these patches 
then
> that'd be great (they are all ifdef'd so you can build with or with 
out
> them enabled), however the only feature that I think would be of
> "general" value is the Wizard selector (see separate email).
> 
> Regards,
> 
> 
> Arthur
>







------------------------ Yahoo! Groups Sponsor --------------------~--> 
Yahoo! Groups gets a make over. See the new email design.
http://us.click.yahoo.com/XISQkA/lOaOAA/yQLSAA/NhFolB/TM
--------------------------------------------------------------------~-> 

 
Yahoo! Groups Links

<*> To visit your group on the web, go to:
    http://groups.yahoo.com/group/cvsgui-dev/

<*> To unsubscribe from this group, send an email to:
    [email protected]

<*> Your use of Yahoo! Groups is subject to:
    http://docs.yahoo.com/info/terms/