Opened 10 years ago
Closed 10 years ago
Last modified 8 years ago
#12762 closed Bug Report - General (fixed)
Raspberry Pi: configure script requires additional exports
| Reported by: | 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)
Change History (10)
by , 10 years ago
| Attachment: | ticket_12762_configure.patch added |
|---|
comment:1 by , 10 years ago
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 by , 10 years ago
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 by , 10 years ago
| Milestone: | unknown → 0.28.1 |
|---|
comment:4 by , 10 years ago
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 by , 10 years ago
| Component: | MythTV - General → MythTV - configure script |
|---|---|
| Owner: | set to |
comment:6 by , 10 years ago
| Owner: | changed from to |
|---|---|
| Status: | new → accepted |
comment:7 by , 10 years ago
| Resolution: | → fixed |
|---|---|
| Status: | accepted → closed |
comment:9 by , 8 years ago
| Owner: | changed from to |
|---|

Patch to fix this