[OE-core] [PATCH] bitbake.conf: add git-lfs to HOSTTOOLS_NONFATAL

[richard.purdie at linuxfoundation.org]

On Wed, 2019-04-17 at 12:35 +0300, Adrian Bunk wrote:
> On Wed, Apr 17, 2019 at 09:46:25AM +0100, 
> richard.purdie at linuxfoundation.org wrote:
> > On Wed, 2019-04-17 at 10:20 +0800, Saini, Naveen Kumar wrote:
> > ...
> > > Should this be added to HOSTTOOLS instead so lfs files are always
> > > fetched for packages that use it?
> > 
> > No, that would mean everyone has to install it.
> > 
> > I'm ok with adding it to HOSTTOOLS_NONFATAL as soon as we have some
> > mechanism in the fetcher or in recipes which detects lfs usage and
> > hard
> > errors if its present but git-lfs is not installed.
> > 
> > That way its deterministic - it builds with git-lfs installed and
> > errors if it is not.
> 
> Is it OK that some packages are not buildable at all on supported
> distributions?

We're talking about a non-core layer and that would be something for
that layer's maintainers to decide.

> Your "everyone has to install it" worry misses the much bigger
> problem 
> that git-lfs is a relatively new tool and not available in
> distributions 
> like Debian 9 or Ubuntu 16.04.

Someone could certainly write a native recipe for lfs and add that as a
dependency which would also solve this problem. I'd be quite happy to
see that and it would neatly solve the problems.

Cheers,

Richard