[Bug 795526] recipes: Add new recipe gst-omx

[GStreamer (GNOME Bugzilla)]

https://bugzilla.gnome.org/show_bug.cgi?id=795526

Olivier Crête <olivier.crete at ocrete.ca> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #371354|none                        |reviewed
             status|                            |

--- Comment #2 from Olivier Crête <olivier.crete at ocrete.ca> ---
Review of attachment 371354:
 --> (https://bugzilla.gnome.org/review?bug=795526&attachment=371354)

I would put gst-omx into the codecs package instead of it's own package.

::: recipes/gst-omx-1.0.recipe
@@ +18,3 @@
+        self.append_env['CFLAGS'] = " -Wno-error "
+        self.append_env['CXXFLAGS'] = " -Wno-error "
+        self.append_env['CPPFLAGS'] = " -Wno-error "

Those shouldn't be needed, the -Werror should only be for pre-releases

@@ +24,3 @@
+
+        if self.config.variants.rpi:
+            eq = '=' if self.config.sysroot else ''

I think you can put the '=' unconditionally

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.