[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[bug#44130] [PATCH] Add a recursive version of mkdir-p
From: |
Ludovic Courtès |
Subject: |
[bug#44130] [PATCH] Add a recursive version of mkdir-p |
Date: |
Fri, 23 Oct 2020 15:12:44 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/27.1 (gnu/linux) |
Hi!
divoplade <d@divoplade.fr> skribis:
> I need this mkdir-p function in any non-trivial program I write. I had
> 3 bad choices:
>
> 1. Lobby guile to provide this function out of the box (this will take
> time);
> 2. Copy that of guix, or gash, or any other: this does not seem
> acceptable to me, because this function will surely evolve (for
> instance, if guile gets suport for mingw and we start running guile
> programs on windows) and I don't want to update more than one version
> of this function;
> 3. Depend on guix, gash or another package: this would be too large a
> dependency for my programs.
>
> So I wrote it in its own package and I intend to depend on it for my
> other projects. It would be best if you could accept this package in
> guix proper.
>
> What do you think?
I have nothing against adding this package to Guix, but…
Do you realize that the package definition is longer than the ‘mkdir-p’
procedure itself? :-)
I think npm packages are too fine-grain; I don’t think this is the
approach to follow for Guile.
It’s likely that packages that need ‘mkdir-p’ also need other high-level
file system operations that Gash (say) provides. In that case, I’d
encourage people to depend on Gash. If Gash is too big a dependency for
the project, including its own copy of this 24-line procedure is
probably acceptable.
All that said, I do think that Guile itself should eventually include
some of the utilities found in (guix build utils) or Gash. For
instance, it recently got a new ‘pipeline’ procedure, which comes from
Gash, and I think it’s a great addition. This is the way to go in the
longer term.
Thoughts?
Ludo’.
[bug#44130] [PATCH] Add a recursive version of mkdir-p, zimoun, 2020/10/23
[bug#44130] [PATCH] Add a recursive version of mkdir-p, Leo Prikler, 2020/10/23