[Buildroot] [PATCH 1/4] core/pkg-infra: introduce download features concept

Vincent Fazio vfazio at xes-inc.com
Fri Jan 17 13:56:53 UTC 2020


On 1/17/20 2:17 AM, Thomas Petazzoni wrote:
> On Thu, 16 Jan 2020 23:59:00 +0100
> Arnout Vandecappelle <arnout at mind.be> wrote:
>
>>   It's good to fire it off early (as an RFC) to avoid wasting work if it's going
>> in the wrong direction.
>>
>>   ... which I think this is.
>>
>>   I guess the idea is to generalize the submodules feature so it can be used by
>> several VCS systems, and to combine it with the transient option.
>>
>>   However, I don't think the additional complexity is worth it.
>>
>>   Separate _GIT_SUBMODULES = YES and _DOWNLOAD_TRANSIENT = YES are IMO easier to
>> understand than _DL_FEATURES = submodules transient, because we don't really
>> have the latter pattern anywhere else in Buildroot. In addition, it's hard to
>> remember if it's comma-separated or space-separated.
> I asked the exact same question on IRC before Vincent sent his patch
> series, and both Yann and me suggested that it should be sent with
> actual examples of how the DL_FEATURES mechanism will be used.
>
> The intention of Vincent is not to use this DL_FEATURES variable for
> "transient", but rather to implement download method specific things,
> and the first example was git-lfs support.
>
> transient would remain a separate variable. transient is independent of
> the download method: it only tells the download infrastructure to
> ignore the cache, and always redownlaod from upstream, regardless of
> the download method.
>
> DL_FEATURES however would be to pass backend-specific flags.
>
> That being said, I agree that with just two flags for the moment
> (submodules and git-lfs), it's not clear this is really needed. Also,
> this DL_FEATURES mechanism only allows to pass booleans, which works
> fine for submodules and git-lfs, but what if we want to pass some
> actual value to a download backend (like a timeout value, or some other
> thing).
I'm not sure this is tied down to boolean values (you could certainly 
specify timeout=X) but switches passed directly to the backend 
(git/hg/etc) can be specified via <pkg>_DL_OPTS already, right?
> So maybe with just two booleans for now, we should stick to
> <pkg>_GIT_SUBMODULES = YES and <pkg>_GIT_LFS = YES ?
I'm fine with just having git-lfs use <pkg>_GIT_LFS = YES and this 
series can be voided if git-lfs using this define will be accepted. The 
original git-lfs patch submission was stalled and after talking with 
Yann it seemed to be due to the requirement of adding yet another switch 
to the dl-wrapper that only applied to git, so this was proposed as a 
way to generalize it and get away from updating multiple files 
(pkg-generic, pkg-download, dl-wrapper, and the backend) to add a single 
feature.
> Best regards,
>
> Thomas

-- 
Vincent Fazio
Embedded Software Engineer - Linux
Extreme Engineering Solutions, Inc
http://www.xes-inc.com




More information about the buildroot mailing list