[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Refactoring rewrite_filename functions
From: |
Bruno Haible |
Subject: |
Re: Refactoring rewrite_filename functions |
Date: |
Wed, 17 Apr 2024 03:35:34 +0200 |
Hi Collin,
> I'm thinking of making this function accept a single filename instead
> of a list
Makes sense, as previously discussed.
> and then moving it to GLConfig.
This does not make sense. GLConfig is meant to hold settings and configuration,
nothing more.
I see the home of this function more in GLFileSystem.py. Maybe in class
GLFileAssistant, maybe in a new class in that same file.
> Also, there is a similar section of code to this new function in
> main.py line 1313 under "mode == 'copy-file'", but it is missing the
> 'tests=lib/' replacement. Would there be a way to simplify that too?
> The use of 'tests=lib/' is escaping my mind at the moment...
'tests=lib/' should not occur in the scope of mode == 'copy-file'.
But this string is an indicator inside gnulib-tool; users of gnulib-tool
should not be allowed to pass it.
Bruno
- gnulib-tool.py: Remove a redundant function., Collin Funk, 2024/04/14
- Re: gnulib-tool.py: Remove a redundant function., Collin Funk, 2024/04/14
- Re: gnulib-tool.py: Remove a redundant function., Bruno Haible, 2024/04/15
- Re: gnulib-tool.py: Remove a redundant function., Collin Funk, 2024/04/15
- Re: gnulib-tool.py: Remove a redundant function., Bruno Haible, 2024/04/16
- Re: gnulib-tool.py: Remove a redundant function., Collin Funk, 2024/04/16
- Refactoring rewrite_filename functions, Collin Funk, 2024/04/16
- Re: Refactoring rewrite_filename functions,
Bruno Haible <=
- Re: Refactoring rewrite_filename functions, Collin Funk, 2024/04/16
- Re: Refactoring rewrite_filename functions, Bruno Haible, 2024/04/17
Re: gnulib-tool.py: Remove a redundant function., Bruno Haible, 2024/04/15