all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Giacomo Leidi <leidigiacomo@outlook.com>
To: "Ludovic Courtès" <ludo@gnu.org>
Cc: 68857@debbugs.gnu.org
Subject: [bug#68857] gnu: home: dotfiles: Avoid creating extra directory in $HOME.
Date: Tue, 27 Feb 2024 12:30:30 +0100	[thread overview]
Message-ID: <AM8P189MB13961838B34C0483F3B2E9E9D0592@AM8P189MB1396.EURP189.PROD.OUTLOOK.COM> (raw)
In-Reply-To: <87bk82i53w.fsf_-_@gnu.org>

[-- Attachment #1: Type: text/plain, Size: 2598 bytes --]

Hi Ludo' ,

First of all thank you for putting the time for working on this, I hope 
we are close to a solution :)

On 2/27/24 11:17, Ludovic Courtès wrote:
> I think we’ll only want to support two layouts: Stow and “plain”; we
> should avoid overengineering that.  That’s why a simple (layout 'stow)
> field seems good enough for me.
>
> WDYT?

I think it is important, if the effort is feasible, to not leave anyone 
behind (in terms of what the features of this service are) . Having a 
single field without further changes will introduce some ambiguity imo. 
We have these requirements in my understanding:

 1. The service should support Stow's users workflows. This is a hard
    requirement in my opinion. Hence we need a way to select a subset
    the applications directories (the applications field of
    home-dotfiles-stow-directory in v2 of the patch).
 2. The service should support Stow's users workflows. This is also a
    hard requirement. For this we just need the path of the dotfiles
    directory.
 3. The service should support multiple dotfiles directories. This is
    not a hard requirement i believe, but we currently have this feature
    (the directories field of home-dotfiles-configuration is a list of
    strings not a string).

Introducing a single layout field makes it impossible to unambiguously 
implement requirement 1. If the user has more than directory it really 
makes no sense to select the same subset of applications for each one of 
them. This is to say that I believe it makes little sense to have 
multiple directories if the layout and applications information are not 
linked somehow with each directory. I wouldn't call this 
overengineering, just implementing the features users need from this 
service. I hope you agree.

Also since I first sent this service around Jan 2023 (when it was still 
the home-stow-migration-service), and it has been broken on master for 
some time I'd like to provide a fix for this situation as soon as 
possible (clearly the definition of brokennes and what changes to the 
API would be breaking user configs depends on the requirements one 
intends for the home-dotfiles-service-type, especially since the changes 
that broke master were introduced without consensus).

I'm changing the code to: use the layout field, support Stow's users 
workflow adding a new optional field called packages, which makes sense 
only if the layout is 'stow, and I'm making the directories field a 
string instead of a list of strings. Please let me know your thoughts on 
this v3.


Thank your for your work,


giacomo

[-- Attachment #2: Type: text/html, Size: 3214 bytes --]

  reply	other threads:[~2024-02-27 13:44 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-01-31 22:15 [bug#68857] gnu: home: dotfiles: Avoid creating extra directory in $HOME paul via Guix-patches via
2024-01-31 22:17 ` [bug#68857] [PATCH] " Giacomo Leidi via Guix-patches via
2024-02-07  0:54 ` paul via Guix-patches via
2024-02-07  8:05   ` Janneke Nieuwenhuizen
2024-02-07  8:37     ` Janneke Nieuwenhuizen
2024-02-16 17:16   ` paul via Guix-patches via
2024-02-16 18:57     ` Sergey Trofimov
2024-02-16 17:17 ` [bug#68857] [PATCH v2] gnu: home: dotfiles: Properly support both plain and Stow directory layouts Giacomo Leidi via Guix-patches via
2024-02-20  9:37   ` [bug#68857] gnu: home: dotfiles: Avoid creating extra directory in $HOME Ludovic Courtès
2024-02-20 18:38     ` paul via Guix-patches via
2024-02-27 10:17       ` Ludovic Courtès
2024-02-27 11:30         ` Giacomo Leidi [this message]
2024-02-27 11:32         ` paul via Guix-patches via
2024-02-27 12:35 ` [bug#68857] [PATCH v3] gnu: home: dotfiles: Properly support both plain and Stow directory layouts Giacomo Leidi via Guix-patches via
2024-03-04 15:46   ` [bug#68857] gnu: home: dotfiles: Avoid creating extra directory in $HOME Ludovic Courtès
2024-03-06 20:51     ` paul via Guix-patches via
2024-03-06 20:52 ` [bug#68857] [PATCH v4] gnu: home: dotfiles: Properly support both plain and Stow directory layouts Giacomo Leidi via Guix-patches via
2024-03-06 22:15   ` bug#68848: bug#68857: gnu: home: dotfiles: Avoid creating extra directory in $HOME Ludovic Courtès

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=AM8P189MB13961838B34C0483F3B2E9E9D0592@AM8P189MB1396.EURP189.PROD.OUTLOOK.COM \
    --to=leidigiacomo@outlook.com \
    --cc=68857@debbugs.gnu.org \
    --cc=ludo@gnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/guix.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.