Bug #3716

VideoSupport.c compilation error

Added by Mateusz Loskot about 2 years ago. Updated about 1 month ago.

Status:Fixed Start date:06/05/2011
Priority:Urgent Due date:06/07/2011
Assignee:Eric Gregory % Done:

100%
Category:-
Target version:0.11
Keywords:

Description

I’m compiling current Git master including most recent commit d9d917e8 on Linux 64-bit (Ubuntu 10.04) and getting the following compilation error:

pkg-config --cflags atk gconf-2.0 gdk-2.0 gdk-x11-2.0 gee-1.0 gexiv2 gio-unix-2.0 glib-2.0 gmodule-2.0 gstreamer-0.10 gstreamer-base-0.10 gstreamer-pbutils-0.10 gtk+-2.0 gudev-1.0 libexif libgphoto2 libsoup-2.4 libxml-2.0 sqlite3 unique-1.0 webkit-1.0 gthread-2.0

It seems something has been broken on 1st of June, in commits 1e561d43 and e4a8ad9b

History

#1
Updated by Bruno Girin about 2 years ago

I can confirm the same compilation problem on Ubuntu 10.10 32 bits. From the tests I've done, it looks like the C code generated expects declarations for GstDiscoverer and GstDiscovererInfo that don't exist in any of the gstreamer .h files. Would that be due to a dodgy .vapi for gstreamer?

#2
Updated by Adam Dingle about 2 years ago

  • Target version set to 0.11
  • Priority set to Urgent

We recently committed a change which uses!GstDiscoverer, which first appeared in GStreamer 0.10.31.

  • Ubuntu 10.04 has only GStreamer 10.28, but we don't support such an old version of Ubuntu and so a build breakage there is acceptable.
  • Ubuntu 10.10 has only GStreamer 10.30, so we've broken compatibility with that version of Ubuntu, which we have been supporting at least until now. This was unintentional. I'll talk to the team at Yorba today and we'll decide what to do about this.

#3
Updated by Adam Dingle about 2 years ago

  • Assignee changed from Anonymous to Eric Gregory

We discussed. We've decided to put this change off until 0.12 and retain Maverick support in 0.11.

#4
Updated by Mateusz Loskot about 2 years ago

Adam,

This is good news. I don't mind to upgrade Shotwell dependencies, but I feel GStreamer is complex and upgrading it manually in Ubuntu 10.04 may cause a lot of problems. Or, it's painless to host 10.28 and 10.30 side by side. Is it?

Thanks guys!

Mat

#5
Updated by Adam Dingle about 2 years ago

Maintaining parallel versions of GStreamer is possible but not so easy (I've done it before). I recommend that you simply wait until we revert this change, which will happen soon – then you should be able to build with an older GStreamer again.

#6
Updated by Eric Gregory about 2 years ago

  • Resolution set to fixed
  • % Done changed from 0 to 100

Fixed in ce0c320a96eabcf7a6ec20386de346aa6712a075

This change reverts ticket #3671

#7
Updated by Mateusz Loskot about 2 years ago

I confirm the fix. Everything builds well now.

Thanks!

#8
Updated by Mateusz Loskot almost 2 years ago

  • Description updated (diff)

Has anything changed regarding the required gstreamer version?

I'm trying to build current Git mainstream and getting this:

Requested 'gstreamer-pbutils-0.10 >= 0.10.32' but version of GStreamer Base Utils Library is 0.10.30

#9
Updated by Eric Gregory almost 2 years ago

This has changed -- after some back and forth, we decided to break Maverick support for other reasons. 

Based on that decision, the change in #3671 was re-applied.

#10
Updated by Mateusz Loskot almost 2 years ago

I see. It looks it's time for me to upgrade and move away from the Maverick :)

#11
Updated by Charles Lindsay about 1 month ago

  • Status changed from 5 to Fixed

Also available in: Atom PDF