[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] bootstrap: Provide a hook for submodule _SRCDIR setup
From: |
Arsen Arsenović |
Subject: |
Re: [PATCH] bootstrap: Provide a hook for submodule _SRCDIR setup |
Date: |
Sun, 27 Nov 2022 22:06:26 +0100 |
Evening,
Bruno Haible <bruno@clisp.org> writes:
>> * build-aux/bootstrap: Regenerate.
>
> Thanks; I pushed that part in your name.
Thanks!
> Yes, all 4 of bootstrap_option_hook, bootstrap_print_option_usage_hook,
> bootstrap_post_pull_hook, bootstrap_post_import_hook then deal with
> JITTER_SRCDIR in one form or the other.
>
> If it works with this reorganization of the code, I would prefer not to add
> a bootstrap_srcdirs_hook.
Hmm, now that I think about it, --jitter-srcdir doesn't really even make
sense in autopull? My thinking is that autopull is pretty fixed on
operate on submodules AFAIU (seemingly backed up by --jitter-srcdir
previously being ignored for the pull stage anway), so just removing all
of that logic and only leaving the defaulting logic in the post_import
hook should work.
Since the autogen case is no longer concerned with pulling at all, or
$use_git, just letting it default to "jitter" should work for all cases
(those that want to override it can anyway), as long as the option hook
exports JITTER_SRCDIR in bootstrap.
In that case, I'll just update to today's revision of gnulib and see
about upstreaming that in poke.
Thanks, and sorry about the fuss ;)
Have a great night.
--
Arsen Arsenović
signature.asc
Description: PGP signature