Bug #125850 for Tcl-Tk: test-for-tk.tcl: `package require tklib` fails

Sun Jul 15 04:33:21 2018 CAC [...] cpan.org - Ticket created

Subject:

test-for-tk.tcl: `package require tklib` fails

The test for tklib in test-for-tk.tcl doesn't work, at least for Tcl-Tk 1.12 with Tcl/Tk 8.6.8, tcllib 1.19, and tklib 0.6 on both macOS 10.13.6 and Fedora 27.

$ tclsh
Show quoted text

% package require tklib

can't find package tklib
Show quoted text

%

Now, since /lib/tklib0.6/pkgIndex.tcl doesn't contain package ifneeded tklib 0.6 nor is there a tklib.tcl file, I'm not sure whether (1) this is even the way to check for tklib itself in the first place, (2) if this used to work with an older tklib, and/or (3) this is a bug in Tklib. I believe the same applies to what's currently in the tklib repository.

Trying to load e.g. a specific widget from tklib does work though, but I'm not sure that's the ideal workaround.

Sun Jul 15 04:46:10 2018 VKON [...] cpan.org - Correspondence added

Le Dim 15 Jui 2018 04:33:21, CAC a écrit : Show quoted text

> The test for tklib in test-for-tk.tcl doesn't work, at least for Tcl- > Tk 1.12 > with Tcl/Tk 8.6.8, tcllib 1.19, and tklib 0.6 on both macOS 10.13.6 > and Fedora > 27. > > $ tclsh > % package require tklib > can't find package tklib > % > > Now, since /lib/tklib0.6/pkgIndex.tcl doesn't contain package ifneeded > tklib > 0.6 nor is there a tklib.tcl file, I'm not sure whether (1) this is > even the > way to check for tklib itself in the first place, (2) if this used to > work with > an older tklib, and/or (3) this is a bug in Tklib. I believe the same > applies > to what's currently in the tklib repository. > > Trying to load e.g. a specific widget from tklib does work though, but > I'm not > sure that's the ideal workaround.

indeed that check was bogus need to check for eg snit::scrolledwindow

Sun Jul 15 04:46:10 2018 The RT System itself - Status changed from 'new' to 'open'

Wed Aug 15 06:22:04 2018 paultcochrane [...] gmail.com - Correspondence added

On Sun Jul 15 10:46:10 2018, VKON wrote: Show quoted text

> Le Dim 15 Jui 2018 04:33:21, CAC a écrit :

> > The test for tklib in test-for-tk.tcl doesn't work, at least for Tcl- > > Tk 1.12 > > with Tcl/Tk 8.6.8, tcllib 1.19, and tklib 0.6 on both macOS 10.13.6 > > and Fedora > > 27. > > > > $ tclsh > > % package require tklib > > can't find package tklib > > % > > > > Now, since /lib/tklib0.6/pkgIndex.tcl doesn't contain package ifneeded > > tklib > > 0.6 nor is there a tklib.tcl file, I'm not sure whether (1) this is > > even the > > way to check for tklib itself in the first place, (2) if this used to > > work with > > an older tklib, and/or (3) this is a bug in Tklib. I believe the same > > applies > > to what's currently in the tklib repository. > > > > Trying to load e.g. a specific widget from tklib does work though, but > > I'm not > > sure that's the ideal workaround.

> > indeed that check was bogus > > need to check for eg snit::scrolledwindow >

I believe this problem has been fixed in [PR#15](https://github.com/vadrer/perl-tcl-tk/pull/15) on GitHub and hence this ticket could be closed as resolved.

Wed Aug 15 07:03:00 2018 VKON [...] cpan.org - Correspondence added

thanks

Wed Aug 15 07:03:02 2018 VKON [...] cpan.org - Status changed from 'open' to 'resolved'

Fri Nov 09 02:24:57 2018 CAC [...] cpan.org - Correspondence added

On Sun Jul 15 03:33:21 2018, CAC wrote:
Show quoted text

> The test for tklib in test-for-tk.tcl doesn't work, at least for Tcl-
> Tk 1.12
> with Tcl/Tk 8.6.8, tcllib 1.19, and tklib 0.6 on both macOS 10.13.6
> and Fedora
> 27.
>
> $ tclsh

> % package require tklib

> can't find package tklib

> %

>
> Now, since /lib/tklib0.6/pkgIndex.tcl doesn't contain package ifneeded
> tklib
> 0.6 nor is there a tklib.tcl file, I'm not sure whether (1) this is
> even the
> way to check for tklib itself in the first place, (2) if this used to
> work with
> an older tklib, and/or (3) this is a bug in Tklib. I believe the same
> applies
> to what's currently in the tklib repository.
>
> Trying to load e.g. a specific widget from tklib does work though, but
> I'm not
> sure that's the ideal workaround.

It turns out I was the one who had proposed the broken tklib test just 3 days before opening this ticket: https://github.com/vadrer/perl-tcl-tk/commit/74e7e3997d5d11045f57f409fcfbe112c5a097ce . I apologize for forgetting I had done so (and as a result, sounding more confused than I should have been). Thanks to Paul for fixing this.