From: Martin Jansa <martin.jansa@gmail.com>
To: Gary Thomas <gary@mlbassoc.com>
Cc: openembedded-devel@lists.openembedded.org
Subject: Re: [meta-oe][PATCH] mplayer2: Remove blacklist
Date: Thu, 12 Feb 2015 23:20:59 +0100 [thread overview]
Message-ID: <20150212222059.GI2272@jama> (raw)
In-Reply-To: <54DD1C8B.90205@mlbassoc.com>
[-- Attachment #1: Type: text/plain, Size: 2768 bytes --]
On Thu, Feb 12, 2015 at 02:35:07PM -0700, Gary Thomas wrote:
> On 2015-02-12 11:30, Martin Jansa wrote:
> > On Tue, Feb 03, 2015 at 06:44:07AM -0700, Gary Thomas wrote:
> >> mplayer2 no longer needs to be blacklisted as OE-core has moved
> >> to libav_9.16
> >
> > Please fix following issues before blacklisting it:
> > mplayer2-2.0+gitrAUTOINC+2c378c71a4: mplayer2 rdepends on libpostproc,
> > but it isn't a build dependency? [build-deps]
> > mplayer2-2.0+gitrAUTOINC+2c378c71a4: mplayer2 rdepends on libvorbis, but
> > it isn't a build dependency? [build-deps]
> > mplayer2-2.0+gitrAUTOINC+2c378c71a4: mplayer2 rdepends on libxscrnsaver,
> > but it isn't a build dependency? [build-deps]
> > mplayer2-2.0+gitrAUTOINC+2c378c71a4: mplayer2 rdepends on mpg123, but it
> > isn't a build dependency? [build-deps]
>
> This must be an inferred dependency as mpg123 is not in meta-oe, but
> rather meta-multimedia and it is possible to build mplayer using only
> meta-oe and not meta-multimedia. How should that be handled?
PACKAGECONFIG[mpg123] which isn't enabled by default (so that people
with meta-multimedia can enable it if they want and by default it's
explicitly disabled for everybody
> > mplayer2-2.0+gitrAUTOINC+2c378c71a4: mplayer2 rdepends on portaudio-v19,
> > but it isn't a build dependency? [build-deps]
> >
> >> Signed-off-by: Gary Thomas <gary@mlbassoc.com>
> >> ---
> >> meta-oe/recipes-multimedia/mplayer/mplayer2_git.bb | 2 --
> >> 1 file changed, 2 deletions(-)
> >>
> >> diff --git a/meta-oe/recipes-multimedia/mplayer/mplayer2_git.bb b/meta-oe/recipes-multimedia/mplayer/mplayer2_git.bb
> >> index 7c04ff1..089aa15 100644
> >> --- a/meta-oe/recipes-multimedia/mplayer/mplayer2_git.bb
> >> +++ b/meta-oe/recipes-multimedia/mplayer/mplayer2_git.bb
> >> @@ -13,8 +13,6 @@ RCONFLICTS_${PN} = "mplayer"
> >> LICENSE = "GPLv3"
> >> LIC_FILES_CHKSUM = "file://LICENSE;md5=d32239bcb673463ab874e80d47fae504"
> >>
> >> -PNBLACKLIST[mplayer2] ?= "Requires newer libav which has negative D_P"
> >> -
> >> SRC_URI = "git://repo.or.cz/mplayer.git \
> >> file://0001-configure-don-t-disable-ASS-support-when-explicitly-.patch \
> >> "
> >> --
> >> 1.9.1
> >>
> >> --
> >> _______________________________________________
> >> Openembedded-devel mailing list
> >> Openembedded-devel@lists.openembedded.org
> >> http://lists.openembedded.org/mailman/listinfo/openembedded-devel
> >
>
> --
> ------------------------------------------------------------
> Gary Thomas | Consulting for the
> MLB Associates | Embedded world
> ------------------------------------------------------------
--
Martin 'JaMa' Jansa jabber: Martin.Jansa@gmail.com
[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 188 bytes --]
next prev parent reply other threads:[~2015-02-12 22:20 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-02-03 13:44 [meta-oe][PATCH] mplayer2: Remove blacklist Gary Thomas
2015-02-12 18:30 ` Martin Jansa
2015-02-12 21:35 ` Gary Thomas
2015-02-12 22:20 ` Martin Jansa [this message]
2015-02-13 10:58 ` Paul Eggleton
2015-02-13 11:10 ` Gary Thomas
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20150212222059.GI2272@jama \
--to=martin.jansa@gmail.com \
--cc=gary@mlbassoc.com \
--cc=openembedded-devel@lists.openembedded.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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.