Opened 8 years ago
Closed 8 years ago
Last modified 6 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)
Changed 8 years ago by
Attachment: | ticket_12762_configure.patch added |
---|
comment:1 Changed 8 years ago by
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 8 years ago by
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 8 years ago by
Milestone: | unknown → 0.28.1 |
---|
comment:4 Changed 8 years ago by
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 8 years ago by
Component: | MythTV - General → MythTV - configure script |
---|---|
Owner: | set to JYA |
comment:6 Changed 8 years ago by
Owner: | changed from JYA to Peter Bennett |
---|---|
Status: | new → accepted |
comment:7 Changed 8 years ago by
Resolution: | → fixed |
---|---|
Status: | accepted → closed |
comment:9 Changed 6 years ago by
Owner: | changed from Peter Bennett to Peter Bennett |
---|
Patch to fix this