[Bug 226480] Merge Review: tclx

[bugzilla at redhat.com]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=226480





--- Comment #19 from Patrice Dumas <pertusus at free.fr>  2008-09-28 10:16:40 EDT ---
Remaining issues:

Still no library installed in ld paths, so the following is unneeded:
%post -p /sbin/ldconfig

%postun -p /sbin/ldconfig


Among the man pages, CmdWrite, Handles and CmdWrite seems to be
part of tcl documentation and not tclx.

It seems to me that TclXInit.3 and Keylist.3 should be in the devel
package, since they are associated with the C interface.


Also TclXInit.3 is partly wrong because doing -ltclx won't work,
one needs to use -L /usr/lib/tcl8.5/tclx8.4/ -ltclx8.4. However
one could imagine that users wanting to use tclx like that
know that the object is in general dlopened and so that they have to find
the library in the tcl paths.


It may be possible to have in devel
%{_librir}/libtcl.so -> /usr/lib/tcl8.5/tclx8.4/libtclx8.4.so
and ship an /etc/ld.so.conf.d/ file to add the path to ldconfig.
However I think that it would be wrong since there is no soname.


There is also a rather innocuous rpmlint warning:
tclx.src: W: mixed-use-of-spaces-and-tabs (spaces: line 60, tab: line 74)


I'd still like to know more precisely why utf-8 locale needed 
to avoid truncating help files.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.