[PATCH][RFC] bitbake.conf: use empty SRC_URI by default

[PATCH][RFC] bitbake.conf: use empty SRC_URI by default

[Andreas Oberritter]

* Otherwise, recipes that only append to SRC_URI instead of
  overwriting it will get the full path of the recipe written
  into the Source: field of the package.

Signed-off-by: Andreas Oberritter <obi@opendreambox.org>
---
* I haven't tested this yet, I don't know whether there
  are any caveeats. What's the purpose of having ${FILE}
  in there, anyway?

 meta/conf/bitbake.conf |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/meta/conf/bitbake.conf b/meta/conf/bitbake.conf
index c8d9a31..216da75 100644
--- a/meta/conf/bitbake.conf
+++ b/meta/conf/bitbake.conf
@@ -577,7 +577,7 @@ SRCREV ??= "INVALID"
 AUTOREV = "${@bb.fetch2.get_autorev(d)}"
 SRCPV = "${@bb.fetch2.get_srcrev(d)}"
 
-SRC_URI = "file://${FILE}"
+SRC_URI = ""
 
 # Use pseudo as the fakeroot implementation
 PSEUDO_LOCALSTATEDIR ?= "${WORKDIR}/pseudo/"
-- 
1.7.5.4

Re: [PATCH][RFC] bitbake.conf: use empty SRC_URI by default

[Andreas Oberritter]

On 08.03.2012 03:51, Andreas Oberritter wrote:
> * Otherwise, recipes that only append to SRC_URI instead of
>   overwriting it will get the full path of the recipe written
>   into the Source: field of the package.
> 
> Signed-off-by: Andreas Oberritter <obi@opendreambox.org>
> ---
> * I haven't tested this yet, I don't know whether there
>   are any caveeats. What's the purpose of having ${FILE}
>   in there, anyway?

Richard, did you commit this by accident? In the meantime I've started a
clean build with this patch applied and noticed eglibc-locale failing,
because opkg-build complains about a missing "Source" field.

Regards,
Andreas

Re: [PATCH][RFC] bitbake.conf: use empty SRC_URI by default

[Saul Wold]

On 03/08/2012 01:38 PM, Andreas Oberritter wrote:
> On 08.03.2012 03:51, Andreas Oberritter wrote:
>> * Otherwise, recipes that only append to SRC_URI instead of
>>    overwriting it will get the full path of the recipe written
>>    into the Source: field of the package.
>>
>> Signed-off-by: Andreas Oberritter<obi@opendreambox.org>
>> ---
>> * I haven't tested this yet, I don't know whether there
>>    are any caveeats. What's the purpose of having ${FILE}
>>    in there, anyway?
>
> Richard, did you commit this by accident? In the meantime I've started a
> clean build with this patch applied and noticed eglibc-locale failing,
> because opkg-build complains about a missing "Source" field.
>
Agreed, this seems to have been committed by accident, we see the same 
failures on the Autobuilder.

Sau!

> Regards,
> Andreas
>
>
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
>

Re: [PATCH][RFC] bitbake.conf: use empty SRC_URI by default

[Richard Purdie]

On Fri, 2012-03-09 at 00:07 -0800, Saul Wold wrote:
> On 03/08/2012 01:38 PM, Andreas Oberritter wrote:
> > On 08.03.2012 03:51, Andreas Oberritter wrote:
> >> * Otherwise, recipes that only append to SRC_URI instead of
> >>    overwriting it will get the full path of the recipe written
> >>    into the Source: field of the package.
> >>
> >> Signed-off-by: Andreas Oberritter<obi@opendreambox.org>
> >> ---
> >> * I haven't tested this yet, I don't know whether there
> >>    are any caveeats. What's the purpose of having ${FILE}
> >>    in there, anyway?
> >
> > Richard, did you commit this by accident? In the meantime I've started a
> > clean build with this patch applied and noticed eglibc-locale failing,
> > because opkg-build complains about a missing "Source" field.
> >
> Agreed, this seems to have been committed by accident, we see the same 
> failures on the Autobuilder.

I meant to do some local testing but it ended up getting merged, sorry
about that. Since there weren't too many issues on the autobuilder, I
pushed a follow up which sets Source to be "None" rather than unset and
the last set of builds are looking pretty green.

Cheers,

Richard