Ticket #262 (closed bug: wontfix)

Opened 6 years ago

Last modified 6 years ago

$linkflags missing for libnci_test.so and dynpmc (solaris 64bit)

Reported by: rurban Owned by: rurban
Priority: normal Milestone:
Component: configure Version:
Severity: high Keywords:
Cc: Language:
Patch status: applied Platform: solaris

Description

Solaris 10 with sunpro cc has a 32 bit perl. Nevertheless I can create a fine 64 bit miniparrot and utils, just some targets miss the required and given linkflags.

perl Configure.pl --ccflags='-m64 -D_LARGEFILE_SOURCE -D_FILE_OFFSET_BITS=64 -D_TS_ERRNO' --linkflags='-m64'
make
...
cc -G   \
    -o runtime/parrot/dynext/libnci_test.so src/nci_test.o -lsocket -lnsl -ldl -lm -lpthread -lrt -lintl
ld: fatal: file src/nci_test.o: wrong ELF class: ELFCLASS64

"-m64 -xlibmieee" is missing

also

gmake[1]: Entering directory `/usr/share/src/perl/parrot/src/dynpmc'
cc -o digest_group.so lib-digest_group.o md2.o md4.o md5.o ripemd160.o sha.o sha1.o sha256.o sha512.o -lcrypto -lsocket -lnsl -ldl -lm -lpthread -lrt -lintl     -G 
ld: fatal: file lib-digest_group.o: wrong ELF class: ELFCLASS64
ld: fatal: File processing errors. No output written to digest_group.so

Attachments

tt262-linkflags.patch Download (1.8 KB) - added by rurban 6 years ago.

Change History

Changed 6 years ago by rurban

  • owner set to rurban
  • status changed from new to assigned
  • patch set to new

Changed 6 years ago by rurban

Changed 6 years ago by rurban

  • status changed from assigned to closed
  • resolution set to fixed
  • patch changed from new to applied

Tested ok on cygwin, mingw, msvc6, gentoo-i386, solaris-64int Applied as r36300.

Changed 6 years ago by rurban

config/gen/makefiles/parrot_embed_pl.in fixed with r36327.

Changed 6 years ago by doughera

No, this patch should be reverted. $linkflags is for using with the $link command. $ldflags is for using with the $ld command. See config/init/defaults.pm for the full definitions of these variables. The correct Configure.pl command line to get this to work should have included explicitly setting --ldflags as well as --linkflags.

Changed 6 years ago by jkeenan

  • status changed from closed to reopened
  • resolution fixed deleted

Reopening ticket per Andy D's request. Reini, can you take another look at this please?

Thank you very much. kid51

Changed 6 years ago by allison

  • status changed from reopened to closed
  • resolution set to wontfix

Reverted in r36450. Reini, please don't commit this kind of change without review.

Note: See TracTickets for help on using tickets.