[oe] [PATCH] gpe-mini-browser2_svn : remove fixes SRCREV

[Martin Jansa]

On Fri, Jul 22, 2011 at 03:18:37PM +0300, Philippe De Swert wrote:
> Hi Paul,
> 
> On 22/07/2011, Paul Menzel <paulepanter at users.sourceforge.net> wrote:
> > thank you for your patch.
> 
> Thank you for the review.
> 
> > Please do not remove the `SRCREV` but replace it by a valid and working
> > revision. Otherwise the builds would not be reproducible.
> >
> 
> I already wondered who added one (as I originally added the svn
> recipe). Anyway I thought the whole idea of _svn etc were to have
> bleeding edge builds. If we wanted stable there would be a versioned
> recipe.

If you want it bleeding then you or your distribution will set

SRCREV_pn-gpe-mini-browser2 = "${AUTOREV}"

in local.conf or some distro .inc file
like there is
conf/distro/include/shr-autorev.inc
conf/distro/include/fso-autorev.inc
etc.

but for rest of world is better to say which SRCREV was tested and
should work for everybody.

> Of course I can add a new SRCREV but it defeats the whole purpose of
> the recipe...

Main purpose of _svn recipes is to allow build project without proper
release or old release etc.. picking random bleeding edge revision and
expect to get building image with it everytime I need to build newer
images is not purpose but insanity :).

Regards,
-- 
Martin 'JaMa' Jansa     jabber: Martin.Jansa at gmail.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://lists.openembedded.org/pipermail/openembedded-devel/attachments/20110722/2ea1d25e/attachment-0002.sig>