unofficial mirror of guix-devel@gnu.org 
 help / color / mirror / Atom feed
* Messed up with (guix ssh)
@ 2020-09-03 10:02 Ludovic Courtès
  2020-09-03 10:59 ` Jan Nieuwenhuizen
  0 siblings, 1 reply; 3+ messages in thread
From: Ludovic Courtès @ 2020-09-03 10:02 UTC (permalink / raw)
  To: Guix-devel

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

Hi Guix,

I recently added a progress bar for ‘guix copy’:

  https://git.savannah.gnu.org/cgit/guix.git/commit/?id=b03267df6d5ec44e9617b6aab0df14a2e79f822e

However that commit also includes a debugging leftover that breaks
‘send-files’ (in some but not all cases).  Can you spot it?


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: Type: text/x-patch, Size: 171 bytes --]

-         (missing (inferior-remote-eval
+         (missing (take files 20) #;(inferior-remote-eval
                    `(begin
                       (use-modules (guix)

[-- Attachment #3: Type: text/plain, Size: 443 bytes --]


I noticed it as I experienced the first problems (offloading would
sometimes fail while sending files) and fixed it:

  https://git.savannah.gnu.org/cgit/guix.git/commit/?id=de83660dd386b73957fe4ad2dc08ce5b8e1f0e7f

I also add to re-update the ‘guix’ package.

So I wanted to give a heads-up and apologize.  It got me thinking; this
patch hadn’t gone through guix-patches, which could have prevented this,
maybe.

Ludo’.

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

* Re: Messed up with (guix ssh)
  2020-09-03 10:02 Messed up with (guix ssh) Ludovic Courtès
@ 2020-09-03 10:59 ` Jan Nieuwenhuizen
  2020-09-07  9:48   ` Ludovic Courtès
  0 siblings, 1 reply; 3+ messages in thread
From: Jan Nieuwenhuizen @ 2020-09-03 10:59 UTC (permalink / raw)
  To: Ludovic Courtès; +Cc: Guix-devel

Ludovic Courtès writes:

Hello!

> So I wanted to give a heads-up and apologize.  It got me thinking; this
> patch hadn’t gone through guix-patches, which could have prevented this,
> maybe.

Thanks for the heads-up and quick fix.  I'm often amazed how well
guix-patches work, notably also for catching/preventing silly mistakes.

I think it's a trade-off and not all black and white, to get a feel
for where the boundary is we have to cross it now and then.  Dunno,
very happy with what you're doing :-)

Janneke

-- 
Jan Nieuwenhuizen <janneke@gnu.org> | GNU LilyPond http://lilypond.org
Freelance IT http://JoyofSource.com | Avatar® http://AvatarAcademy.com


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

* Re: Messed up with (guix ssh)
  2020-09-03 10:59 ` Jan Nieuwenhuizen
@ 2020-09-07  9:48   ` Ludovic Courtès
  0 siblings, 0 replies; 3+ messages in thread
From: Ludovic Courtès @ 2020-09-07  9:48 UTC (permalink / raw)
  To: Jan Nieuwenhuizen; +Cc: Guix-devel

Hi!

Jan Nieuwenhuizen <janneke@gnu.org> skribis:

>> So I wanted to give a heads-up and apologize.  It got me thinking; this
>> patch hadn’t gone through guix-patches, which could have prevented this,
>> maybe.
>
> Thanks for the heads-up and quick fix.  I'm often amazed how well
> guix-patches work, notably also for catching/preventing silly mistakes.
>
> I think it's a trade-off and not all black and white, to get a feel
> for where the boundary is we have to cross it now and then.  Dunno,
> very happy with what you're doing :-)

Heh, thanks for the kind words.

It reminded me of things like “goto fail;” though, so I thought like
giving a heads-up was the least I could do.  A similar mistake in
git-authenticate.scm or substitute.scm would have a very different
effect…

Ludo’.


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

end of thread, other threads:[~2020-09-07  9:49 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-09-03 10:02 Messed up with (guix ssh) Ludovic Courtès
2020-09-03 10:59 ` Jan Nieuwenhuizen
2020-09-07  9:48   ` Ludovic Courtès

unofficial mirror of guix-devel@gnu.org 

This inbox may be cloned and mirrored by anyone:

	git clone --mirror https://yhetil.org/guix-devel/0 guix-devel/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 guix-devel guix-devel/ https://yhetil.org/guix-devel \
		guix-devel@gnu.org
	public-inbox-index guix-devel

Example config snippet for mirrors.
Newsgroups are available over NNTP:
	nntp://news.yhetil.org/yhetil.gnu.guix.devel
	nntp://news.gmane.io/gmane.comp.gnu.guix.devel


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git