[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[bug#36404] [PATCH v4 2/4] gnu: Add machine type for deployment specific
From: |
Jakob L. Kreuze |
Subject: |
[bug#36404] [PATCH v4 2/4] gnu: Add machine type for deployment specifications. |
Date: |
Thu, 04 Jul 2019 11:59:49 -0400 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/26.2 (gnu/linux) |
Hi Ludovic,
Ludovic Courtès <address@hidden> writes:
> To me the end goal was to move these “effectful” bits into a script,
> such that both ‘guix system reconfigure’ and ‘guix deploy’ would only
> have to run that script, locally or remotely. That would avoid
> duplicating these somewhat tricky procedures.
Ah, that's starting to ring a bell now. I believe you mentioned that
when 'guix deploy' was initially being proposed, but at the time I
didn't quite register that we'd be extracting the behavior in that way.
> Now, perhaps we can start like this, and leave factorization for
> later? I just want to make sure we don’t forget about that and let it
> evolve into something we have a hard time maintaining.
>
> WDYT?
I agree. I'm getting the impression that people don't want this to sit
in review limbo for too long, and in terms of "commit history hygiene,"
I think it would be better to recognize refactoring out the common
behavior as a distinct change.
Thanks!
Regards,
Jakob
signature.asc
Description: PGP signature
- [bug#36404] [PATCH v4 2/4] gnu: Add machine type for deployment specifications., (continued)
- [bug#36404] [PATCH v4 2/4] gnu: Add machine type for deployment specifications., Jakob L. Kreuze, 2019/07/02
- [bug#36404] [PATCH v4 3/4] Add 'guix deploy'., Jakob L. Kreuze, 2019/07/02
- [bug#36404] [PATCH v4 4/4] doc: Add section for 'guix deploy'., Jakob L. Kreuze, 2019/07/02
- [bug#36404] [PATCH v4 4/4] doc: Add section for 'guix deploy'., Christopher Lemmer Webber, 2019/07/03
- [bug#36404] [PATCH v4 4/4] doc: Add section for 'guix deploy'., Ludovic Courtès, 2019/07/04
- [bug#36404] [PATCH v4 4/4] doc: Add section for 'guix deploy'., Thompson, David, 2019/07/04
- [bug#36404] [PATCH v4 4/4] doc: Add section for 'guix deploy'., Ludovic Courtès, 2019/07/05
- [bug#36404] [PATCH v4 3/4] Add 'guix deploy'., Thompson, David, 2019/07/04
- [bug#36404] [PATCH v4 3/4] Add 'guix deploy'., Ludovic Courtès, 2019/07/05
- [bug#36404] [PATCH v4 2/4] gnu: Add machine type for deployment specifications., Ludovic Courtès, 2019/07/04
- [bug#36404] [PATCH v4 2/4] gnu: Add machine type for deployment specifications.,
Jakob L. Kreuze <=
- [bug#36404] [PATCH v4 2/4] gnu: Add machine type for deployment specifications., Thompson, David, 2019/07/04
- [bug#36404] [PATCH v4 2/4] gnu: Add machine type for deployment specifications., Ludovic Courtès, 2019/07/05
- [bug#36404] [PATCH v4 2/4] gnu: Add machine type for deployment specifications., Ludovic Courtès, 2019/07/05
- [bug#36404] [PATCH v5 0/4] Add 'guix deploy'., Jakob L. Kreuze, 2019/07/05
- [bug#36404] [PATCH v5 1/4] ssh: Add 'identity' keyword to 'open-ssh-session'., Jakob L. Kreuze, 2019/07/05
- [bug#36404] [PATCH v5 2/4] gnu: Add machine type for deployment specifications., Jakob L. Kreuze, 2019/07/05
- [bug#36404] [PATCH v5 3/4] Add 'guix deploy'., Jakob L. Kreuze, 2019/07/05
- [bug#36404] [PATCH v5 4/4] doc: Add section for 'guix deploy'., Jakob L. Kreuze, 2019/07/05
- bug#36404: [PATCH v5 4/4] doc: Add section for 'guix deploy'., Christopher Lemmer Webber, 2019/07/05
- [bug#36404] [PATCH v5 4/4] doc: Add section for 'guix deploy'., Jakob L. Kreuze, 2019/07/05