bugGnash - The GNU Flash player - Bugs: bug #36642, extensions don't load (FileIO test...

 
 
Show feedback again

bug #36642: extensions don't load (FileIO test failing)

Submitted by:  Sandro Santilli <strk>
Submitted on:  Tue Jun 12 17:24:36 2012  
 
Category: coreSeverity: 3 - Normal
Release: masterStatus: Confirmed
Privacy: PublicAssigned to: Rob Savoye <rsavoye>
Open/Closed: Open

Add a New Comment (Rich MarkupRich Markup):
   

You are not logged in

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

 

(Jump to the original submission Jump to the original submission)

Thu Feb 28 16:47:16 2013, comment #16:

That's as of commit ceaf5e4a988750da050514de5d36f22787d5257e

Sandro Santilli <strk>
Project Member
Thu Feb 28 16:46:51 2013, comment #15:

Still fails with --enable-visibility, succeeds without

Sandro Santilli <strk>
Project Member
Tue Feb 26 21:23:12 2013, comment #14:

Any status update on this bug ?

Rob Savoye <rsavoye>
Project AdministratorIn charge of this item.
Fri Feb 1 06:07:21 2013, comment #13:

Oh, nice, I see commits of you about visibility and flags.
Will try again after pulling 118600cd59d4ebfc6b557b162cfe25e9f510da53

Sandro Santilli <strk>
Project Member
Fri Feb 1 06:05:46 2013, comment #12:

I filed bug #38217 to track the build issue. Let's keep this one for the extensions. I can tell you that even with a vanilla ./configure (all the rest builds) I'm still having issue with FileIO:

Test Result Totals:
Total passes: 48974
Total unresolved: 9
Total expected failures: 1944
Total untested: 423
* Total real failures: 9

Unexpected failures follow:
--=[ extensions/fileio ]=--
FAIL: test: FileIO::open() [31]
FAIL: test: FileIO::fputs() [37]
FAIL: test: FileIO::fseek() [46]
FAIL: test: FileIO::fflush() [52]
FAIL: test: FileIO::ftell() [58]
FAIL: test: FileIO::fgetc() [65]
FAIL: test: FileIO::fgets() [73]
FAIL: test: FileIO::putchar() [85]
FAIL: test: FileIO::puts() [91]

make: *** [anal] Error 1

Sandro Santilli <strk>
Project Member
Thu Jan 31 19:26:18 2013, comment #11:

I'd suggest doing a "make clean" or "make distclean". I can't reproduce this symbol problem on any of my machines, so I assume it's a problem with your build. Your CXXFLAGS looks wrong, btw...

Rob Savoye <rsavoye>
Project AdministratorIn charge of this item.
Thu Jan 31 19:24:10 2013, comment #10:

Uff with CFLAGS/CXXFLAGS not overridden the build is still not 100% fine. There are undefined references to core symbols:
make[1]: Entering directory `/home/src/gnash/build/head/vanilla/testsuite/samples'
CXXLD clip_as_button2-TestRunner
../../testsuite/.libs/libtestsuite.a(libtestsuite_la-MovieTester.o): In function `gnash::MovieTester::findDisplayItemByName(gnash::MovieClip const&, std::basic_string<char, std::char_traits<char>, std::allocator<char> > const&)':
/usr/src/gnash/gnash-head/testsuite/MovieTester.cpp:334: undefined reference to `gnash::getStringTable(gnash::as_object const&)'

make dumpconfig shows:
CXXFLAGS: -g -O2 -W -Wall -Wcast-align -Wcast-qual -Wpointer-arith -Wreturn-type -Wnon-virtual-dtor -Wunused -fvisibility-inlines-hidden -fvisibility=hidden

Can it be the visibility flag ? Does it default to hidden now ?

Sandro Santilli <strk>
Project Member
Thu Jan 31 15:29:40 2013, comment #9:

The way you are setting them in your environment will override the configure determined settings. The way it's supposed to work is to let configure do it's thing, then supply any modified CFLAGS/CXXFLAGS on the command line to make. So we are honoring user flags by forcibly redefining them when you configure.

Rob Savoye <rsavoye>
Project AdministratorIn charge of this item.
Thu Jan 31 13:03:35 2013, comment #8:

It turns out that my overrides of CFLAGS/CXXFLAGS are the offending ones. If I don't defined them, build is fine.
Is that expected ? Should we always honour user flags ?

Sandro Santilli <strk>
Project Member
Thu Jan 31 11:13:32 2013, comment #7:

ouch, actually also the build is not completely fixed either.
The build of "Dejagnu" itself is, but there are other part of the testsuite which are failing due to missing symbols.

Example, in testsuite/misc-ming.all:
CXXLD ResolveEventsTest-Runner
../../testsuite/.libs/libtestsuite.a(libtestsuite_la-MovieTester.o): In function `gnash::MovieTester::findDisplayItemByName(gnash::MovieClip const&, std::basic_string<char, std::char_traits<char>, std::allocator<char> > const&)':
/usr/src/gnash/gnash-head/testsuite/MovieTester.cpp:334: undefined reference to `gnash::getStringTable(gnash::as_object const&)'
/usr/src/gnash/gnash-head/testsuite/MovieTester.cpp:335: undefined reference to `gnash::getVM(gnash::as_object const&)'
/usr/src/gnash/gnash-head/testsuite/MovieTester.cpp:336: undefined reference to `gnash::DisplayList::getDisplayObjectByName(gnash::string_table&, gnash::ObjectURI const&, bool) const'

I'm setting CFLAGS and CXXFLAGS in the env before configuring:
export CFLAGS="${CFLAGS} -O2 -g -m64"
export CXXFLAGS="${CXXFLAGS} -O2 -g -m64"

Sandro Santilli <strk>
Project Member
Thu Jan 31 10:54:00 2013, comment #6:

Thanks, your commit fixes the build of tests for me, but FileIO test still fails (rebuilt from scratch)

Sandro Santilli <strk>
Project Member
Wed Jan 30 18:58:28 2013, comment #5:

Should be fixed with commit #d58d080. I added the shared compilation flag to CFLAGS, it was already in CXXFLAGS. You
also might want to start by removing any old plugins .so files to get a clean start. It's working fine for me on several distros now.

Rob Savoye <rsavoye>
Project AdministratorIn charge of this item.
Tue Jan 29 14:30:12 2013, comment #4:

There seems to be something really bad with my build tree as beside the FileIO test (which interrupt the whole 'make check' process) I can't build Dejagnu under testsuite/misc-ming.all:

/usr/bin/ld: Dejagnu.o: relocation R_X86_64_32 against `.rodata.str1.1' can not be used when making a shared object; recompile with -fPIC

