[OE-core] [PATCH v2 1/1] buildtools-tarball: restore missing git tools

Paul Eggleton paul.eggleton at linux.intel.com
Sat Dec 6 17:08:47 UTC 2014


On Saturday 06 December 2014 11:01:49 Otavio Salvador wrote:
> On Fri, Dec 5, 2014 at 11:39 PM, Paul Eggleton
> 
> <paul.eggleton at linux.intel.com> wrote:
> > On Friday 05 December 2014 16:30:44 Otavio Salvador wrote:
> >> On Fri, Dec 5, 2014 at 4:09 PM, Paul Eggleton
> >> 
> >> <paul.eggleton at linux.intel.com> wrote:
> >> > Since the split out of git-perltools, some git tools (such as "git am",
> >> > "git send-email" and "git-submodule") have no longer been part of the
> >> > buildtools. We need these, so add them back in.
> >> > 
> >> > However, adding git-perltools to buildtools triggers perl itself being
> >> > brought into buildtools as well, and we don't want that; but we also
> >> > don't want to have to hack the git recipe or indeed anything else that
> >> > starts depending on perl. Thus, add a dummy package which gets
> >> > installed
> >> > in its place, in a separate package architecture that is only enabled
> >> > for buildtools to ensure it doesn't start appearing in place of
> >> > nativesdk-perl anywhere else.
> >> > 
> >> > Fixes [YOCTO #7033].
> >> > 
> >> > Signed-off-by: Paul Eggleton <paul.eggleton at linux.intel.com>
> >> 
> >> This dummy thing looks like a hack to me :-(
> > 
> > Perhaps - I'm all ears for an alternative solution, but it absolutely has
> > to be fixed, and soon. Shipping an incomplete git (or incomplete perl) in
> > buildtools basically defeats a major part of having the thing in the
> > first place.
> 
> We should split out the tools we really want (so we skip the perl
> runtime dependency) 

This is exactly what caused this problem in the first place. We cannot do 
without the perl-dependent pieces of git.

> or include perl. Use a dummy package for it is
> wrong in my opinion. You cannot guarantee host perl will work as
> expected for our git release.

It hasn't presented a problem up to now, and until it does I'd rather not ship 
perl in buildtools. CentOS 6 is the oldest distro we support, and the latest 
version of git works perfectly fine with the version of perl that's provided by 
the distro there. On the other hand, adding perl means we need to add a bunch 
of additional perl modules as well, and I really don't think we want to do 
that because it just increases the compexity and size of the buildtools and we 
don't actually need it anyway.

I have actually thought about this.

Cheers,
Paul

-- 

Paul Eggleton
Intel Open Source Technology Centre



More information about the Openembedded-core mailing list