bug-gnulib
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: pygnulib: progress and some questions (func_emit_autoconf_snippets)


From: Bruno Haible
Subject: Re: pygnulib: progress and some questions (func_emit_autoconf_snippets)
Date: Fri, 29 Dec 2017 00:32:23 +0100
User-agent: KMail/5.1.3 (Linux/4.4.0-101-generic; KDE/5.18.0; x86_64; ; )

Hi Dmitry,

> > 1. At line 4260, what's the place where $modules variable gets assigned?
> 
> Indeed the comments for function func_emit_autoconf_snippets are misleading:
> This functions receives two module lists:
>   $modules (implicit),
>   $1 (explicit: first argument).
> 
> Where does $modules come from?
>   - For the invocation in func_import, line 5640 and 5659
>     it is set at line 5298. But I'm not sure this was intended this way, maybe
>     another list should be used instead of "$main_modules"?
>   - For the invocation in func_create_testdir, line 6288 and 6290 and 6402 
> and 6404 and 6417
>     it does not appear to be set in a consistent and correct way.
>     (It is set differently if $single_configure and ! $single_configure.)
>     This is probably a bug.
> The set of modules that _ought_ to be used here is the set of modules
> being used for the current configure file. That's my current understanding.
> But I'd really need to look at a couple of test cases in order to
> firmly answer the question.

I've understood it now: the code in func_create_testdir was not really a bug,
because --conditional-dependencies was not allowed in this case anyway.

Here's the patch that implements --conditional-dependencies also with
--with-tests.

Bruno

Attachment: 0001-gnulib-tool-Make-conditional-dependencies-work-bette.patch
Description: Text Data


reply via email to

[Prev in Thread] Current Thread [Next in Thread]