Re: Some Macros topic.

"Oliver Giesen" <[email protected]>
Newsgroups gmane.comp.version-control.cvs.gui.devel
Message-ID <005b01c3febf$d9d21ba0$fe78a8c0@g01>
> (1)cvs2cl.py
>    (1-1)Line 185:
>         - print 'CVS returned Exit Code %d. Aborting ChangeLog
> creation.'
>         + print 'CVS returned Exit Code %d. Aborting ChangeLog
> creation.' % code

Oops, yes, of course. I'll correct that ASAP.


>    (1-2)BuildModuleLog does really work?

I use it frequently, yes.


>         I think the code does treat by mistake the relationship
>         between module and basedir.

Well, it's just a proposal. You could edit it before executing the macro.
Often the directory name corresponds directly to a module name. However, in
that case you could just as well use the "regular" ChangeLog macro instead.
The point of the Build ChangeLog by module is to be able to build logs for
"virtual" modules that are not directly deductible from anything that's in
the sandbox. For me the directory name is still often a good starting point
however.


>         Like this? :
<.snip>
> +        tmp = module
> +        while (not tmp == ''):
> +          tmp,b = os.path.split(tmp)
> +          module = '../' + module
<.snip.>
>         I have tested with slightly old CVSNT(2.0.14).

Hmm, it'd be news to me that you could give a relative module path. Relative
to what would that be?
I'll have a look at it tonight, though. No more time right now... we're in
the process of preparing to move out.


> (2)discrepancy between the code and the error message on
> AddRootModule.py:
>       if os.path.exists(os.path.join(basedir, 'CVS')):
>         console << kRed << 'Parent directory may not be CVS-
> controlled!\n' << kNormal
>         return
>    'Parent directory may be already CVS-controlled!'? or lack of
> `not'?

No, it's correct. For the macro to work, the parent directory may NOT
already be CVS-controlled, as the macro will temporarily create a ./CVS
directory within it (by non-recursively checking out the repository root).
If there already was a ./CVS directory the information in there would
override the settings entered by the user and the macro would not work as
expected.


> (3)It would be nice if 'Admin/Admin Macros' had 'Reload Macro', I
> think.

Agreed. Note though that even the Macros/Reload Macro item has some glitches
still. Repeat it a few times and suddenly all macros are greyed out... I
think there's a tracker item for this already. No time to look it up right
now...


> (4)I want `flexibility cvs add' Macro like RecursiveAddTk.py but no-
> recurse.
>    # I actually need `cvs add -ko'.

That's one for the issue tracker. What we really need is a full-fledged Add
dialog which exposes the full functionality of the commandline client.
There's many more options supported by recent CVSNT versions, among others
the improved successor to the infamous "Check-out with Unix LF" (-kL) and an
option to enable binary deltas in the repository (-kB). Furthermore all
these options could be combined freely (e.g. -kLo or -kuzv, etc.). Just take
a look at the output of cvs -H admin for the availabe -k options.


> P.S.
> w.r.t TCL on cygwin problem:
>   * I think at least we should not load any DLL if the settings
>     (WinCvs -> TCL DLL) is wrong.

Most of the time it works best if the settings are just kept blank. However,
it might indeed be more transparent if the checkbox was initially checked by
default and the entry simply read "tcl84.dll" (or whichever was the one that
succeeded to load the first time from the PATH) without full path.
Unchecking would result in WinCvs not even trying to load it. The same for
the Python DLL.


>   * It would be nice if WinCvs had the setting for process local
>     PATH environment variable.

I think the above would take care of that, or did I maybe misunderstand you?

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
---------------------------------------------------------------------~->

 
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/
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.