* [meta-oe][PATCH] mpv: Media Player @ 2016-03-09 6:20 Gary Thomas 2016-03-09 19:15 ` Martin Jansa 0 siblings, 1 reply; 10+ messages in thread From: Gary Thomas @ 2016-03-09 6:20 UTC (permalink / raw) To: openembedded-devel; +Cc: Gary Thomas 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 <gary@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 \ + --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/" -- 2.5.0 ^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [meta-oe][PATCH] mpv: Media Player 2016-03-09 6:20 [meta-oe][PATCH] mpv: Media Player Gary Thomas @ 2016-03-09 19:15 ` Martin Jansa 2016-03-10 4:42 ` Gary Thomas 0 siblings, 1 reply; 10+ messages in thread From: Martin Jansa @ 2016-03-09 19:15 UTC (permalink / raw) To: openembedded-devel; +Cc: Gary Thomas [-- Attachment #1: Type: text/plain, Size: 2801 bytes --] 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 <gary@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. > + --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? -- Martin 'JaMa' Jansa jabber: Martin.Jansa@gmail.com [-- Attachment #2: Digital signature --] [-- Type: application/pgp-signature, Size: 188 bytes --] ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [meta-oe][PATCH] mpv: Media Player 2016-03-09 19:15 ` Martin Jansa @ 2016-03-10 4:42 ` Gary Thomas 2016-03-10 7:33 ` Martin Jansa 0 siblings, 1 reply; 10+ messages in thread From: Gary Thomas @ 2016-03-10 4:42 UTC (permalink / raw) To: Martin Jansa, openembedded-devel 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 <gary@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. > >> + --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. -- ------------------------------------------------------------ Gary Thomas | Consulting for the MLB Associates | Embedded world ------------------------------------------------------------ ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [meta-oe][PATCH] mpv: Media Player 2016-03-10 4:42 ` Gary Thomas @ 2016-03-10 7:33 ` Martin Jansa 2016-03-10 7:40 ` Gary Thomas 0 siblings, 1 reply; 10+ messages in thread From: Martin Jansa @ 2016-03-10 7:33 UTC (permalink / raw) To: Gary Thomas; +Cc: openembedded-devel [-- Attachment #1: Type: text/plain, Size: 3473 bytes --] 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 <gary@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? -- Martin 'JaMa' Jansa jabber: Martin.Jansa@gmail.com [-- Attachment #2: Digital signature --] [-- Type: application/pgp-signature, Size: 188 bytes --] ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [meta-oe][PATCH] mpv: Media Player 2016-03-10 7:33 ` Martin Jansa @ 2016-03-10 7:40 ` Gary Thomas 2016-03-10 7:57 ` Martin Jansa 0 siblings, 1 reply; 10+ messages in thread From: Gary Thomas @ 2016-03-10 7:40 UTC (permalink / raw) To: openembedded-devel 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 <gary@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 ------------------------------------------------------------ ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [meta-oe][PATCH] mpv: Media Player 2016-03-10 7:40 ` Gary Thomas @ 2016-03-10 7:57 ` Martin Jansa 2016-03-10 8:18 ` Gary Thomas 0 siblings, 1 reply; 10+ messages in thread From: Martin Jansa @ 2016-03-10 7:57 UTC (permalink / raw) To: openembedded-devel I did and there was still: FILES_${PN} += "${prefix}" that's why I've asked again here. On Thu, Mar 10, 2016 at 8:40 AM, Gary Thomas <gary@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 <gary@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 > ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [meta-oe][PATCH] mpv: Media Player 2016-03-10 7:57 ` Martin Jansa @ 2016-03-10 8:18 ` Gary Thomas 2016-03-10 13:02 ` Gary Thomas 0 siblings, 1 reply; 10+ messages in thread From: Gary Thomas @ 2016-03-10 8:18 UTC (permalink / raw) To: openembedded-devel 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? > > On Thu, Mar 10, 2016 at 8:40 AM, Gary Thomas <gary@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 <gary@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 ------------------------------------------------------------ ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [meta-oe][PATCH] mpv: Media Player 2016-03-10 8:18 ` Gary Thomas @ 2016-03-10 13:02 ` Gary Thomas 2016-03-10 13:11 ` Martin Jansa 0 siblings, 1 reply; 10+ messages in thread From: Gary Thomas @ 2016-03-10 13:02 UTC (permalink / raw) To: openembedded-devel 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" > >> >> On Thu, Mar 10, 2016 at 8:40 AM, Gary Thomas <gary@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 <gary@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 ------------------------------------------------------------ ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [meta-oe][PATCH] mpv: Media Player 2016-03-10 13:02 ` Gary Thomas @ 2016-03-10 13:11 ` Martin Jansa 2016-03-10 13:10 ` Gary Thomas 0 siblings, 1 reply; 10+ messages in thread From: Martin Jansa @ 2016-03-10 13:11 UTC (permalink / raw) To: openembedded-devel [-- Attachment #1: Type: text/plain, Size: 5780 bytes --] 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 <gary@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 <gary@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.Jansa@gmail.com [-- Attachment #2: Digital signature --] [-- Type: application/pgp-signature, Size: 188 bytes --] ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [meta-oe][PATCH] mpv: Media Player 2016-03-10 13:11 ` Martin Jansa @ 2016-03-10 13:10 ` Gary Thomas 0 siblings, 0 replies; 10+ messages in thread From: Gary Thomas @ 2016-03-10 13:10 UTC (permalink / raw) To: openembedded-devel On 2016-03-10 14:11, Martin Jansa wrote: > 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? > I'll look at it and see if I can get it to play nice. >> >>> >>>> >>>> On Thu, Mar 10, 2016 at 8:40 AM, Gary Thomas <gary@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 <gary@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 ------------------------------------------------------------ ^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2016-03-10 13:10 UTC | newest] Thread overview: 10+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2016-03-09 6:20 [meta-oe][PATCH] mpv: Media Player Gary Thomas 2016-03-09 19:15 ` Martin Jansa 2016-03-10 4:42 ` Gary Thomas 2016-03-10 7:33 ` Martin Jansa 2016-03-10 7:40 ` Gary Thomas 2016-03-10 7:57 ` Martin Jansa 2016-03-10 8:18 ` Gary Thomas 2016-03-10 13:02 ` Gary Thomas 2016-03-10 13:11 ` Martin Jansa 2016-03-10 13:10 ` Gary Thomas
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.