bug-gnulib
[Top][All Lists]
Advanced

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

Re: [PATCH] gnulib-tool.py: Fix write failure due to bad sourcebase.


From: Bruno Haible
Subject: Re: [PATCH] gnulib-tool.py: Fix write failure due to bad sourcebase.
Date: Thu, 14 Mar 2024 15:38:44 +0100

Hi Collin,

> This patch should fix gnulib-tool.py when running
> 'gnulib-tool.py --add-import' in OATH Toolkit.
> 
> It appears that gnulib-tool.py was not picking up the m4 directories
> correctly. This means that when the GLImport object was created it
> could not find the gnulib-cache.m4 file containing the sourcebase.

This patch looks better than the previous one, that added a
setSourceBase('lib') invocation. Applied. Thanks!

> This appeared as a failure to write since gnulib-tool.py was trying to
> write to '$sourcebase/dummy.c' where $sourcebase was an empty string.
> 
> The other change prevents calling 'obj.split()' where obj is not a
> string:
> 
>          sequence = [ True if value == 'true' else value
>                       for value in sequence ]
>          sequence = [ value.strip()
> +                     if type(value) is str else value
>                       for value in sequence ]

Is converting 'true' to True a good thing to do here anyway?
Which invocations of the 'cleaner' function need this?

Bruno






reply via email to

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