[bugs #11000] UNICOS 9.0 Cray Y-MP EL build problems

[James Youngman]

This mail is an automated notification from the bugs tracker
 of the project: findutils.

/**************************************************************************/
[bugs #11000] Latest Modifications:

Changes by: 
                James Youngman <address@hidden>
'Date: 
                Fri 11/19/04 at 23:22 (GMT)

            What     | Removed                   | Added
---------------------------------------------------------------------------
              Status | Open                      | Closed
       Fixed Release | None                      | 4.2.5


------------------ Additional Follow-up Comments ----------------------------
You can download a release of findutils in which this problem is
fixed from ftp://alpha.gnu.org/gnu/findutils.

The releases on alpha.gnu.org are for testing purposes, so please
take the time to download the release and verify that your
problem has been solved.  Once the release has been sufficiently
tested, it can be uploaded to ftp.gnu.org for everybody to use it.







/**************************************************************************/
[bugs #11000] Full Item Snapshot:

URL: <http://savannah.gnu.org/bugs/?func=detailitem&item_id=11000>
Project: findutils
Submitted by: Mark D Baushke
On: Sun 11/14/04 at 21:43

Category:  None
Severity:  5 - Average
Item Group:  None
Resolution:  Fixed
Privacy:  Public
Assigned to:  jay
Originator Name:  
Originator Email:  
Status:  Closed
Release:  4.2.3
Fixed Release:  4.2.5


Summary:  UNICOS 9.0 Cray Y-MP EL build problems

Original Submission:  Source: findutils-4.2.3
OS: UNICOS 9.0
Hardware: Cray Y-MP EL

Problem 1: If --enable-nls is given or defaulted, you should still see
if it is posible on the host to use it...

% ./configure
...
% make
...
cc -DHAVE_CONFIG_H -I. -I. -I../..     -g -c `test -f 'argmatch.c' 
|| echo './' `argmatch.c
cc-315 cc: ERROR argmatch.c, Line = 35, File = ./gettext.h, Line = 25
  The include file "libintl.h" is unavailable.
cc-515 cc: ERROR File = argmatch.c, Line = 138
  Both sides of the assignment operator are not compatible.
cc-269 cc: WARNING File = argmatch.c, Line = 157
  The type of argument #2 to function "fprintf" is not compatible with a 
previous declaration.

TOTAL WARNINGS DETECTED IN argmatch.c: 1
TOTAL ERRORS DETECTED IN argmatch.c: 2
make[4]: *** [argmatch.o] Error 1
%

Manually hacking the config.h to '#undef
ENABLE_NLS' lets the compilation get past the
argmatch error.

So, the lack of a <libintl.h> is a good indicator
that --disable-nls should be considered highly
desirable.

Problem 2: You need to do a refresh of the GNULIB
packages like mktime.c that did not previously
deal with Cray Y-MP EL right shift not propagating
the sign of negative numbers in mktime.c ... I
copied in the version used by ccvs to work around
the problem and continue the compile...

Problem 3: You need to add alloca.h to the -I
search path. Altering intl/Makefile to add
-I$(top_srcdir)/gnulib/lib like this:
  INCLUDES = -I.. -I. -I$(top_srcdir)/intl -I$(top_srcdir)/gnulib/lib
works around ths problem.

Problem 4: in find/pred.c, you have executable
statements before declarations which is allowed in
C99, but NOT allowed in C90:

| boolean
| pred_exec (char *pathname, struct stat *stat_buf, struct predicate *pred_ptr)
| {
|   (void) &pathname;
|   (void) &stat_buf;
|
|   int i;
|   int path_pos;
|   struct exec_val *execp;     /* Pointer for efficiency. */

should be rewritten as

| boolean
| pred_exec (char *pathname, struct stat *stat_buf, struct predicate *pred_ptr)
| {
|   int i;
|   int path_pos;
|   struct exec_val *execp;     /* Pointer for efficiency. */
|
|   (void) &pathname;
|   (void) &stat_buf;

Similarly in pred_iname, the '(void) stat_buf;'
needs to follow the 'const char *base;' line.

Problem 5: Adding -lsun to the link line when you
don't know if the library is available is a bad
idea.

Problem 6: basename is referenced in locate.c, but
the gnulib/lib/basename.o appears to provide the
base_name() function rather than the basename()
function. I modified the locate.c file to replace
occurances of basename() with base_name() to work
around this problem.

Here are a list of some of the warnings seen
during 'make'

...
cc -DHAVE_CONFIG_H -I. -I. -I../..     -g -c `test -f 'quotearg.c' 
|| echo './'`quotearg.c
cc-101 cc: WARNING File = quotearg.c, Line = 57
  The macro "iswprint" is redefined.

TOTAL WARNINGS DETECTED IN quotearg.c: 1
...
cc -DHAVE_CONFIG_H -I. -I. -I../..     -g -c `test -f 'mktime.c' 
|| echo './'`mktime.c
cc-445 cc: ERROR File = mktime.c, Line = 69
  The constant array dimension size is not a positive integral type.
cc-555 cc: ERROR File = mktime.c, Line = 69
  A member of a structure or union has an incomplete type.
cc-479 cc: ERROR File = mktime.c, Line = 69
  The "struct" or "union" "right_shift_propagates_sign" declaration has zero 
size.

TOTAL ERRORS DETECTED IN mktime.c: 3
make[4]: *** [mktime.o] Error 1
make[4]: Leaving directory `/home/mdb/findutils-4.2.3/gnulib/lib'
make[3]: *** [all] Error 2
...
cc -c -DLOCALEDIR="/usr/local/share/locale" 
 -DLOCALE_ALIAS_PATH="/usr/local/share/locale" 
 -DLIBDIR="/usr/local/lib" -DHAVE_CONFIG_H -I.. -I. -I../intl 
 -g  loadmsgcat.c
cc-315 cc: ERROR File = loadmsgcat.c, Line = 40
  The include file "alloca.h" is unavailable.

TOTAL ERRORS DETECTED IN loadmsgcat.c: 1
make[1]: *** [loadmsgcat.o] Error 1
make[1]: Leaving directory `/home/mdb/findutils-4.2.3/intl'
...
cc -DHAVE_CONFIG_H -I. -I. -I.. -I../gnulib/lib -I../lib -I../gnulib/lib 
 -I../intl -DLOCALEDIR="/usr/local/share/locale"    -g 
 -c `test -f 'pred.c' || echo './'`pred.c
cc-518 cc: ERROR File = pred.c, Line = 456
  At the beginning of a statement, "int" is not valid.
cc-518 cc: ERROR File = pred.c, Line = 458
  At the beginning of a statement, "struct" is not valid.
cc-478 cc: ERROR File = pred.c, Line = 463
  The identifier "path_pos" is not declared.
cc-478 cc: ERROR File = pred.c, Line = 463
  The identifier "execp" is not declared.
cc-478 cc: ERROR File = pred.c, Line = 467
  The identifier "i" is not declared.
cc-478 cc: ERROR File = pred.c, Line = 482
  The identifier "i" is not declared.
cc-518 cc: ERROR File = pred.c, Line = 861
  At the beginning of a statement, "const" is not valid.
cc-478 cc: ERROR File = pred.c, Line = 867
  The identifier "base" is not declared.

TOTAL ERRORS DETECTED IN pred.c: 8
make[3]: *** [pred.o] Error 1
make[3]: Leaving directory `/home/mdb/findutils-4.2.3/find'
make[2]: *** [all-recursive] Error 1
...
cc -DHAVE_CONFIG_H -I. -I. -I.. -I../lib -I../gnulib/lib 
 -I../gnulib/lib -I../intl 
 -DLOCATE_DB="/usr/local/var/locatedb" 
 -DLOCALEDIR="/usr/local/share/locale" 
 -g -c `test -f 'locate.c' || echo './'`locate.c
cc-269 cc: WARNING File = locate.c, Line = 438
  The type of argument #2 to function "gnu_fnmatch" is not compatible with a 
previous declaration.
cc-269 cc: WARNING File = locate.c, Line = 442
  The type of argument #2 to function "gnu_fnmatch" is not compatible with a 
previous declaration.

TOTAL WARNINGS DETECTED IN locate.c: 2
cc  -g   -o locate  locate.o ../find/version.o 
 ../lib/libfind.a ../gnulib/lib/libgnulib.a 
 ../intl/libintl.a  -lsun -lsun  -lcposix
 ldr-334 cc: CAUTION
     File 'libsun.a' cannot be found in any of the search directories.
 ldr-133 cc: WARNING
     Unsatisfied external references have been encountered.

 Unsatisfied external references
 Entry name      Modules referencing entry

 basename        locate$c

 ldr-112 cc: WARNING
     Because of previous errors, file 'locate' is not executable.
make[3]: *** [locate] Error 1
make[3]: Leaving directory `/home/mdb/findutils-4.2.3/locate'
...


Follow-up Comments
------------------


-------------------------------------------------------
Date: Fri 11/19/04 at 23:22         By: James Youngman <jay>
You can download a release of findutils in which this problem is
fixed from ftp://alpha.gnu.org/gnu/findutils.

The releases on alpha.gnu.org are for testing purposes, so please
take the time to download the release and verify that your
problem has been solved.  Once the release has been sufficiently
tested, it can be uploaded to ftp.gnu.org for everybody to use it.


-------------------------------------------------------
Date: Fri 11/19/04 at 22:30         By: James Youngman <jay>
All dependent bugs now fixed in CVS.  Hence this bug is fixed, I think.   bug 
#11008 is no longer a dependency because it doesn't prevent a build.

-------------------------------------------------------
Date: Mon 11/15/04 at 09:52         By: Mark D Baushke <mbaushke>
The CVS version of findutils builds (with some warnings).
Savannah bug #11008 has been opened to address the -lsun being
added (twice) to the LIBS= makefile macro.

-------------------------------------------------------
Date: Mon 11/15/04 at 01:30         By: James Youngman <jay>
Five of the six problems are now fixed in CVS, and the last one (prolem number 
5, incorrectly adding -lsun to LIBS) should, I think, not be a problem - 
indeed, it shouldn't happen.  Could you retest the CVS code?

-------------------------------------------------------
Date: Mon 11/15/04 at 00:53         By: James Youngman <jay>
Problem 1 - raised as bug #11001
Problem 2 - fixed in CVS.
Problem 3 - raised as bug #11002
Problem 4 - fixed in CVS.
Problem 5 - configure should only add -lsun to $LIBS if it is available.  If 
this is not happening as designed, please raise a separate bug for it (ideally 
with a bit of detail).
Problem 6 - raised as bug #11003








File Attachments
-------------------

-------------------------------------------------------
Date: Sun 11/14/04 at 21:43  Name: config.log  Size: 345.62KB   By: mbaushke
config.log output when run on UNICOS 9.0 Cray system
http://savannah.gnu.org/bugs/download.php?item_id=11000&amp;item_file_id=1869






For detailed info, follow this link:
<http://savannah.gnu.org/bugs/?func=detailitem&item_id=11000>

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