Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

dlltkanim.so wrongly linked #4700

Closed
vicuna opened this issue Jan 20, 2009 · 5 comments
Closed

dlltkanim.so wrongly linked #4700

vicuna opened this issue Jan 20, 2009 · 5 comments
Assignees

Comments

@vicuna
Copy link

vicuna commented Jan 20, 2009

Original bug ID: 4700
Reporter: @glondu
Assigned to: @garrigue
Status: closed (set by @garrigue on 2010-04-03T11:43:05Z)
Resolution: suspended
Priority: normal
Severity: major
Version: 3.11.0
Fixed in version: 3.12.0+dev
Category: ~DO NOT USE (was: OCaml general)
Related to: #5011

Bug description

The stublib dlltkanim.so is wrongly linked:

$ ocaml -I +labltk tkanim.cma
Cannot load required shared library dlltkanim.
Reason: /usr/lib/ocaml/3.11.0/stublibs/dlltkanim.so: /usr/lib/ocaml/3.11.0/stublibs/dlltkanim.so: undefined symbol: cltclinterp.

The problem has been observed on OCaml 3.10.2 and 3.11.0.

@vicuna
Copy link
Author

vicuna commented Mar 31, 2009

Comment author: @xavierleroy

I think you're supposed to load labltk.cma before tkanim.cma.

But even so, another problem appears:

Cannot load required shared library dlltkanim.
Reason: ./dlltkanim.so: ./dlltkanim.so: undefined symbol: Tk_ImageObjCmd.

There is an explanation in otherlibs/tabltk/tkanim/Makefile:

tkAnimGIF.c used the function Tk_ImageObjCmd, which is not available

in a plain Tk installation. Should we disable this subdirectory ?

Should we, indeed?

@vicuna
Copy link
Author

vicuna commented Apr 2, 2010

Comment author: furuse

Let's remove the tkanim directory.
Since from tk8.4 (or somewhere), animated gifs can be loaded by pure tcl/tk.
labltk does not handle animated gifs unless it is extended somehow, but anyway, the current code does not work.

@vicuna
Copy link
Author

vicuna commented Apr 2, 2010

Comment author: @glondu

To xleroy: indeed. I've probably over-simplified my original bugreport to what I thought was the actual issue.

I was not expecting this error message for this buggy situation... When I usually get such a message, it is indeed a stublib that was linked with a missing -lfoo. But here, cltclinterp is defined in the C stubs of labltk.cma (dlllabltk.so), so I guess it's fine (even though some might argue that dlltkanim.so should be linked against dlllabltk.so...)

@vicuna
Copy link
Author

vicuna commented Apr 3, 2010

Comment author: furuse

tkanim is removed from the source code in the trunk,
due to its old and unmaintained status.
If someone really wants to have animated gif in labltk, he/she should try
more pure tcl way, using image photo command.

Please close the issue.

@vicuna
Copy link
Author

vicuna commented Apr 3, 2010

Comment author: @garrigue

tkanim removed by Jun Furuse on 2010-04-02.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants