* [bug#54000] [PATCH 0/2] Not showing upgraded/added packages in 'guix pull'
@ 2022-02-14 16:38 Ludovic Courtès
2022-02-14 16:41 ` [bug#54000] [PATCH 1/2] pull: '--news' no longer shows package lists Ludovic Courtès
2022-02-15 13:55 ` [bug#54000] [PATCH 0/2] Not showing upgraded/added packages in 'guix pull' Liliana Marie Prikler
0 siblings, 2 replies; 12+ messages in thread
From: Ludovic Courtès @ 2022-02-14 16:38 UTC (permalink / raw)
To: 54000; +Cc: Ludovic Courtès
Hello!
As a followup to <https://issues.guix.gnu.org/53909>, these patches
remove the list of added/upgraded packages from the output of
‘guix pull’ (upon completion) and ‘guix pull --news’.
It’s not spring yet in this hemisphere, but it looks like a spring cleanup!
Ludo’.
Ludovic Courtès (2):
pull: '--news' no longer shows package lists.
pull: No longer print upgraded/added packages upon completion.
doc/guix.texi | 10 ++---
guix/scripts/pull.scm | 98 +++++++++++++++++++++++--------------------
2 files changed, 57 insertions(+), 51 deletions(-)
base-commit: 41000d16c5c1586482a76d856c3152a6b8fcce8a
--
2.34.0
^ permalink raw reply [flat|nested] 12+ messages in thread
* [bug#54000] [PATCH 1/2] pull: '--news' no longer shows package lists.
2022-02-14 16:38 [bug#54000] [PATCH 0/2] Not showing upgraded/added packages in 'guix pull' Ludovic Courtès
@ 2022-02-14 16:41 ` Ludovic Courtès
2022-02-14 16:41 ` [bug#54000] [PATCH 2/2] pull: No longer print upgraded/added packages upon completion Ludovic Courtès
2022-02-15 13:55 ` [bug#54000] [PATCH 0/2] Not showing upgraded/added packages in 'guix pull' Liliana Marie Prikler
1 sibling, 1 reply; 12+ messages in thread
From: Ludovic Courtès @ 2022-02-14 16:41 UTC (permalink / raw)
To: 54000; +Cc: Ludovic Courtès
* guix/scripts/pull.scm (display-channel-news): Return #t when news were
displayed.
(display-news): Remove call to 'display-profile-news'. Print something
when there were no news.
(process-query): For 'display-news', call 'display-channel-news'
directly.
* doc/guix.texi (Invoking guix pull): Adjust accordingly.
---
doc/guix.texi | 10 ++---
guix/scripts/pull.scm | 93 ++++++++++++++++++++++++-------------------
2 files changed, 55 insertions(+), 48 deletions(-)
diff --git a/doc/guix.texi b/doc/guix.texi
index a05f073c3d..534414ab8a 100644
--- a/doc/guix.texi
+++ b/doc/guix.texi
@@ -4486,13 +4486,11 @@ information.
@cindex channel news
@item --news
@itemx -N
-Display the list of packages added or upgraded since the previous
-generation, as well as, occasionally, news written by channel authors
-for their users (@pxref{Channels, Writing Channel News}).
+Display news written by channel authors for their users for changes made
+since the previous generation (@pxref{Channels, Writing Channel News}).
-The package information is the same as displayed upon @command{guix
-pull} completion, but without ellipses; it is also similar to the output
-of @command{guix pull -l} for the last generation (see below).
+You can view that information for previous generations with
+@command{guix pull -l}.
@item --list-generations[=@var{pattern}]
@itemx -l [@var{pattern}]
diff --git a/guix/scripts/pull.scm b/guix/scripts/pull.scm
index 205697b3bd..af1b27d7dd 100644
--- a/guix/scripts/pull.scm
+++ b/guix/scripts/pull.scm
@@ -352,45 +352,48 @@ (define* (display-channel-news profile
(previous
(and=> (relative-generation profile -1)
(cut generation-file-name profile <>))))
- "Display news about the channels of PROFILE compared to PREVIOUS."
- (when previous
- (let ((old-channels (profile-channels previous))
- (new-channels (profile-channels profile)))
- (and (pair? old-channels) (pair? new-channels)
- (begin
- (match (lset-difference channel=? new-channels old-channels)
- (()
- #t)
- (new
- (let ((count (length new)))
- (format (current-error-port)
- (N_ " ~a new channel:~%"
- " ~a new channels:~%" count)
- count)
- (for-each display-channel new))))
- (match (lset-difference channel=? old-channels new-channels)
- (()
- #t)
- (removed
- (let ((count (length removed)))
- (format (current-error-port)
- (N_ " ~a channel removed:~%"
- " ~a channels removed:~%" count)
- count)
- (for-each display-channel removed))))
+ "Display news about the channels of PROFILE compared to PREVIOUS. Return
+true if news were displayed, false otherwise."
+ (and previous
+ (let ((old-channels (profile-channels previous))
+ (new-channels (profile-channels profile)))
+ (and (pair? old-channels) (pair? new-channels)
+ (begin
+ (match (lset-difference channel=? new-channels old-channels)
+ (()
+ #t)
+ (new
+ (let ((count (length new)))
+ (format (current-error-port)
+ (N_ " ~a new channel:~%"
+ " ~a new channels:~%" count)
+ count)
+ (for-each display-channel new))))
+ (match (lset-difference channel=? old-channels new-channels)
+ (()
+ #t)
+ (removed
+ (let ((count (length removed)))
+ (format (current-error-port)
+ (N_ " ~a channel removed:~%"
+ " ~a channels removed:~%" count)
+ count)
+ (for-each display-channel removed))))
- ;; Display channel-specific news for those channels that were
- ;; here before and are still around afterwards.
- (for-each (match-lambda
- ((new old)
- (display-channel-specific-news new old)))
- (filter-map (lambda (new)
- (define old
- (find (cut channel=? new <>)
- old-channels))
+ ;; Display channel-specific news for those channels that were
+ ;; here before and are still around afterwards.
+ (fold (match-lambda*
+ (((new old) news?)
+ (or (display-channel-specific-news new old)
+ news?)))
+ #f
+ (filter-map (lambda (new)
+ (define old
+ (find (cut channel=? new <>)
+ old-channels))
- (and old (list new old)))
- new-channels)))))))
+ (and old (list new old)))
+ new-channels)))))))
(define* (display-channel-news-headlines profile)
"Display the titles of news about the channels of PROFILE compared to its
@@ -422,12 +425,18 @@ (define more?
(any ->bool more?))))))
(define (display-news profile)
- ;; Display profile news, with the understanding that this process represents
- ;; the newest generation.
- (display-profile-news profile
- #:current-is-newer? #t)
+ "Display channel news for PROFILE compared to its previous generation."
+ (define previous
+ (relative-generation profile -1))
- (display-channel-news profile))
+ (if previous
+ (unless (display-channel-news profile
+ (generation-file-name profile previous))
+ (info (G_ "no channel news since generation ~a~%") previous)
+ (display-hint (G_ "Run @command{guix pull -l} to view the
+news for earlier generations.")))
+ (leave (G_ "profile ~a does not have a previous generation~%")
+ profile)))
(define* (build-and-install instances profile)
"Build the tool from SOURCE, and install it in PROFILE. When DRY-RUN? is
--
2.34.0
^ permalink raw reply related [flat|nested] 12+ messages in thread
* [bug#54000] [PATCH 2/2] pull: No longer print upgraded/added packages upon completion.
2022-02-14 16:41 ` [bug#54000] [PATCH 1/2] pull: '--news' no longer shows package lists Ludovic Courtès
@ 2022-02-14 16:41 ` Ludovic Courtès
0 siblings, 0 replies; 12+ messages in thread
From: Ludovic Courtès @ 2022-02-14 16:41 UTC (permalink / raw)
To: 54000; +Cc: Ludovic Courtès
* guix/scripts/pull.scm (build-and-install): Remove call to
'display-profile-news' and adjust accordingly.
---
guix/scripts/pull.scm | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/guix/scripts/pull.scm b/guix/scripts/pull.scm
index af1b27d7dd..f557aaf79a 100644
--- a/guix/scripts/pull.scm
+++ b/guix/scripts/pull.scm
@@ -454,10 +454,9 @@ (define guix-command
#:hooks %channel-profile-hooks)
(return
- (let ((more? (list (display-profile-news profile #:concise? #t)
- (display-channel-news-headlines profile))))
+ (let ((more? (display-channel-news-headlines profile)))
(newline)
- (when (any ->bool more?)
+ (when more?
(display-hint
(G_ "Run @command{guix pull --news} to read all the news.")))))
(if guix-command
--
2.34.0
^ permalink raw reply related [flat|nested] 12+ messages in thread
* [bug#54000] [PATCH 0/2] Not showing upgraded/added packages in 'guix pull'
2022-02-14 16:38 [bug#54000] [PATCH 0/2] Not showing upgraded/added packages in 'guix pull' Ludovic Courtès
2022-02-14 16:41 ` [bug#54000] [PATCH 1/2] pull: '--news' no longer shows package lists Ludovic Courtès
@ 2022-02-15 13:55 ` Liliana Marie Prikler
2022-02-17 10:19 ` Ludovic Courtès
1 sibling, 1 reply; 12+ messages in thread
From: Liliana Marie Prikler @ 2022-02-15 13:55 UTC (permalink / raw)
To: Ludovic Courtès, 54000
Am Montag, dem 14.02.2022 um 17:38 +0100 schrieb Ludovic Courtès:
> Hello!
>
> As a followup to <https://issues.guix.gnu.org/53909>, these patches
> remove the list of added/upgraded packages from the output of
> ‘guix pull’ (upon completion) and ‘guix pull --news’.
>
> It’s not spring yet in this hemisphere, but it looks like a spring
> cleanup!
>
> Ludo’.
>
> Ludovic Courtès (2):
> pull: '--news' no longer shows package lists.
> pull: No longer print upgraded/added packages upon completion.
What if I wanted that information however? Does `guix pull --news --
details` still give it to me? I don't really want to do `guix pull -l
1d --details` instead.
Cheers
^ permalink raw reply [flat|nested] 12+ messages in thread
* [bug#54000] [PATCH 0/2] Not showing upgraded/added packages in 'guix pull'
2022-02-15 13:55 ` [bug#54000] [PATCH 0/2] Not showing upgraded/added packages in 'guix pull' Liliana Marie Prikler
@ 2022-02-17 10:19 ` Ludovic Courtès
2022-02-17 10:30 ` Liliana Marie Prikler
0 siblings, 1 reply; 12+ messages in thread
From: Ludovic Courtès @ 2022-02-17 10:19 UTC (permalink / raw)
To: Liliana Marie Prikler; +Cc: 54000
Hi,
Liliana Marie Prikler <liliana.prikler@ist.tugraz.at> skribis:
> Am Montag, dem 14.02.2022 um 17:38 +0100 schrieb Ludovic Courtès:
>> Hello!
>>
>> As a followup to <https://issues.guix.gnu.org/53909>, these patches
>> remove the list of added/upgraded packages from the output of
>> ‘guix pull’ (upon completion) and ‘guix pull --news’.
[...]
>> pull: '--news' no longer shows package lists.
>> pull: No longer print upgraded/added packages upon completion.
> What if I wanted that information however?
Is it an actual use case or speculation?
> Does `guix pull --news -- details` still give it to me?
Not with this patch, but we could do that *if* there’s a need.
> I don't really want to do `guix pull -l 1d --details` instead.
OK. So far, my impression was that this information was hardly used at
all, which is why it seemed reasonable to keep it behind ‘--details
--list-generations’.
I’m open to supporting ‘--news --details’, but only if it corresponds to
a real use case.
Thanks,
Ludo’.
^ permalink raw reply [flat|nested] 12+ messages in thread
* [bug#54000] [PATCH 0/2] Not showing upgraded/added packages in 'guix pull'
2022-02-17 10:19 ` Ludovic Courtès
@ 2022-02-17 10:30 ` Liliana Marie Prikler
2022-02-17 12:04 ` zimoun
0 siblings, 1 reply; 12+ messages in thread
From: Liliana Marie Prikler @ 2022-02-17 10:30 UTC (permalink / raw)
To: Ludovic Courtès; +Cc: 54000
Hi Ludo,
Am Donnerstag, dem 17.02.2022 um 11:19 +0100 schrieb Ludovic Courtès:
> Hi,
>
> Liliana Marie Prikler <liliana.prikler@ist.tugraz.at> skribis:
>
> > Am Montag, dem 14.02.2022 um 17:38 +0100 schrieb Ludovic Courtès:
> > > Hello!
> > >
> > > As a followup to <https://issues.guix.gnu.org/53909>, these
> > > patches
> > > remove the list of added/upgraded packages from the output of
> > > ‘guix pull’ (upon completion) and ‘guix pull --news’.
>
> [...]
>
> > > pull: '--news' no longer shows package lists.
> > > pull: No longer print upgraded/added packages upon completion.
> > What if I wanted that information however?
>
> Is it an actual use case or speculation?
That's a use case. While --dry-run exists, I don't really want it to
serve double duty here. If I previously guix pulled and only three new
packages were added, none of which I'm interested in, I would not have
to meaninglessly run further build commands like guix package or guix
system. Even if the list is potentially longer, I could visually grep
for a few packages I'm interested in and determine whether it'd make
sense to build now or wait for a little while as I'm processing other
things.
> > Does `guix pull --news -- details` still give it to me?
>
> Not with this patch, but we could do that *if* there’s a need.
>
> > I don't really want to do `guix pull -l 1d --details` instead.
>
> OK. So far, my impression was that this information was hardly used
> at all, which is why it seemed reasonable to keep it behind ‘--
> details --list-generations’.
>
> I’m open to supporting ‘--news --details’, but only if it corresponds
> to a real use case.
In general, it doesn't have to be named ‘--news --details’, but a means
of diffing two generations via the CLI -- in particular the current one
to the last -- would be very welcome, because then all information we
previously had would still be available quite easily, albeit no longer
printed by default.
Cheers
^ permalink raw reply [flat|nested] 12+ messages in thread
* [bug#54000] [PATCH 0/2] Not showing upgraded/added packages in 'guix pull'
2022-02-17 10:30 ` Liliana Marie Prikler
@ 2022-02-17 12:04 ` zimoun
2022-02-17 12:52 ` Liliana Marie Prikler
0 siblings, 1 reply; 12+ messages in thread
From: zimoun @ 2022-02-17 12:04 UTC (permalink / raw)
To: Liliana Marie Prikler, Ludovic Courtès; +Cc: 54000
Hi Liliana,
On Thu, 17 Feb 2022 at 11:30, Liliana Marie Prikler <liliana.prikler@ist.tugraz.at> wrote:
>> Is it an actual use case or speculation?
>
> That's a use case. While --dry-run exists, I don't really want it to
> serve double duty here. If I previously guix pulled and only three new
> packages were added, none of which I'm interested in, I would not have
> to meaninglessly run further build commands like guix package or guix
> system. Even if the list is potentially longer, I could visually grep
> for a few packages I'm interested in and determine whether it'd make
> sense to build now or wait for a little while as I'm processing other
> things.
Stats, ~24 updates and ~13 additions per day on average (over the past
year). If you pull twice a day, then yes you can read this information.
But, I bet people pull once a week, at best, so it looks like more
“noise“ and I guess most people miss the news.
About grepping, it is not straightforward. For instance, I get this:
--8<---------------cut here---------------start------------->8---
$ guix pull --news | grep python
openshadinglanguage, perl-date-range, pgcli, plfit, poweralertd, pproxy, python-aiosignal, python-android-backup,
python-asdf-astropy, python-astral, python-astropy-healpix, python-astroquery, python-canvasapi, python-cmarkgfm,
python-cucumber-tag-expressions, python-cython-next, python-doit, python-esprima, python-executing,
python-flask-assets, python-flit-core-bootstrap, python-frozenlist, python-fs, python-geojson, python-gwcs,
python-ipython-sql, python-markdownify, python-miio, python-pgspecial, python-photutils, python-phpserialize,
python-piexif, python-psycopg, python-psycopg-pool, python-pydbus, python-pyftpdlib, python-pylru, python-pyowm,
python-pypdf3, python-pyrss2gen, python-pyscss, python-pysendfile, python-pystitcher,
python-pytest-doctest-custom, python-pytest-metadata, python-pytest-pydocstyle, python-pyvo, python-reedsolo,
python-regions, python-retry, python-roundrobin, python-sarge, python-sentry-sdk, python-setuptools-rust,
python-sphinx-click, python-sphinxcontrib-apidoc, python-tomli-w, python-tweepy, python-typeguard,
--8<---------------cut here---------------end--------------->8---
So I do not think the current display is adequate for grepping.
Somehow, the feature you want should be separated. For instance,
guix pull --updated
guix pull --added
guix pull --new-versions
> In general, it doesn't have to be named ‘--news --details’, but a means
> of diffing two generations via the CLI -- in particular the current one
> to the last -- would be very welcome, because then all information we
> previously had would still be available quite easily, albeit no longer
> printed by default.
That feature is interesting but it appears to me orthogonal with the
current proposal.
Cheers,
simon
^ permalink raw reply [flat|nested] 12+ messages in thread
* [bug#54000] [PATCH 0/2] Not showing upgraded/added packages in 'guix pull'
2022-02-17 12:04 ` zimoun
@ 2022-02-17 12:52 ` Liliana Marie Prikler
2022-02-17 14:41 ` zimoun
0 siblings, 1 reply; 12+ messages in thread
From: Liliana Marie Prikler @ 2022-02-17 12:52 UTC (permalink / raw)
To: zimoun, Ludovic Courtès; +Cc: 54000
Hi zimoun,
Am Donnerstag, dem 17.02.2022 um 13:04 +0100 schrieb zimoun:
> Hi Liliana,
>
> Stats, ~24 updates and ~13 additions per day on average (over the
> past year). If you pull twice a day, then yes you can read this
> information.
> But, I bet people pull once a week, at best, so it looks like more
> “noise“ and I guess most people miss the news.
For the record, that's < 150 updates and < 100 additions per week. Of
course, these numbers tend to get higher as Guix grows, but for now I
personally find this both manageable and helpful. Which doesn't mean I
want it done by default, just that I want a way of doing it.
Furthermore, news are displayed at the bottom, so if you're not
interested in the noise, just don't scroll up :P
Granted, if you're piping the output to a pager, that doesn't help you.
> About grepping, it is not straightforward. For instance, I get this:
>
> --8<---------------cut here---------------start------------->8---
> $ guix pull --news | grep python
> openshadinglanguage, perl-date-range, pgcli, plfit, poweralertd,
> pproxy, python-aiosignal, python-android-backup,
> python-asdf-astropy, python-astral, python-astropy-healpix,
> python-astroquery, python-canvasapi, python-cmarkgfm,
> python-cucumber-tag-expressions, python-cython-next, python-doit,
> python-esprima, python-executing,
> python-flask-assets, python-flit-core-bootstrap, python-
> frozenlist, python-fs, python-geojson, python-gwcs,
> python-ipython-sql, python-markdownify, python-miio, python-
> pgspecial, python-photutils, python-phpserialize,
> python-piexif, python-psycopg, python-psycopg-pool, python-
> pydbus, python-pyftpdlib, python-pylru, python-pyowm,
> python-pypdf3, python-pyrss2gen, python-pyscss, python-
> pysendfile, python-pystitcher,
> python-pytest-doctest-custom, python-pytest-metadata, python-
> pytest-pydocstyle, python-pyvo, python-reedsolo,
> python-regions, python-retry, python-roundrobin, python-sarge,
> python-sentry-sdk, python-setuptools-rust,
> python-sphinx-click, python-sphinxcontrib-apidoc, python-tomli-w,
> python-tweepy, python-typeguard,
> --8<---------------cut here---------------end--------------->8---
>
> So I do not think the current display is adequate for grepping.
I agree that with certain packages that's a little harder to do than
with others, but particularly with python the way to resolve this would
be to grep for 'python@'
> Somehow, the feature you want should be separated. For instance,
>
> guix pull --updated
> guix pull --added
> guix pull --new-versions
That would perhaps help if your aim is to optimize for computation
time, but I'd still prefer all changes. It also helps that we already
have that code, so we only have to tell people to e.g. use --changes
instead of --news if that is the thing they wanted.
W.r.t. only listing some of the changes, one could later implement
options like --changes=added,removed,updated,rewritten,...
> > In general, it doesn't have to be named ‘--news --details’, but a
> > means of diffing two generations via the CLI -- in particular the
> > current one to the last -- would be very welcome, because then all
> > information we previously had would still be available quite
> > easily, albeit no longer printed by default.
>
> That feature is interesting but it appears to me orthogonal with the
> current proposal.
Orthogonal in which way? In that we could implement such a feature
without changing the way `guix pull' normally works and vice versa?
Sure. In that we'd not be losing any information if we changed `guix
pull' without providing such an option? Eh...
^ permalink raw reply [flat|nested] 12+ messages in thread
* [bug#54000] [PATCH 0/2] Not showing upgraded/added packages in 'guix pull'
2022-02-17 12:52 ` Liliana Marie Prikler
@ 2022-02-17 14:41 ` zimoun
2022-02-18 9:08 ` Ludovic Courtès
0 siblings, 1 reply; 12+ messages in thread
From: zimoun @ 2022-02-17 14:41 UTC (permalink / raw)
To: Liliana Marie Prikler, Ludovic Courtès; +Cc: 54000
Hi Liliana,
On Thu, 17 Feb 2022 at 13:52, Liliana Marie Prikler <liliana.prikler@ist.tugraz.at> wrote:
>> Stats, ~24 updates and ~13 additions per day on average (over the
>> past year). If you pull twice a day, then yes you can read this
>> information.
>> But, I bet people pull once a week, at best, so it looks like more
>> “noise“ and I guess most people miss the news.
>
> For the record, that's < 150 updates and < 100 additions per week. Of
> course, these numbers tend to get higher as Guix grows, but for now I
> personally find this both manageable and helpful. Which doesn't mean I
> want it done by default, just that I want a way of doing it.
On my poor laptop, I barely pull. Last time, more than 2 weeks ago.
--8<---------------cut here---------------start------------->8---
$ guix describe
Generation 76 Feb 04 2022 11:15:54 (current)
guix ff093f5
repository URL: https://git.savannah.gnu.org/git/guix.git
branch: master
commit: ff093f5739a61e77b296feccc48d260b9bb574c0
--8<---------------cut here---------------end--------------->8---
and a typical ‘guix pull --news’ looks like:
--8<---------------cut here---------------start------------->8---
$ time guix pull --news
New in this revision:
416 new packages: alfis,
[...]
877 packages upgraded: amsynth@1.12.4,
[...]
News for channel 'guix'
New `--execute' option to `guix deploy'
commit 5c13484646069064c834bbd3cd02c3bc80d94cb6
The `guix deploy' command has a new `--execute' or
`-x' option, which allows you to execute a command on
all the machines that your configuration file
specifies, as in this example:
guix deploy deploy.scm -x -- herd restart guix-daemon
This is no substitute for full-featured tools such as
pdsh but it is a useful helper.
`guix style' can format package definitions
commit c4fe13c294cc1e31dd8a49ce3981f603fb169e0a
The recently-introduced `guix style' command can now
be used to automatically format package definitions
according to the Guix project's formatting guidelines.
If you contribute packages to Guix or to a third-party
channel, you may find it useful.
The new `--styling' option can currently be passed one
of the following "styling rules": `format', to format
package definitions, or `inputs', to remove labels
from package inputs. Omitting `--styling' is
equivalent to passing `--styling=format'; previously
it was equivalent to `--styling=inputs'.
Run `info "(guix) Invoking guix style"', for more
info.
real 0m7.796s
user 0m8.131s
sys 0m0.500s
--8<---------------cut here---------------end--------------->8---
> Furthermore, news are displayed at the bottom, so if you're not
> interested in the noise, just don't scroll up :P
> Granted, if you're piping the output to a pager, that doesn't help you.
First, because it is too much information, I never do it.
Second, because it is really slow, I never do it.
Therefore, I never read the news.
Well, I read them from Git, browsing after fetching. :-) Not sure it is
adequate.
I barely upgrade my base system – I made it work once and just use it –
and instead I heavily rely on time-machine and shell.
I agree with your request and I understand the need. Even, I think
diffing generations can be useful. But we should keep the default as
simple and fast as possible.
>> Somehow, the feature you want should be separated. For instance,
>>
>> guix pull --updated
>> guix pull --added
>> guix pull --new-versions
>
> That would perhaps help if your aim is to optimize for computation
> time, but I'd still prefer all changes. It also helps that we already
> have that code, so we only have to tell people to e.g. use --changes
> instead of --news if that is the thing they wanted.
> W.r.t. only listing some of the changes, one could later implement
> options like --changes=added,removed,updated,rewritten,...
We agree.
My point was just to say that ‘--news --details’ is not the correct UI
for the feature you would like because, for instance, currently
’--details’ alone is equivalent to ’--details -l’. Therefore, what
happens if the user only provides ’--details’?
The option ’--changes=’ makes also sense and it is probably a good
direction; instead of ’--news --details’.
Well, my point is just to say that the feature you want should be
separated from ‘--news’. :-)
>> That feature is interesting but it appears to me orthogonal with the
>> current proposal.
>
> Orthogonal in which way? In that we could implement such a feature
> without changing the way `guix pull' normally works and vice versa?
> Sure. In that we'd not be losing any information if we changed `guix
> pull' without providing such an option? Eh...
Well, from my point of view, ‘guix pull -l 1d --details’ fits the job
and I miss why it would be an issue; since such display would be barely
used.
To be precise, the option ’-l’ should accept ’last’. From my point of
view, it is better to have:
guix pull --news # just the last news
guix pull -l last --details
(better meaming encourage people to read news :-)
And we could also imagine some options as ’--details=added’. Note we
could also imagine to make ‘guix git log’ show some information. Lot of
imagination. ;-)
Cheers,
simon
^ permalink raw reply [flat|nested] 12+ messages in thread
* [bug#54000] [PATCH 0/2] Not showing upgraded/added packages in 'guix pull'
2022-02-17 14:41 ` zimoun
@ 2022-02-18 9:08 ` Ludovic Courtès
2022-02-18 9:20 ` Liliana Marie Prikler
0 siblings, 1 reply; 12+ messages in thread
From: Ludovic Courtès @ 2022-02-18 9:08 UTC (permalink / raw)
To: zimoun; +Cc: Liliana Marie Prikler, 54000
Hey comrades, hold your horses! :-)
Since there’s a need for this, we can make ‘--details’ print package
lists when combined with ‘--news’; it doesn’t cost us anything, that’s
OK.
There may be a need for more general package version diffing and such
things, but we can address it separately.
Thanks,
Ludo’.
^ permalink raw reply [flat|nested] 12+ messages in thread
* [bug#54000] [PATCH 0/2] Not showing upgraded/added packages in 'guix pull'
2022-02-18 9:08 ` Ludovic Courtès
@ 2022-02-18 9:20 ` Liliana Marie Prikler
2022-02-27 22:15 ` bug#54000: " Ludovic Courtès
0 siblings, 1 reply; 12+ messages in thread
From: Liliana Marie Prikler @ 2022-02-18 9:20 UTC (permalink / raw)
To: Ludovic Courtès, zimoun; +Cc: 54000
Am Freitag, dem 18.02.2022 um 10:08 +0100 schrieb Ludovic Courtès:
> Hey comrades, hold your horses! :-)
>
> Since there’s a need for this, we can make ‘--details’ print package
> lists when combined with ‘--news’; it doesn’t cost us anything,
> that’s OK.
Regarding the issue simon raised w.r.t. --details implying -l, I think
--details should only imply listing the latest generation if -l is not
specified. That way --news --details would (hopefully) work as I
intended, and specifying one without the other would only print that
part.
On a separate note, would/should guix describe be affected by this?
Its output appear currently the same as `guix pull -l
$CURRENT_GENERATION', but I haven't tested this on a generation
including news. My guess would be "no" as guix describe currently
doesn't accept --news.
Cheers
^ permalink raw reply [flat|nested] 12+ messages in thread
* bug#54000: [PATCH 0/2] Not showing upgraded/added packages in 'guix pull'
2022-02-18 9:20 ` Liliana Marie Prikler
@ 2022-02-27 22:15 ` Ludovic Courtès
0 siblings, 0 replies; 12+ messages in thread
From: Ludovic Courtès @ 2022-02-27 22:15 UTC (permalink / raw)
To: Liliana Marie Prikler; +Cc: 54000-done, zimoun
Hi,
Liliana Marie Prikler <liliana.prikler@ist.tugraz.at> skribis:
> Am Freitag, dem 18.02.2022 um 10:08 +0100 schrieb Ludovic Courtès:
>> Hey comrades, hold your horses! :-)
>>
>> Since there’s a need for this, we can make ‘--details’ print package
>> lists when combined with ‘--news’; it doesn’t cost us anything,
>> that’s OK.
> Regarding the issue simon raised w.r.t. --details implying -l, I think
> --details should only imply listing the latest generation if -l is not
> specified. That way --news --details would (hopefully) work as I
> intended, and specifying one without the other would only print that
> part.
Right, I went with something like you describe:
2ddfb7b99b pull: No longer print upgraded/added packages upon completion.
bc8bea1739 pull: '--news' no longer shows package lists.
That is, one can run ‘guix pull -N --details’ and get what you asked
for. Running ‘guix pull --details’ alone still implies ‘-l’.
Let me know if anything’s amiss.
> On a separate note, would/should guix describe be affected by this?
> Its output appear currently the same as `guix pull -l
> $CURRENT_GENERATION', but I haven't tested this on a generation
> including news. My guess would be "no" as guix describe currently
> doesn't accept --news.
No, ‘guix describe’ doesn’t display news, profile changes, and all
that. It’s meant to just show the raw channel info.
Thanks to both of you for your feedback!
Ludo’.
^ permalink raw reply [flat|nested] 12+ messages in thread
end of thread, other threads:[~2022-02-27 22:16 UTC | newest]
Thread overview: 12+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2022-02-14 16:38 [bug#54000] [PATCH 0/2] Not showing upgraded/added packages in 'guix pull' Ludovic Courtès
2022-02-14 16:41 ` [bug#54000] [PATCH 1/2] pull: '--news' no longer shows package lists Ludovic Courtès
2022-02-14 16:41 ` [bug#54000] [PATCH 2/2] pull: No longer print upgraded/added packages upon completion Ludovic Courtès
2022-02-15 13:55 ` [bug#54000] [PATCH 0/2] Not showing upgraded/added packages in 'guix pull' Liliana Marie Prikler
2022-02-17 10:19 ` Ludovic Courtès
2022-02-17 10:30 ` Liliana Marie Prikler
2022-02-17 12:04 ` zimoun
2022-02-17 12:52 ` Liliana Marie Prikler
2022-02-17 14:41 ` zimoun
2022-02-18 9:08 ` Ludovic Courtès
2022-02-18 9:20 ` Liliana Marie Prikler
2022-02-27 22:15 ` bug#54000: " Ludovic Courtès
Code repositories for project(s) associated with this public inbox
https://git.savannah.gnu.org/cgit/guix.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).