[bug #19656] strcmpi does not exist on my system, better use strcasecmp instead

[Paul D. Smith]

Update of bug #19656 (project make):

  Status:None => Fixed  
 Assigned to:None => psmith 
 Open/Closed:Open => Closed 
   Fixed Release:None => CVS

___

Follow-up Comment #4:

Fixed this by converting all uses of case-insensitive string comparisons to
use POSIX strcasecmp(), and either aliasing strcasecmp to OS-specific
versions like strcmpi/stricmp, or else using our own version (for things like
VMS).

___

Reply to this item at:

  

___
  Message sent via/by Savannah
  http://savannah.gnu.org/



___
Bug-make mailing list
Bug-make@gnu.org
http://lists.gnu.org/mailman/listinfo/bug-make

[bug #19656] strcmpi does not exist on my system, better use strcasecmp instead

[Paul D. Smith]

Follow-up Comment #3, bug #19656 (project make):

True; I suppose it was never anticipated that anyone would build with
--enable-case-insensitive-file-system on a POSIX system (because POSIX
systems don't HAVE case-insensitive file systems).

Nevertheless, the flag is generic and not tied to Windows, etc. so this
should be fixed.

That code (hash.{c,h}) was actually taken from the GNU id-utils source so we
should check and see if there have been enhancements.

___

Reply to this item at:

  

___
  Message sent via/by Savannah
  http://savannah.gnu.org/



___
Bug-make mailing list
Bug-make@gnu.org
http://lists.gnu.org/mailman/listinfo/bug-make

[bug #19656] strcmpi does not exist on my system, better use strcasecmp instead

[Colin Finck]

Follow-up Comment #2, bug #19656 (project make):

Yes, you're right, the problem does not occur if I only run the "configure"
script without any parameters.

But when I add the parameter "--enable-case-insensitive-file-system", it does
not build.
Here is the error message:


gcc -DLOCALEDIR=\"/usr/local/share/locale\" -DLIBDIR=\"/usr/local/lib\"
-DINCLUDEDIR=\"/usr/local/include\" -DHAVE_CONFIG_H -I. -Wall -Wextra
-Wdeclaration-after-statement -Wshadow -Wpointer-arith -Wbad-function-cast -g
-O2 -MT hash.o -MD -MP -MF .deps/hash.Tpo -c -o hash.o hash.c
mv -f .deps/hash.Tpo .deps/hash.Po
gcc -Wall -Wextra -Wdeclaration-after-statement -Wshadow -Wpointer-arith
-Wbad-function-cast -g -O2   -o make ar.o arscan.o commands.o default.o dir.o
expand.o file.o function.o getopt.o getopt1.o implicit.o job.o main.o misc.o
read.o remake.o remote-stub.o rule.o signame.o strcache.o variable.o
version.o vpath.o hash.o   -lrt 
dir.o: In function `dir_contents_file_exists_p':
/home/colin/Desktop/Downloads/make-cvs/make/dir.c:712: undefined reference to
`strcmpi'
dir.o: In function `dirfile_hash_cmp':
/home/colin/Desktop/Downloads/make-cvs/make/dir.c:404: undefined reference to
`strcmpi'
dir.o: In function `directory_hash_cmp':
/home/colin/Desktop/Downloads/make-cvs/make/dir.c:361: undefined reference to
`strcmpi'
file.o: In function `file_hash_cmp':
/home/colin/Desktop/Downloads/make-cvs/make/file.c:58: undefined reference to
`strcmpi'
strcache.o: In function `str_hash_cmp':
/home/colin/Desktop/Downloads/make-cvs/make/strcache.c:118: undefined
reference to `strcmpi'


___

Reply to this item at:

  

___
  Nachricht geschickt von/durch Savannah
  http://savannah.gnu.org/



___
Bug-make mailing list
Bug-make@gnu.org
http://lists.gnu.org/mailman/listinfo/bug-make

[bug #19656] strcmpi does not exist on my system, better use strcasecmp instead

[Paul D. Smith]

Follow-up Comment #1, bug #19656 (project make):

Can you please provide the error messages you received?  As far as I know GNU
make sources don't use strcmpi() except when compiling for non-POSIX systems.

___

Reply to this item at:

  

___
  Message sent via/by Savannah
  http://savannah.gnu.org/



___
Bug-make mailing list
Bug-make@gnu.org
http://lists.gnu.org/mailman/listinfo/bug-make

[bug #19656] strcmpi does not exist on my system, better use strcasecmp instead

[Colin Finck]

URL:
  

 Summary: strcmpi does not exist on my system, better use
strcasecmp instead
 Project: make
Submitted by: colinfinck
Submitted on: Samstag 21.04.2007 um 17:17
Severity: 3 - Normal
  Item Group: Build/Install
  Status: None
 Privacy: Public
 Assigned to: None
 Open/Closed: Open
 Discussion Lock: Any
   Component Version: CVS
Operating System: POSIX-Based
   Fixed Release: None

___

Details:

If I try to compile the latest CVS revision of "make", the compiler complains
about a missing strcmpi function.
Now I added a macro, which replaces all strcmpi's with strcasecmp. Then
"make" compiles correctly.

I attached a patch file, which contains the macro I use.

My Operating System is Ubuntu 7.04.



___

File Attachments:


---
Date: Samstag 21.04.2007 um 17:17  Name: make-strcmpi-strcasecmp.patch  Size:
523B   By: colinfinck



___

Reply to this item at:

  

___
  Nachricht geschickt von/durch Savannah
  http://savannah.gnu.org/



___
Bug-make mailing list
Bug-make@gnu.org
http://lists.gnu.org/mailman/listinfo/bug-make