[Buildroot] [v4 01/13] core/pkg-download: change all helpers to use common options
Peter Korsgaard
peter at korsgaard.com
Mon Apr 2 10:26:25 UTC 2018
>>>>> "Maxime" == Maxime Hadjinlian <maxime.hadjinlian at gmail.com> writes:
> From: "Yann E. MORIN" <yann.morin.1998 at free.fr>
> Currently all download helpers accepts the local output file, the remote
> locations, the changesets and so on... as positional arguments.
> This was well and nice when that's was all we needed.
> But then we added an option to quiesce their verbosity, and that was
> shoehorned with a trivial getopts, still keeping all the existing
> positional arguments as... positional arguments.
> Adding yet more options while keeping positional arguments will not be
> very easy, even if we do not envision any new option in the foreseeable
> future (but 640K ought to be enough for everyone, remember? ;-) ).
> Change all helpers to accept a set of generic options (-q for quiet and
> -o for the output file) as well as helper-specific options (like -r for
> the repository, -c for a changeset...).
> Maxime:
> Changed -R to -r for recurse (only for the git backend)
> Changed -r to -u for URI (for all backend)
> Change -R to -c for cset (for CVS and SVN backend)
> Add the export of the BR_BACKEND_DL_GETOPTS so all the backend wrapper
> can use the same option easily
> Now all the backends use the same common options.
> Signed-off-by: "Yann E. MORIN" <yann.morin.1998 at free.fr>
> Signed-off-by: Maxime Hadjinlian <maxime.hadjinlian at gmail.com>
> Cc: Thomas Petazzoni <thomas.petazzoni at free-electrons.com>
> ---
> v1 -> v2:
> - Fix comment in bzr backend
> - Remove 'R' from BR_BACKEND_DL_GETOPTS
> - check-hash shouldn't have had its options changed since it's not a
> download backend (Arnout)
Committed, thanks.
--
Bye, Peter Korsgaard
More information about the buildroot
mailing list