[oe] [PATCH [1/2]] eio: New recipe for eio
Paul Menzel
paulepanter at users.sourceforge.net
Fri Apr 1 10:00:35 UTC 2011
Dear Philippe,
Am Freitag, den 01.04.2011, 10:25 +0300 schrieb Philippe De Swert:
> Thanks for the review.
No problem.
> On 31 March 2011 23:23, Paul Menzel <paulepanter at users.sourceforge.net> wrote:
> > Am Donnerstag, den 31.03.2011, 22:39 +0300 schrieb Philippe De Swert:
> >
> > »eio: add initial recipe (svn)« or something like that would be a
> > better commit summary in my opinion.
>
> I'll keep that in mind
Additionally »[PATCH [1/2]]« seems to confuse `git am` to use
] eio: New recipe for eio
as the commit summary. So only no nested brackets please.
I found one other problem. The license is LGPL 2.1+ and not MIT or BSD
[1].
> >> +
> >> +# Some upgrade path tweaking
> >> +AUTO_LIBNAME_PKGS = ""
> >
> > What upgrade patch?
>
> Yeah I should have double checked that... Just put it there as all the
> other enlightenment .inc files had it.
Maybe Martin can comment on that.
> >> diff --git a/recipes/efl1/eio_svn.bb b/recipes/efl1/eio_svn.bb
> >> new file mode 100644
> >> index 0000000..6a12dcf
> >> --- /dev/null
> >> +++ b/recipes/efl1/eio_svn.bb
> >> @@ -0,0 +1,13 @@
> >> +require eio.inc
> >> +
> >> +SRCREV = "${EFL_SRCREV}"
> >> +PV = "0.1.0+svnr${SRCPV}"
> >> +PR = "${INC_PR}.0"
> >> +DEFAULT_PREFERENCE = "-1"
> >
> > It is the only recipe, so `DP = "-1"` should not be useful.
>
> Just wanted to be future proof for when the actual versioned recipes come.
I think it should be adapted if there is a recipe for a released
version.
Please resend the patch as [PATCH 1/2 v2] after Martin has commented.
Thanks,
Paul
[1] http://trac.enlightenment.org/e/browser/trunk/eio/COPYING?rev=57852
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: This is a digitally signed message part
URL: <http://lists.openembedded.org/pipermail/openembedded-devel/attachments/20110401/4fa3db0c/attachment-0002.sig>
More information about the Openembedded-devel
mailing list