Re: [oe] [meta-oe][PATCH] mpv: Media Player
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 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
---
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 000..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
Re: [oe] [meta-oe][PATCH] mpv: Media Player
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 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
> >>> ---
> >>> 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 000..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}${p
Re: [oe] [meta-oe][PATCH] mpv: Media Player
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 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
---
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 000..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
Re: [oe] [meta-oe][PATCH] mpv: Media Player
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 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
---
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 000..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
Re: [oe] [meta-oe][PATCH] mpv: Media Player
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 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
> ---
>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 000..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
>
--
___
Openembedded-devel mailing list
Openembedded-devel@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-devel
Re: [oe] [meta-oe][PATCH] mpv: Media Player
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
---
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 000..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
Re: [oe] [meta-oe][PATCH] mpv: Media Player
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
> >> ---
> >> 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 000..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.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
Re: [oe] [meta-oe][PATCH] mpv: Media Player
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
---
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 000..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
--
___
Openembedded-devel mailing list
Openembedded-devel@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-devel
Re: [oe] [meta-oe][PATCH] mpv: Media Player
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
> ---
> 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 000..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.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
[oe] [meta-oe][PATCH] mpv: Media Player
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
---
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 000..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
--
___
Openembedded-devel mailing list
Openembedded-devel@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-devel
