all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* [bug#61864] [PATCH] scripts: expose 'allow-collisions' for 'guix install'
@ 2023-02-28  6:28 Andy Tai
  2023-03-02 20:02 ` Leo Famulari
  2023-03-03  6:07 ` [bug#61864] [PATCH v2] scripts: Expose " Andy Tai
  0 siblings, 2 replies; 5+ messages in thread
From: Andy Tai @ 2023-02-28  6:28 UTC (permalink / raw)
  To: 61864; +Cc: Andy Tai

* guix/scripts/install.scm: 'allow-collisions' option of 'package'
exposed for 'install' alias
---
 guix/scripts/install.scm | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/guix/scripts/install.scm b/guix/scripts/install.scm
index 63e625f266..4ab64b5b54 100644
--- a/guix/scripts/install.scm
+++ b/guix/scripts/install.scm
@@ -34,6 +34,8 @@ (define (show-help)
   (display (G_ "
   -p, --profile=PROFILE  use PROFILE instead of the user's default profile"))
   ;; '--bootstrap' not shown here.
+  (display (G_ "
+      --allow-collisions do not treat collisions in the profile as an error"))
   (display (G_ "
   -v, --verbosity=LEVEL  use the given verbosity LEVEL"))
   (newline)
@@ -61,7 +63,8 @@ (define %options
          ;; Preserve some of the 'guix package' options.
          (append (filter (lambda (option)
                            (any (cut member <> (option-names option))
-                                '("profile" "dry-run" "verbosity" "bootstrap")))
+                                '("profile" "dry-run" "verbosity" "bootstrap"
+                                  "allow-collisions")))
                          %package-options)
 
                  %transformation-options

base-commit: cb0d8100b288b5b0d130820207db17764b7d2140
prerequisite-patch-id: 2ab0872d80951bb6f8df7bc8c4211f008a5cc6b8
prerequisite-patch-id: 1fa93a6e05a30a2a3a92d11ff1d7bdf7912e2724
prerequisite-patch-id: 8027f51c310a66f545421b43e1283ed516d41afc
-- 
2.39.1





^ permalink raw reply related	[flat|nested] 5+ messages in thread

* [bug#61864] [PATCH] scripts: expose 'allow-collisions' for 'guix install'
  2023-02-28  6:28 [bug#61864] [PATCH] scripts: expose 'allow-collisions' for 'guix install' Andy Tai
@ 2023-03-02 20:02 ` Leo Famulari
  2023-03-02 20:38   ` Andy Tai
  2023-03-03  6:07 ` [bug#61864] [PATCH v2] scripts: Expose " Andy Tai
  1 sibling, 1 reply; 5+ messages in thread
From: Leo Famulari @ 2023-03-02 20:02 UTC (permalink / raw)
  To: Andy Tai; +Cc: 61864

On Mon, Feb 27, 2023 at 10:28:46PM -0800, Andy Tai wrote:
> * guix/scripts/install.scm: 'allow-collisions' option of 'package'
> exposed for 'install' alias

I'm curious, what's the rationale for this?




^ permalink raw reply	[flat|nested] 5+ messages in thread

* [bug#61864] [PATCH] scripts: expose 'allow-collisions' for 'guix install'
  2023-03-02 20:02 ` Leo Famulari
@ 2023-03-02 20:38   ` Andy Tai
  2023-04-01 16:54     ` bug#61864: " Leo Famulari
  0 siblings, 1 reply; 5+ messages in thread
From: Andy Tai @ 2023-03-02 20:38 UTC (permalink / raw)
  To: Leo Famulari; +Cc: 61864

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

I run into collisions in profile from time to time and I can resolve it via

guix package --allow-collosions -i (or -u)

so it is useful to make that available in the two "alias" commands as well

On Thu, Mar 2, 2023 at 12:02 PM Leo Famulari <leo@famulari.name> wrote:

> On Mon, Feb 27, 2023 at 10:28:46PM -0800, Andy Tai wrote:
> > * guix/scripts/install.scm: 'allow-collisions' option of 'package'
> > exposed for 'install' alias
>
> I'm curious, what's the rationale for this?
>


-- 
Andy Tai, atai@atai.org, Skype: licheng.tai, Line: andy_tai, WeChat:
andytai1010
Year 2023 民國112年
自動的精神力是信仰與覺悟
自動的行為力是勞動與技能

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

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [bug#61864] [PATCH v2] scripts: Expose 'allow-collisions' for 'guix install'
  2023-02-28  6:28 [bug#61864] [PATCH] scripts: expose 'allow-collisions' for 'guix install' Andy Tai
  2023-03-02 20:02 ` Leo Famulari
@ 2023-03-03  6:07 ` Andy Tai
  1 sibling, 0 replies; 5+ messages in thread
From: Andy Tai @ 2023-03-03  6:07 UTC (permalink / raw)
  To: 61864; +Cc: Andy Tai

* guix/scripts/install.scm: Expose 'allow-collisions' option
of 'package' for 'install' alias
---
 guix/scripts/install.scm | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/guix/scripts/install.scm b/guix/scripts/install.scm
index 63e625f266..4ab64b5b54 100644
--- a/guix/scripts/install.scm
+++ b/guix/scripts/install.scm
@@ -34,6 +34,8 @@ (define (show-help)
   (display (G_ "
   -p, --profile=PROFILE  use PROFILE instead of the user's default profile"))
   ;; '--bootstrap' not shown here.
+  (display (G_ "
+      --allow-collisions do not treat collisions in the profile as an error"))
   (display (G_ "
   -v, --verbosity=LEVEL  use the given verbosity LEVEL"))
   (newline)
@@ -61,7 +63,8 @@ (define %options
          ;; Preserve some of the 'guix package' options.
          (append (filter (lambda (option)
                            (any (cut member <> (option-names option))
-                                '("profile" "dry-run" "verbosity" "bootstrap")))
+                                '("profile" "dry-run" "verbosity" "bootstrap"
+                                  "allow-collisions")))
                          %package-options)
 
                  %transformation-options

base-commit: ea870a70e93e7ea93e35fa8e0afc3cfdb1b66ba2
-- 
2.39.1





^ permalink raw reply related	[flat|nested] 5+ messages in thread

* bug#61864: [PATCH] scripts: expose 'allow-collisions' for 'guix install'
  2023-03-02 20:38   ` Andy Tai
@ 2023-04-01 16:54     ` Leo Famulari
  0 siblings, 0 replies; 5+ messages in thread
From: Leo Famulari @ 2023-04-01 16:54 UTC (permalink / raw)
  To: Andy Tai; +Cc: 61864-done

On Thu, Mar 02, 2023 at 12:38:45PM -0800, Andy Tai wrote:
> I run into collisions in profile from time to time and I can resolve it via
> 
> guix package --allow-collosions -i (or -u)
> 
> so it is useful to make that available in the two "alias" commands as well

I see, that makes sense. Sorry I'm slow to reply, I thought about it for
a while.

Allowing profile collisions is a risky action that can cause subtle and
hard to diagnose bugs, and it often is done without understanding the
consequences. In my experience, collisions are often allowed by people
who don't understand the negative effects of collisions, or how profiles
are constructed.

When reported, the consequent problems can take up a lot of time for the
people trying to help.

Because of those reasons, I prefer to only expose this option in `guix
package`, which is a more advanced interface. My hope is that
segregating the option like this will discourage use of
'--allow-collisions' and prevent an increase in bug reports of the "user
error" type.

If you disagree, please feel free to reopen the bug and optionally CC
some other reviewers, or bring it up on IRC.




^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2023-04-01 16:55 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-02-28  6:28 [bug#61864] [PATCH] scripts: expose 'allow-collisions' for 'guix install' Andy Tai
2023-03-02 20:02 ` Leo Famulari
2023-03-02 20:38   ` Andy Tai
2023-04-01 16:54     ` bug#61864: " Leo Famulari
2023-03-03  6:07 ` [bug#61864] [PATCH v2] scripts: Expose " Andy Tai

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.