Opened 3 years ago

Closed 3 years ago

Last modified 13 months ago

#12762 closed Bug Report - General (fixed)

Raspberry Pi: configure script requires additional exports

Reported by: Peter Bennett <pgbennett@…> Owned by: Peter Bennett
Priority: minor Milestone: 0.28.1
Component: MythTV - configure script Version: 0.28.0
Severity: medium Keywords:
Cc: Ticket locked: no

Description

When building for raspberry pi, the configure script will not enable openmax unless you supply a bunch of exports first.

Attachments (1)

ticket_12762_configure.patch (1.8 KB) - added by Peter Bennett <pgbennett@…> 3 years ago.
Patch to fix this

Download all attachments as: .zip

Change History (10)

Changed 3 years ago by Peter Bennett <pgbennett@…>

Patch to fix this

comment:1 Changed 3 years ago by Peter Bennett <pgbennett@…>

This patch allows you to configure without the exports, also will throw an error if you explicitly enable openmax and it cannot enable it.

I have tested this with raspberry pi builds but not yet with desktop build. I do not anticipate problems with the desktop build, but it needs to be tested.

comment:2 Changed 3 years ago by Gary Buhrmaster <gary.buhrmaster@…>

This patch appears to hard code certain implementation specific paths. It would seem to be a better (if not easier to accomplish) approach is to get upstream to provide appropriate pkg-config include/library settings. I seem to recall some of these issues were discussed in excess of 3 years ago in the gstreamer forums (who had openmax support for some time). Might be worth looking at their approach for platform specific libraries.

comment:3 Changed 3 years ago by Stuart Auchterlonie

Milestone: unknown0.28.1

comment:4 Changed 3 years ago by Peter Bennett <pgbennett@…>

I agree this is far from perfect. However the situation is worse without the patch, where you have to supply the implementation specific paths in export statements before running configure, if you have any hope of getting openmax support to be enabled.

I can confirm that the paths used are correct for Raspberry Pi Raspbian Jessie as well as Raspberry Pi Ubuntu Mate Xenial.

comment:5 Changed 3 years ago by Karl Egly

Component: MythTV - GeneralMythTV - configure script
Owner: set to JYA

comment:6 Changed 3 years ago by Peter Bennett

Owner: changed from JYA to Peter Bennett
Status: newaccepted

comment:7 Changed 3 years ago by Peter Bennett <pgbennett@…>

Resolution: fixed
Status: acceptedclosed

In 92b1f3981df7b65f3f05207e180216eccc20dfec/mythtv:

configure - fix so that build for raspberry pi does not need a bunch of overrides to get openmax enabled

Fixes #12762

comment:8 Changed 3 years ago by Peter Bennett <pgbennett@…>

In 104b104fa5db7604d5e525c04d9a65d650dc6e33/mythtv:

configure - fix so that build for raspberry pi does not need a bunch of overrides to get openmax enabled

Fixes #12762

(cherry picked from commit 92b1f3981df7b65f3f05207e180216eccc20dfec)

comment:9 Changed 13 months ago by Peter Bennett

Owner: changed from Peter Bennett to Peter Bennett
Note: See TracTickets for help on using tickets.