On Thu, Mar 10, 2016 at 02:02:23PM +0100, Gary Thomas wrote: > On 2016-03-10 09:18, Gary Thomas wrote: > > On 2016-03-10 08:57, Martin Jansa wrote: > >> I did and there was still: > >> FILES_${PN} += "${prefix}" > >> > >> that's why I've asked again here. > > > > Now I understand why you're asking. I'll verify that it works correctly > > without that line, just using the standard packaging. If so, need I send > > another patch revision or can you just leave that one line out when you > > merge? > > The answer is yes, it does install to some non-standard paths. At least as > the configuration currently stands, it needs > FILES_${PN} += "${prefix}/etc ${prefix}/share"
Why doesn't it respect ${sysconfdir} and ${datadir}? Maybe you need to pass more configure flags? > > > > >> > >> On Thu, Mar 10, 2016 at 8:40 AM, Gary Thomas <g...@mlbassoc.com> wrote: > >> > >>> On 2016-03-10 08:33, Martin Jansa wrote: > >>> > >>>> On Thu, Mar 10, 2016 at 05:42:39AM +0100, Gary Thomas wrote: > >>>> > >>>>> On 2016-03-09 20:15, Martin Jansa wrote: > >>>>> > >>>>>> On Wed, Mar 09, 2016 at 07:20:33AM +0100, Gary Thomas wrote: > >>>>>> > >>>>>>> mpv is a fork of mplayer2 and MPlayer. It shares some features with > >>>>>>> the former projects while introducing many more. > >>>>>>> > >>>>>>> Signed-off-by: Gary Thomas <g...@mlbassoc.com> > >>>>>>> --- > >>>>>>> meta-oe/recipes-multimedia/mplayer/mpv_0.15.0.bb | 60 > >>>>>>> ++++++++++++++++++++++++ > >>>>>>> 1 file changed, 60 insertions(+) > >>>>>>> create mode 100644 > >>>>>>> meta-oe/recipes-multimedia/mplayer/mpv_0.15.0.bb > >>>>>>> > >>>>>>> diff --git a/meta-oe/recipes-multimedia/mplayer/mpv_0.15.0.bb > >>>>>>> b/meta-oe/recipes-multimedia/mplayer/mpv_0.15.0.bb > >>>>>>> new file mode 100644 > >>>>>>> index 0000000..37e4b02 > >>>>>>> --- /dev/null > >>>>>>> +++ b/meta-oe/recipes-multimedia/mplayer/mpv_0.15.0.bb > >>>>>>> @@ -0,0 +1,60 @@ > >>>>>>> +SUMMARY = "Open Source multimedia player" > >>>>>>> +SECTION = "multimedia" > >>>>>>> +HOMEPAGE = "http://www.mpv.io/" > >>>>>>> +DEPENDS = "zlib ffmpeg jpeg virtual/libx11 xsp libxv libxscrnsaver" > >>>>>>> + > >>>>>>> +# Depends on xsp, libxv, virtual/libx11, libxscrnsaver > >>>>>>> +REQUIRED_DISTRO_FEATURES = "x11" > >>>>>>> + > >>>>>>> +LICENSE = "GPLv2+" > >>>>>>> +LIC_FILES_CHKSUM = > >>>>>>> "file://LICENSE;md5=91f1cb870c1cc2d31351a4d2595441cb" > >>>>>>> + > >>>>>>> +SRC_URI = " > >>>>>>> https://github.com/mpv-player/mpv/archive/v${PV}.tar.gz;name=mpv \ > >>>>>>> + > >>>>>>> http://www.freehackers.org/~tnagy/release/waf-1.8.12;name=waf \ > >>>>>>> +" > >>>>>>> +SRC_URI[mpv.md5sum] = "9042bd3fbff2bc8ba0b7fadaa4a22101" > >>>>>>> +SRC_URI[mpv.sha256sum] = > >>>>>>> "7d31217ba8572f364fcea2955733f821374ae6d8c6d8f22f8bc63c44c0400bdc" > >>>>>>> +SRC_URI[waf.md5sum] = "cef4ee82206b1843db082d0b0506bf71" > >>>>>>> +SRC_URI[waf.sha256sum] = > >>>>>>> "01bf2beab2106d1558800c8709bc2c8e496d3da4a2ca343fe091f22fca60c98b" > >>>>>>> + > >>>>>>> +EXTRA_OECONF = " \ > >>>>>>> + --prefix=${D}/usr \ > >>>>>>> > >>>>>> why not ${prefix} and does it really need to be prefixed with ${D} > >>>>>> (unlike mandir bellow)? Ugly waf. > >>>>>> > >>>>> > >>>> What about ${D}? > >>>> > >>>> + --mandir=${mandir} \ > >>>>>>> + --target=${SIMPLE_TARGET_SYS} \ > >>>>>>> + --disable-gl \ > >>>>>>> + --disable-libsmbclient \ > >>>>>>> + --disable-libass \ > >>>>>>> + --disable-lua \ > >>>>>>> + --disable-libass-osd \ > >>>>>>> + --disable-encoding \ > >>>>>>> + --disable-libbluray \ > >>>>>>> + --disable-dvdread \ > >>>>>>> + --disable-dvdnav \ > >>>>>>> + --disable-cdda \ > >>>>>>> + --disable-enca \ > >>>>>>> + --disable-libguess \ > >>>>>>> + --disable-uchardet \ > >>>>>>> + --disable-rubberband \ > >>>>>>> + --disable-lcms2 \ > >>>>>>> + --disable-vapoursynth \ > >>>>>>> + --disable-vapoursynth-lazy \ > >>>>>>> + --enable-libarchive \ > >>>>>>> +" > >>>>>>> + > >>>>>>> +do_configure() { > >>>>>>> + if [ ! -L ../waf ]; then > >>>>>>> + chmod a+x ../waf-1.8.12 > >>>>>>> + ln -s waf-1.8.12 ../waf > >>>>>>> + fi > >>>>>>> + ../waf configure ${EXTRA_OECONF} > >>>>>>> +} > >>>>>>> + > >>>>>>> +do_compile () { > >>>>>>> + ../waf build > >>>>>>> +} > >>>>>>> + > >>>>>>> +do_install() { > >>>>>>> + ../waf install > >>>>>>> +} > >>>>>>> + > >>>>>>> +FILES_${PN} += "/usr/" > >>>>>>> > >>>>>> > >>>>>> ${prefix}? > >>>>>> > >>>>>> Does it install files in strange directories not covered by standard > >>>>>> FILES? > >>>>>> > >>>>>> > >>>>> Use of ${prefix} does seem correct, I've updated the recipe. > >>>>> Patch v2 has been dispatched, including a required update to > >>>>> the libass package which was necessary to get a fully functional > >>>>> mpv. > >>>>> > >>>> > >>>> Does it install files in strange directories not covered by standard > >>>> FILES? > >>>> > >>> > >>> No, but it doesn't seem to have any notion of adjusting the install > >>> directory except at configure time, hence ${D}${prefix} > >>> > >>> Please refer to the new patchset to see how I've changed the recipe. > >>> > > > > > -- > ------------------------------------------------------------ > Gary Thomas | Consulting for the > MLB Associates | Embedded world > ------------------------------------------------------------ > -- > _______________________________________________ > Openembedded-devel mailing list > Openembedded-devel@lists.openembedded.org > http://lists.openembedded.org/mailman/listinfo/openembedded-devel -- Martin 'JaMa' Jansa jabber: martin.ja...@gmail.com
signature.asc
Description: Digital signature
-- _______________________________________________ Openembedded-devel mailing list Openembedded-devel@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-devel