Community technical support mailing list was retired 2010 and replaced with a professional technical support team. For assistance please contact: Pre-sales Technical support via email to firstname.lastname@example.org.
Arthur Barrett wrote: > Chuck, > >> 1. Print out line and column number from the script if it fails. Very >> convenient when creating scripts. >> 2. Allow a centralized script location defined in the "script.name" >> file. With many repositories on a single box, making a change to a >> script that must live in every CVSROOT is an administration nightmare. >> 3. String was not null-terminated correctly; it was dereferencing char >> ** as if it was char * >> 4. Same as 3 >> 5. Putting 11 args into an array of only 9 elements >> >> 6. COM says that you give a copy of strings, not the address of your >> only copy >> 7. Same as 6 >> 8. Missing AddRef >> 9. Same as 8 > > I've made item 2 to require a setting the registry (and added it to the > control panel). > If what you're saying is that the script name is stored in the registry, I considered that approach but rejected it as less flexible than the approach used by all the other files (loginfo, commitinfo, etc.) By limiting to one per box rather than on a per-repository basis it made it hard to have groups of repositories, and harder to develop/test because you couldn't set one repository to point to the script you're working on. But it's certainly close enough to meet the administration requirement for us. We only have one script to date. > I've also modified the debug messages so the physical repository name is not > revealed unless it's a per-machine location (for security reasons we cant > ever display the phsycal repository location in a trace). > http://customer.march-hare.com/webtools/bugzilla/ttshow_bug.cgi?tt=1&id=5252 > That sounds good. I mostly needed it for debugging anyway. > Patch to 2.5.04: > http://customer.march-hare.com/webtools/bugzilla/attachment.cgi?tt=1&id=1221&action=view > > Committed change to 2.5.04 - it will be in the next RC. > > Regards, > > > Arthur > Thanks for the update. chuck