Any idea ?

Sandro Santilli <strk>
Project Member
Mon Jan 28 09:12:10 2013, comment #3:

I'm on Ubuntu 12.04.1 LTS ("precise")

Sandro Santilli <strk>
Project Member
Fri Jan 25 21:03:07 2013, comment #2:

When I try master on a Fedora machine, it works just fine. Are you on Debian or Ubuntu ? If so, I'll have to test there too. Here's what I get from "make check"

19090:1] 3 SECURITY: Extensions enabled, scanning plugin dir for load
19090:1] 3 SECURITY: Loading module: fileio from /home/rob/projects/gnu/i686-pc-linux-gnu/gnash/master/extensions/fileio/.libs
19090:1] 3 SECURITY: Initializing module: "fileio" from /home/rob/projects/gnu/i686-pc-linux-gnu/gnash/master/extensions/fileio/.libs
19090:1] 6 TRACE: [dejagnu.as master-21616-673c5de]
19090:1] 6 TRACE: SWF - LNX 10,0,12,36

PASS: test: FileIO::open() [29]
PASS: test: FileIO::fputs() [35]
PASS: test: FileIO::fseek() [44]
PASS: test: FileIO::fflush() [50]
PASS: test: FileIO::ftell() [56]
PASS: test: FileIO::fgetc() [63]
PASS: test: FileIO::fgets() [71]
PASS: test: FileIO::putchar() [83]
PASS: test: FileIO::puts() [89]

=== Summary ===

# of expected passes 9

Rob Savoye <rsavoye>
Project AdministratorIn charge of this item.
Fri Jan 25 16:19:36 2013, comment #1:

Still happens as of Gnash 0.8.11dev (master-21615-45a1c2f)
I have libtool and libltdl 2.4.2

Sandro Santilli <strk>
Project Member
Tue Jun 12 17:24:36 2012, original submission:

7068:1] 6 SECURITY: Initializing module: "fileio" from /extra/gnash-dev/lib/gnash/plugins
27068:1] 6 DEBUG: Trying to open shared library "/extra/gnash-dev/lib/gnash/plugins/fileio.so"
27068:1] 7 ERROR: lt_dlopenext(/extra/gnash-dev/lib/gnash/plugins/fileio.so): file not found

/extra/gnash-dev/lib/gnash/plugins/fileio.so: ELF 64-bit LSB shared object, x86-64, version 1 (SYSV), dynamically linked, not stripped

Gnash 0.8.11dev (master-21585-e0bc76c)

Sandro Santilli <strk>
Project Member

 

(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 rsavoye (Updated the item)
  • -unavailable- added by strk (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 7 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Thu Feb 28 16:46:51 2013strkStatusReady For Test=>Confirmed
    Tue Feb 26 21:23:12 2013rsavoyeStatusConfirmed=>Ready For Test
    Thu Jan 31 19:26:18 2013rsavoyeAssigned toNone=>rsavoye
    Thu Jan 31 19:24:10 2013strkStatusReady For Test=>Confirmed
    Thu Jan 31 15:29:40 2013rsavoyeStatusConfirmed=>Ready For Test
    Thu Jan 31 10:54:00 2013strkStatusReady For Test=>Confirmed
    Wed Jan 30 18:58:28 2013rsavoyeStatusNone=>Ready For Test
    Show feedback again

    Back to the top


    Powered by Savane 3.1-cleanup