bugWeeChat - Bugs: bug #30907, tclConfig.sh search in...

 
 

bug #30907: tclConfig.sh search in configure.in not optimal

Submitted by:  C de-Avillez <hggdh>
Submitted on:  Mon 30 Aug 2010 03:40:42 PM UTC  
 
Category: compilationSeverity: 3 - Normal
Item Group: compile/installStatus: Fixed
Privacy: PublicAssigned to: Sebastien Helleu <flashcode>
Originator Name: Open/Closed: Closed
Release: 0.3.4IRC nick: hggdh

Add a New Comment(Rich Markup)
   

You are not logged in

Please log in, so followups can be emailed to you.

 

Sun 05 Sep 2010 07:04:58 AM UTC, comment #1:

Yes I know this problem, I'll fix this problem soon (of course before next release).
By the way, there is same problem with cmake build (which is recommended over autotools/configure).

Sebastien Helleu <flashcode>
Project AdministratorIn charge of this item.
Mon 30 Aug 2010 03:40:42 PM UTC, original submission:

After a git pull today, a build of weechat failed on src/plugins/scripts/tcl with on weechat-tcl.c.

Root cause was the dependency update from tcl8.4 to tcl8.5. Now, configure.in sets the path to search for tclConfig.sh as:

cl_dirs="/lib /usr/lib /usr/tcl/lib /usr/lib/tcl8.5 /usr/lib/tcl8.4 /usr/lib/tcl8.3 /usr/local/lib /usr/local/tcl-8.5/lib /usr/local/tcl-8.4/lib /usr/local/tcl-8.3/lib /usr/local/tcl/lib /opt/lib"

At least for Debian/Ubuntu (and derivatives) this is not ideal: Debian (and Ubuntu) still consider tcl8.4 as the default TCL to be used (up to unstable/Maverick, only Debian experimental carries tcl8.5 as the default); as such, the default 'tcl-dev' package sets a soft link on /usr/lib/tclConfig.sh pointing to the tcl8.4's tclConfig.sh.

Since /usr/lib is before /usr/lib/tcl8.5 in the search path, the incorrect tclConfig.sh is selected and used -- causing a compile failure due to changes in API/ABI.

Additionally, there is no need for the 8.4 and 8.3 TCL versions in the path anymore (since they are API/ABI incompatible).

C de-Avillez <hggdh>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach File(s):
   
   
Comment:
   

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by flashcode (Posted a comment)
  • -unavailable- added by hggdh (Submitted the item)
  •  

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    Follow 5 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Sun 16 Jan 2011 10:10:28 AM UTCflashcodeOpen/ClosedOpen=>Closed
      Release2.1-dev=>0.3.4
    Tue 09 Nov 2010 08:54:32 PM UTCflashcodeStatusConfirmed=>Fixed
    Sun 05 Sep 2010 07:04:58 AM UTCflashcodeStatusNone=>Confirmed
      Assigned toNone=>flashcode

    Back to the top


    Powered by Savane 3.1-cleanup1