RE: [Tkdiff-discuss] tkdiff 4.0b1 available

[John K. <jo...@kl...>]

I applied your patch...  Everything's cool (at least under cygwin), =
except
that when you run tkdiff without a filename, it flashes and thrashes and
generally goes nuts.  If you supply a filename, all is well.  Any ideas?

Your version is 1.12 in CVS.

As for your other changes (and David's proposed changes), let's wait =
until
the dust settles on b2 and get 4.0 out the door...  After that, we can
figure out how best to add the feature you're after, in light of David's
work.  Sound okay with everybody?

Thanks, Dorothy!

John

--
John Klassa / jo...@kl...

Please use this address for all personal correspondence.  Thanks!

> -----Original Message-----
> From: John Klassa [mailto:kl...@qw...] On Behalf Of Dorothy =
Robinson
> Sent: Wednesday, May 14, 2003 2:33 AM
> To: tkd...@so...
> Subject: Re: [Tkdiff-discuss] tkdiff 4.0b1 available
>=20
> I found a mistake which caused filenames loaded from the Browse dialog =
-
> ie., on
> a PC - to not withstand spaces in the filenames.  I fixed it, along =
with
> the
> setcolor problem, and submitted a patch on Sourceforge.  (I'm sure the
> upload
> "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 a file =
that
> isn't
> checked out.  Right now it won't do that, because it doesn't know from
> rdiff and
> because it checks to see if the file given in the argument exists, =
which
> it
> won't in this case.  So I thought of adding a command line option for
> doing an
> rdiff, and removing the check for the existence of the file.  =
Something
> like
> "tkdiff -d repository -rdiff -r1.1 -r1.2 modulepath/filename"
> Objections?  Better ideas?  I can't help thinking there might be an =
easier
> way.
>=20
> Dorothy
>=20
> Daniel H=F6pfl wrote:
> > Hi!
> >
> > In line 153-158 you have:
> >
> > ------------------------------------------------------
> > 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 gray for =
the
> > # selected color (rh8 with tk 8.3.3-74) "#c3c3c3"
> > set w(selcolor) "#b03060"
> > ------------------------------------------------------
> >
> > I think the problem is the index 3 when setting w(selcolor):
> >
> > ------------------------------------------------------
> > set w(selcolor) [lindex [.testent configure -selectbackground] 4]
> > set entryfont [lindex [.testent configure -font] 3]
> > destroy .testent
> > ------------------------------------------------------
> >
> > ... sets the value of "*selectBackground" from your .Xdefaults (or
> > #bfdfff, a light blue, as default) into w(selcolor).
> >
> > An other solution would be:
> >
> > ------------------------------------------------------
> > set w(selcolor) [lindex [.testent configure -selectforeground] 4]
> > set entryfont [lindex [.testent configure -font] 3]
> > destroy .testent
> > ------------------------------------------------------
> >
> > In this case you get the value of "*selectForeground". Better than =
using
> > the background color for the foreground of a checkbox.
> >
> > Is there a way to get the value of "*selector" from the .Xdefaults? =
I
> > think this would be the "correct" value for this case.
> >
> > Greetings,
> >    Daniel
> >
>=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
>=20
>=20
> -------------------------------------------------------
> Enterprise Linux Forum Conference & Expo, June 4-6, 2003, Santa Clara
> The only event dedicated to issues related to Linux enterprise =
solutions
> www.enterpriselinuxforum.com
>=20
> _______________________________________________
> Tkdiff-discuss mailing list
> Tkd...@li...
> https://lists.sourceforge.net/lists/listinfo/tkdiff-discuss