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

 
 

bug #33223: Compile error in latest git master

Submitter:  None
Submitted:  Tue 03 May 2011 03:47:51 PM UTC
   
 
Category:  build Severity:  3 - Normal
Release:  master Status:  Fixed
Privacy:  Public Assigned to:  None
Open/Closed:  Closed
* Mandatory Fields

Add a New Comment Rich Markup
   

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>
Group 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>
Group 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>
Group Member
Tue 10 May 2011 05:45:10 PM UTC, comment #7: 

Committed as of e3c511f47714ce0e503786f55cd34fea980af1bc.

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

Please try the attached patch against 371d1df5f3401899c314b9c05b850481e026da31

(file #23382)

Sandro Santilli <strk>
Group Member
Sat 07 May 2011 01:34:13 PM UTC, comment #5: 
Sandro Santilli <strk>
Group 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>
Group 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>
Group 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:  
Anonymous

 

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

Attach Files:
   
   
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
  • -email is unavailable- added by romah (Posted a comment)
  • -email is unavailable- added by strk (Posted a comment)
  • -email is unavailable- added by strk (reporter)
  • -email is unavailable- added by rsavoye (Updated the item)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

    Only logged-in users can vote.

     

    Follow 10 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2011-05-11 strk StatusWorks For Me Fixed
        Open/ClosedOpen Closed
    2011-05-10 rsavoye StatusReady For Test Works For Me
    2011-05-10 strk StatusWorks For Me Ready For Test
    2011-05-10 rsavoye StatusReady For Test Works For Me
    2011-05-10 strk Attached File- Added vaapi.patch, #23382
    2011-05-07 strk Carbon-Copy- Added johannes obermayr <johannesobermayr@gmx.de>
    2011-05-04 rsavoye StatusNone Ready For Test
        Open/ClosedClosed Open
    2011-05-03 rsavoye Open/ClosedOpen Closed

    Back to the top

    Powered by Savane 3.13-caa5.
    Corresponding source code