On Tue, Apr 29, 2025 at 11:08 AM David E. Wheeler <da...@justatheory.com> wrote:
> Right. My point is a minor one, but I thin you can use an if/ else there:
>
> ```c
> if (strcmp(piece, "$system") == 0) {
> /* Substitute the path macro if needed */
> mangled = substitute_path_macro(piece, "$system", system_dir);
> } else {
> /*
> * Append "extension" suffix in case is a custom extension
> * control path.
> */
> mangled = psprintf("%s/extension", mangled);
> }
> ```
>The substitute_path_macro() already handles the if/else on "piece" but I think that this if/else version looks nicer. Fixed. I've also included some documentation changes for this v5 version to remove the "extension" from the examples and also mention the scenario when using the "directory" on the .control file. -- Matheus Alcantara
v5-0001-Make-directory-work-with-extension-control-path.patch
Description: Binary data
