bugGnash - The GNU Flash player - Bugs: bug #19562, server\parser\movie_def_impl.h...

 
 

bug #19562: server\parser\movie_def_impl.h contains pthread.h dependency, not available on win32

Submitted by:  Eric Hughes <eh9>
Submitted on:  Mon 09 Apr 2007 08:58:13 PM UTC  
 
Category: buildSeverity: 3 - Normal
Release: NoneStatus: Fixed
Privacy: PublicAssigned to: Bastiaan Jacques <bjacques>
Open/Closed: Closed

Add a New Comment(Rich 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)

Fri 13 Apr 2007 07:42:08 AM UTC, comment #6:

As a note to bug starter: I've removed all references to pthreads in the codebase. (You will need to rerun ./autogen.sh.)

Also, I'm interested in hearing about your attempts to build Gnash on windows. Perhaps you could visit #gnash on irc.freenode.net some time.

Bastiaan Jacques <bjacques>
Project MemberIn charge of this item.
Thu 12 Apr 2007 07:50:40 PM UTC, comment #5:

I'm taking this bug, as I had locally made this change a while ago. I will commit it after a little more testing.

Bastiaan Jacques <bjacques>
Project MemberIn charge of this item.
Tue 10 Apr 2007 01:43:06 PM UTC, comment #4:

But we'd like to run as close to "native" as possible, and it's an easy change to use Boost mutexes anyway.

Rob Savoye <rsavoye>
Project Administrator
Tue 10 Apr 2007 10:40:29 AM UTC, comment #3:

Sure, it should be replaced, but the current compilation problem could possible be solved with pthread-win32

Tomas Groth <tgc>
Tue 10 Apr 2007 10:25:24 AM UTC, comment #2:

Actually, Eric's right. It ought to be replaced by boost-threads.

Markus Gothe <nihilus>
Project Member
Tue 10 Apr 2007 08:58:56 AM UTC, comment #1:

Have you tried using the win32 pthread implementation: http://sourceware.org/pthreads-win32/ ?

Tomas Groth <tgc>
Mon 09 Apr 2007 08:58:13 PM UTC, original submission:

server\parser\movie_def_impl.h contains inclusion and use of the pthread library, which isn't available on win32. Should be replaced with Boost::Thread.

Extract:
========
class MovieLoader {
...
pthread_mutex_t _mutex;
pthread_t _thread;

Eric Hughes <eh9>

 

(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 bjacques (Posted a comment)
  • -unavailable- added by rsavoye (Posted a comment)
  • -unavailable- added by nihilus (Posted a comment)
  • -unavailable- added by tgc (Posted a comment)
  • -unavailable- added by eh9 (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 6 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Thu 12 Apr 2007 09:54:24 PM UTCbjacquesStatusIn Progress=>Fixed
      Open/ClosedOpen=>Closed
    Thu 12 Apr 2007 07:50:40 PM UTCbjacquesStatusNone=>In Progress
      Assigned tonihilus=>bjacques
    Tue 10 Apr 2007 10:25:24 AM UTCnihilusAssigned toNone=>nihilus
    Mon 09 Apr 2007 08:58:13 PM UTCeh9Carbon-Copy-=>Added rsavoye

    Back to the top


    Powered by Savane 3.1-cleanup1