bugGnash - The GNU Flash player - Bugs: bug #33223, Compile error in latest git master

 
 

bug #33223: Compile error in latest git master

Submitted by:  None
Submitted on:  Tue 03 May 2011 03:47:51 PM UTC  
 
Category: buildSeverity: 3 - Normal
Release: masterStatus: Fixed
Privacy: PublicAssigned to: None
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)

Wed 11 May 2011 08:07:35 AM UTC, comment #12:

This one seems fixed (two confirmations).

There's surely another laying around, which was the point of Benjamin's commits (configure looking for liblibva, which would hardly find).

Please file a ticket before fixing that bug, so there's a track of motivations.

Sandro Santilli <strk>
Project Member
Tue 10 May 2011 11:49:36 PM UTC, comment #11:

I didn't git revert cause I was looking for a better solution than just just reverting it. I assume we can close this now ?

Rob Savoye <rsavoye>
Project Administrator
Tue 10 May 2011 09:46:53 PM UTC, comment #10:

from the mailing list:

in configure.ac change

AM_CONDITIONAL(HAVE_VAAPI, test x"${found_va_incl}" = xyes)

to

AM_CONDITIONAL(HAVE_VAAPI, test x"${found_libva_incl}" = xyes)

Now, I'm able to build gnash on openSUSE 11.4. Hope you too!

Roman <romah>
Tue 10 May 2011 08:49:19 PM UTC, comment #9:

In general git master compiles again.

But I must use this additional patch:
http://pmbs.links2linux.de/package/view_file?file=susepatches.diff&package=gnash&project=Extra&srcmd5=141a46a00bba840e899d761d7b01fe80

(But this patch was declined by gnash developers on IRC ~ 1 month ago to go upstream although the right ffmpeg include dir is only chosen/set if this patch was applied ...)

Thanks for reverting the two commits (although the right way would be 'git revert' ...)

Anonymous
Tue 10 May 2011 05:46:19 PM UTC, comment #8:

Johannes, can you confirm the fix with latest git master ?

Sandro Santilli <strk>
Project Member
Tue 10 May 2011 05:45:10 PM UTC, comment #7:

Committed as of e3c511f47714ce0e503786f55cd34fea980af1bc.

Rob Savoye <rsavoye>
Project Administrator
Tue 10 May 2011 05:12:57 PM UTC, comment #6:

Please try the attached patch against 371d1df5f3401899c314b9c05b850481e026da31

(file #23382)

Sandro Santilli <strk>
Project Member
Sat 07 May 2011 01:34:13 PM UTC, comment #5:

I think this thread is related:
http://lists.gnu.org/archive/html/gnash-dev/2011-05/msg00011.html

Sandro Santilli <strk>
Project Member
Wed 04 May 2011 08:44:08 PM UTC, comment #4:

Compile error still exists because you didn't revert this commit (2nd one I mentioned):
http://git.savannah.gnu.org/cgit/gnash.git/commit/?id=77da6838eed4e2d01e732887fb2c6a1bded05c1b

Anonymous
Wed 04 May 2011 04:15:35 AM UTC, comment #3:

Sorry, test this please, even though it should fix your problem.

Rob Savoye <rsavoye>
Project Administrator
Tue 03 May 2011 08:51:49 PM UTC, comment #2:

I revert the two changes, although now it won't configure on the platform that forced this change in the beginning. Guess I'll look for a more portable solution.

Rob Savoye <rsavoye>
Project Administrator
Tue 03 May 2011 08:42:00 PM UTC, comment #1:

Revert these commits

http://git.savannah.gnu.org/cgit/gnash.git/commit/?id=2042ef675a2e7048236cd3c2a04cc239c7e70a7a

and

http://git.savannah.gnu.org/cgit/gnash.git/commit/?id=77da6838eed4e2d01e732887fb2c6a1bded05c1b

and all is fine again.

Please note that the file is named libva.pc and not va.pc as you suggest.

I had the same error on mesa/pipe-video - changes to libva and also there was all fine ...

Anonymous
Tue 03 May 2011 03:47:51 PM UTC, original submission:

see: http://pmbs.links2linux.de/package/rawlog?arch=x86_64&package=gnash&project=Extra&repository=openSUSE_11.3

Anonymous

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach File(s):
   
   
Comment:
   

Attached Files
file #23382:  vaapi.patch added by strk (568B - application/octet-stream)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by romah (Posted a comment)
  • -unavailable- added by strk (Posted a comment)
  • -unavailable- added by strk (reporter)
  • -unavailable- added by rsavoye (Updated 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 10 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Wed 11 May 2011 08:07:35 AM UTCstrkStatusWorks For Me=>Fixed
      Open/ClosedOpen=>Closed
    Tue 10 May 2011 11:49:36 PM UTCrsavoyeStatusReady For Test=>Works For Me
    Tue 10 May 2011 05:46:19 PM UTCstrkStatusWorks For Me=>Ready For Test
    Tue 10 May 2011 05:45:10 PM UTCrsavoyeStatusReady For Test=>Works For Me
    Tue 10 May 2011 05:12:57 PM UTCstrkAttached File-=>Added vaapi.patch, #23382
    Sat 07 May 2011 01:33:06 PM UTCstrkCarbon-Copy-=>Added johannes obermayr
    Wed 04 May 2011 04:15:35 AM UTCrsavoyeStatusNone=>Ready For Test
      Open/ClosedClosed=>Open
    Tue 03 May 2011 08:51:49 PM UTCrsavoyeOpen/ClosedOpen=>Closed

    Back to the top


    Powered by Savane 3.1-cleanup1