Tcl Source Code

View Ticket
Login
Ticket UUID: 1786647
Title: wrong variables in compat/memcmp.c
Type: Bug Version: obsolete: 8.4.15
Submitter: mladenk Created on: 2007-09-02 14:19:51
Subsystem: 52. Portability Support Assigned To: jenglish
Priority: 3 Low Severity:
Status: Open Last Modified: 2007-09-07 03:18:37
Resolution: None Closed By:
    Closed on:
Description:
Unable to compile memcmp.c
i looked in source and i think that used
variables are wrong.
User Comments: dkf added on 2007-09-07 03:18:37:
Logged In: YES 
user_id=79902
Originator: NO

Fixed the obvious fault in compat/memcmp.c; I won't try to think about the bugs relating to why that file exists or is used.

jenglish added on 2007-09-06 08:56:38:
Logged In: YES 
user_id=68433
Originator: NO

From comments in #1631017:

| The real bug is that compat/memcmp.c is being used in the first place -
| if the build system is trying to use this file at all, it indicates that
| the configure script has failed to detect a working C compiler.

Other reports indicate that this happens when trying to cross-compile.
One proposed solution is to use AC_TRY_LINK instead of AC_TRY_RUN.
(sorry, reference not handy, I just vaguely remember reading this somewhere).

msofer added on 2007-09-03 23:11:58:
Logged In: YES 
user_id=148712
Originator: NO

This is a duplicate of bug #1631017, which is reported as fixed in both HEAD and 8.4. The files on HEAD and core-8-4-branch differ: is there a reason for that?

Assigning to dkf as he reported this fixed.

[email protected] added on 2007-09-03 22:25:27:
Logged In: NO 

I have Fedora 7 with gcc-4.1.2-12 and was trying to compile LFS.
Something went wrong and i opened compat/memcmp.c to see what.
Can you just look at the code i think it is trivial.

Thanks Mladen

msofer added on 2007-09-02 21:54:48:
Logged In: YES 
user_id=148712
Originator: NO

Could you please explain what "unable to compile" means? What platform are you on?

msofer added on 2007-09-02 21:42:49:

data_type - 110894

mladenk added on 2007-09-02 21:19:51:

File Added - 243911: memcmp-tcl.patch

Attachments: