Hi Sebastian,

Sebastian Harl wrote:

> When initializing the RRDtool Tcl bindings, the following is done (see
> bindings/tcl/tclrrd.c:init() in the RRDtool sources):
> 
>   Tcl_InitStubs(interp, TCL_VERSION, 0)
> 
> and then
> 
>   Tcl_PkgRequire(interp, "Tcl", TCL_VERSION, 1)

I am not so familiar with tcl ABI, unfortunately; sorry.  But this is
almost certainly wrong.  If we are going to require a specific tcl
version, why use stubs at all?

If you use

  Tcl_InitStubs(interp, TCL_VERSION, 0)
  Tcl_PkgRequire(interp, "Tcl", TCL_VERSION, 0)

or just

  Tcl_InitStubs(interp, TCL_VERSION, 0)

(the PkgRequire is implicit) then it will check a dependency of the form
tcl (>= 8.5.whatever).  tcl 8.5.8 and tcl 8.6 will be allowed as they
should be.

Since your package is known to work with tcl 8.4 already, I’d use

  Tcl_InitStubs(interp, "8.4", 0)

Hope that helps,
Jonathan



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org

Reply via email to