RE: [Tkdiff-discuss] tkdiff 4.0b1 available

[Brown, D. <DBr...@ha...>]

I use tkdiff quite a bit, with PVCS. I originally submitted the PVCS =
support for
tkdiff.
Anyway, I've been waiting for the dust to settle, ready to submit some =
stuff:
probably good for inclusion after the next release. I mention this now =
because
Dorothy's idea is similar yet different.

I have added -s [CVS | RCS | Perforce | AccuRev | PVCS | SCCS] for the =
cases
where tkdiff can't guess what source control system is being used. This =
might be
necessary or helpful when specifying the repository. =20

I also added -c <vcs.cfg>. For PVCS, the key to finding the repository =
is the
file "vcs.cfg". If I'm running tkdiff outside of the diretory where =
vcs.cfg
resides, I need to specify -s PVCS and also -c c:/path-to/vcs.cfg.

Perhaps the better approach would be:
-s CVS -d <path-to-repository>
-s PVCS -d <path-to-vcs.cfg>
etc.

My other local modification is the addition of PVCS support for =
promotion groups
and version labels, using the -g and -v prefixes.=20

My main gripe is that the -a ancestor option has to be an actual file. =
I've
tried to fix this, but the limited command-line syntax has gotten in my =
way. If
anyone has a suggestion please let me know. I use -a quite often.
--
Dave Brown <dbr...@ha...>

> -----Original Message-----
> From: Dorothy Robinson [mailto:mo...@ar...]
> Sent: Wednesday, May 14, 2003 2:33 AM
> To: tkd...@so...
> Subject: Re: [Tkdiff-discuss] tkdiff 4.0b1 available
>=20
>=20
> I found a mistake which caused filenames loaded from the=20
> Browse dialog - ie., on=20
> a PC - to not withstand spaces in the filenames.  I fixed it,=20
> along with the=20
> setcolor problem, and submitted a patch on Sourceforge.  (I'm=20
> sure the upload=20
> "took" this time.)
>=20
> On another subject - I want to allow tkdiff to do a cvs rdiff like
> "cvs -d repository rdiff -r1.1 -r1.2 modulepath/filename" on=20
> a file that isn't=20
> checked out.  Right now it won't do that, because it doesn't=20
> know from rdiff and=20
> because it checks to see if the file given in the argument=20
> exists, which it=20
> won't in this case.  So I thought of adding a command line=20
> option for doing an=20
> rdiff, and removing the check for the existence of the file. =20
> Something like
> "tkdiff -d repository -rdiff -r1.1 -r1.2 modulepath/filename"
> Objections?  Better ideas?  I can't help thinking there might=20
> be an easier way.
>=20
> Dorothy
>=20
> Daniel H=F6pfl wrote:
> > Hi!
> >=20
> > In line 153-158 you have:
> >=20
> > ------------------------------------------------------
> > set w(selcolor) [lindex [.testent configure -selectbackground] 3]
> > set entryfont [lindex [.testent configure -font] 3]
> > destroy .testent
> > # the above results in a nearly undistinguishable darker=20
> gray for the
> > # selected color (rh8 with tk 8.3.3-74) "#c3c3c3"
> > set w(selcolor) "#b03060"
> > ------------------------------------------------------
> >=20
> > I think the problem is the index 3 when setting w(selcolor):
> >=20
> > ------------------------------------------------------
> > set w(selcolor) [lindex [.testent configure -selectbackground] 4]
> > set entryfont [lindex [.testent configure -font] 3]
> > destroy .testent
> > ------------------------------------------------------
> >=20
> > ... sets the value of "*selectBackground" from your .Xdefaults (or=20
> > #bfdfff, a light blue, as default) into w(selcolor).
> >=20
> > An other solution would be:
> >=20
> > ------------------------------------------------------
> > set w(selcolor) [lindex [.testent configure -selectforeground] 4]
> > set entryfont [lindex [.testent configure -font] 3]
> > destroy .testent
> > ------------------------------------------------------
> >=20
> > In this case you get the value of "*selectForeground".=20
> Better than using=20
> > the background color for the foreground of a checkbox.
> >=20
> > Is there a way to get the value of "*selector" from the=20
> .Xdefaults? I=20
> > think this would be the "correct" value for this case.
> >=20
> > Greetings,
> >    Daniel
> >=20
>=20
> --=20
> =
-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D=
-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-
> Dorothy Robinson
> http://www.twobarleycorns.net
>=20