RE: Re: CvsAlert changes

"Oliver Giesen" <[email protected]>
Newsgroups gmane.comp.version-control.cvs.gui.devel
Message-ID <H000006700041e6c.1071061191.mail01.lucatec.de@MHS>
> Possibly. I only created the alert class itself for now so build can 
> be fixed and stabilized for all platforms. The next step is to 
> eliminate the code duplication/unneded wrappers which means 
> PromptMessage and AskYesNo (and all AfxMessageBox calls). I also need 
> to make the resizing to fit the message a little bit smarter...

That'd be great. :)

 
> > Didn't have time to do a build anymore though (did I 
> > mention already how *slow* C++ compiles?... ;) )
> 
> It's not *that* slow unless you make the full rebuild on a slow 
> machine ;)

It _is_ *that* slow if you've got Delphi as a comparison (a full build 
of our ~100.000 line project takes under a minute on a 600MHz P2 with 
192MB RAM and SETI still running in the background) ... ;] 
...OTOH, I _am_ currently compiling on a very old and slow system at 
home (P2 350MHz with 128MB RAM). It takes almost a minute to just do an 
incremental build for a single change in CvsCommands.cpp . However, just 
yesterday I finally got to the point of ordering a new box and I'm 
looking forward to being able to compare compile speeds on a 2.6GHz P4 
with 1GB RAM by the end of next week. :)

 
> Actually that "slowness" is very usefull - it gives you time to 
> *think* about what you've just done and what your next step will be. 
> Makes up for a better though-out implementation in the long run -
> e.g. you try to work "smarter" rather than "harder"... But I digress 
;)

I seem to hear that argument a lot from C++-coders... ;)


> My major concern is that the CVSNT version that was 
> shipped with latest WinCvs has that silly "cvs ls /" error so I might 
> simply release 1.3b14 build 2 to cover for that ;)

Might be a good idea.
BTW: I started playing around a bit with the install script, trying your 
idea of creating an unselected component for "obsolete macros". I was 
also thinking if it might be possible to create an alternative installer 
that excludes the CVSNT files, but instead offers to either specify 
their location (but then where do I put it - registry is useless as 
noone in their right minds should run the program under the same user 
account that it was installed with anyway) or alternatively downloads 
the latest installer on demand via ISTool's Download DLL (the obvious 
problem is determining the file name of the latest release - this would 
probably have to be coordinated with Tony). For people who regularly 
update CVSNT anyway this would significantly cut the download size.
If you want I could commit the macro stuff already.

 
> What exactly does the MarkResolved suppose to do?

You'd probably say it's just cosmetics... ;)
It recursively scans the selection for Conflict files that have been 
modified since the conflict occured and that do not contain any conflict 
markers. If it finds such files, it replaces the Timestamp+Conflict 
token of the corresponding Entries line with the text "Resolved", 
effectively changing the status displayed in WinCvs from Conflict to 
Modified, while still keeping them easily identifiable. It dumps a 
detailed report of scanned files and highlights files that still contain 
conflict markers. It also uses the Entries.Backup file when rewriting 
the Entries file for safety (although it doesn't quite use it by the 
specs which honestly don't make a lot of sense to me). Following Tony's 
suggestions, it will skip directories where an Entries.Backup still 
exists with an appropriate warning.


> I will let you know about the checkbox after I take a closer look at 
> the code - stay tuned.

I will. ;)

Cheers,

Oliver
----	------------------
JID:	[email protected]
ICQ:	18777742	(http://wwp.icq.com/18777742)


------------------------ Yahoo! Groups Sponsor ---------------------~-->
Buy Ink Cartridges or Refill Kits for your HP, Epson, Canon or Lexmark
Printer at MyInks.com. Free s/h on orders $50 or more to the US & Canada.
http://www.c1tracking.com/l.asp?cid=5511
http://us.click.yahoo.com/mOAaAA/3exGAA/qnsNAA/NhFolB/TM
---------------------------------------------------------------------~->

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/
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.