unofficial mirror of bug-guix@gnu.org 
 help / color / mirror / code / Atom feed
* bug#30916: Request: add a short description field for os-configuration
@ 2018-03-23 12:51 Martin Castillo
  2018-03-23 14:15 ` ng0
  0 siblings, 1 reply; 8+ messages in thread
From: Martin Castillo @ 2018-03-23 12:51 UTC (permalink / raw)
  To: 30916


[-- Attachment #1.1: Type: text/plain, Size: 315 bytes --]

hi,

the grub entries for old system generations aren't very helpful. It
would be nice, if there was a field in the operating system declaration
stating what was changed in this generation, that would be added to the
boot entry label.

Martin
-- 
GPG: 7FDE 7190 2F73 2C50 236E  403D CC13 48F1 E644 08EC


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

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

* bug#30916: Request: add a short description field for os-configuration
  2018-03-23 12:51 bug#30916: Request: add a short description field for os-configuration Martin Castillo
@ 2018-03-23 14:15 ` ng0
  2018-03-24 12:33   ` Martin Castillo
  0 siblings, 1 reply; 8+ messages in thread
From: ng0 @ 2018-03-23 14:15 UTC (permalink / raw)
  To: Martin Castillo; +Cc: 30916

Martin Castillo transcribed 1.6K bytes:
> hi,
> 
> the grub entries for old system generations aren't very helpful. It
> would be nice, if there was a field in the operating system declaration
> stating what was changed in this generation, that would be added to the
> boot entry label.
> 
> Martin
> -- 
> GPG: 7FDE 7190 2F73 2C50 236E  403D CC13 48F1 E644 08EC
> 

Hi,

Could you be a more specific what you think is missing?
Your request is written in a very open way, and the space in GRUB menus is
limited as far as I assume.


-- 
A88C8ADD129828D7EAC02E52E22F9BBFEE348588
https://n0.is

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

* bug#30916: Request: add a short description field for os-configuration
  2018-03-23 14:15 ` ng0
@ 2018-03-24 12:33   ` Martin Castillo
  2018-03-24 12:45     ` ng0
  0 siblings, 1 reply; 8+ messages in thread
From: Martin Castillo @ 2018-03-24 12:33 UTC (permalink / raw)
  To: ng0; +Cc: 30916


[-- Attachment #1.1: Type: text/plain, Size: 1094 bytes --]

Hi,

On 23.03.2018 15:15, ng0 wrote:

> Could you be a more specific what you think is missing?
> Your request is written in a very open way, and the space in GRUB menus is
> limited as far as I assume.

Sure,

Currently the grub menu looks like
GNU with Linux-Libre 4.15.12 (beta)
GNU system, old configurations...

and in the submenu
GNU with Linux-Libre 4.15.6 (beta) (#1, 2017-12-18 13:45)
GNU with Linux-Libre 4.15.8 (beta) (#2, 2017-12-22 12:15)
GNU with Linux-Libre 4.15.11 (beta) (#3, 2018-03-23 15:32)

but I'd like to have something like

GNU with Linux-Libre 4.15.12 (beta) [Add ssh-service on port 2222]
GNU system, old configurations...

and in the submenu
GNU with Linux-Libre 4.15.6 (beta) (#1, 2017-12-18 13:45)
GNU with Linux-Libre 4.15.8 (beta) (#2, 2017-12-22 12:15) [Add user bob]
GNU with Linux-Libre 4.15.11 (beta) (#3, 2018-03-23 15:32) [Add xfce]

where the description at the end comes from a field in the operating
system configuration (or the bootloader configuration).

-- 
GPG: 7FDE 7190 2F73 2C50 236E  403D CC13 48F1 E644 08EC


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

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

* bug#30916: Request: add a short description field for os-configuration
  2018-03-24 12:33   ` Martin Castillo
@ 2018-03-24 12:45     ` ng0
  2018-03-24 13:56       ` Martin Castillo
  0 siblings, 1 reply; 8+ messages in thread
From: ng0 @ 2018-03-24 12:45 UTC (permalink / raw)
  To: Martin Castillo; +Cc: 30916, ng0

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

Martin Castillo transcribed 2.5K bytes:
> Hi,
> 
> On 23.03.2018 15:15, ng0 wrote:
> 
> > Could you be a more specific what you think is missing?
> > Your request is written in a very open way, and the space in GRUB menus is
> > limited as far as I assume.
> 
> Sure,
> 
> Currently the grub menu looks like
> GNU with Linux-Libre 4.15.12 (beta)
> GNU system, old configurations...
> 
> and in the submenu
> GNU with Linux-Libre 4.15.6 (beta) (#1, 2017-12-18 13:45)
> GNU with Linux-Libre 4.15.8 (beta) (#2, 2017-12-22 12:15)
> GNU with Linux-Libre 4.15.11 (beta) (#3, 2018-03-23 15:32)
> 
> but I'd like to have something like
> 
> GNU with Linux-Libre 4.15.12 (beta) [Add ssh-service on port 2222]
> GNU system, old configurations...
> 
> and in the submenu
> GNU with Linux-Libre 4.15.6 (beta) (#1, 2017-12-18 13:45)
> GNU with Linux-Libre 4.15.8 (beta) (#2, 2017-12-22 12:15) [Add user bob]
> GNU with Linux-Libre 4.15.11 (beta) (#3, 2018-03-23 15:32) [Add xfce]
> 
> where the description at the end comes from a field in the operating
> system configuration (or the bootloader configuration).
> 
> -- 
> GPG: 7FDE 7190 2F73 2C50 236E  403D CC13 48F1 E644 08EC
> 

So basically you want a field in the operating-system declaration where you
can _manually_ set a description of a certain maximum length which will be
added to the GRUB entry of the generated system generation?
I don't see automatic generation happening, as there's so much that can be
changed and automated summary would easily mess up the GRUB list. I wouldn't
want that. If anything, optional entry with a manual note is the way to do it.


-- 
A88C8ADD129828D7EAC02E52E22F9BBFEE348588
https://n0.is

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

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

* bug#30916: Request: add a short description field for os-configuration
  2018-03-24 12:45     ` ng0
@ 2018-03-24 13:56       ` Martin Castillo
  2018-03-24 16:54         ` Danny Milosavljevic
  0 siblings, 1 reply; 8+ messages in thread
From: Martin Castillo @ 2018-03-24 13:56 UTC (permalink / raw)
  To: ng0; +Cc: 30916


[-- Attachment #1.1: Type: text/plain, Size: 291 bytes --]

> So basically you want a field in the operating-system declaration where you
> can _manually_ set a description of a certain maximum length which will be
> added to the GRUB entry of the generated system generation?
yes

-- 
GPG: 7FDE 7190 2F73 2C50 236E  403D CC13 48F1 E644 08EC


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

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

* bug#30916: Request: add a short description field for os-configuration
  2018-03-24 13:56       ` Martin Castillo
@ 2018-03-24 16:54         ` Danny Milosavljevic
  2018-03-24 17:00           ` ng0
  2018-03-26 13:05           ` Ludovic Courtès
  0 siblings, 2 replies; 8+ messages in thread
From: Danny Milosavljevic @ 2018-03-24 16:54 UTC (permalink / raw)
  To: Martin Castillo; +Cc: 30916, ng0

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

Hi Martin,

On Sat, 24 Mar 2018 14:56:03 +0100
Martin Castillo <castilma@uni-bremen.de> wrote:

> ng0 wrote:
> > So basically you want a field in the operating-system declaration where you
> > can _manually_ set a description of a certain maximum length which will be
> > added to the GRUB entry of the generated system generation?  
> yes

I wonder whether this description can be generated instead - we have all
the information we need - the packages, the users that are there etc.

Or a description could be generated only if a custom description is not specified.

In fact it's easy to add this and would be a nice intro project for a
person interested in Guix development.  I can mentor.

The thing used to fill the Guix bootloader entries is <boot-parameters>.

There's a procedure "operating-system-boot-parameters" which is used
to generate <boot-parameters> instances from an <operating-system>
declaration.

(operating-system-bootcfg calls operating-system-boot-parameters)
(perform-action calls operating-system-bootcfg)
(perform-action is in the top-level guix script)

<boot-parameters> are serialized to disk into:

/var/guix/profiles/system-704-link$ cat parameters 
(boot-parameters (version 0) (label "GNU with Linux-Libre 4.14.14 (beta)") (root-device "dayas:/") (kernel "/gnu/store/fnk2xhicbrjsvbq082p6x0ch6npkrg0z-linux-libre-4.14.14/bzImage") (kernel-arguments ("crashkernel=256M" "modprobe.blacklist=pcspkr,snd_pcsp" "quiet" "acpi_osi=Linux" "clocksource=acpi_pm" "allow-discards" "root_trim=yes")) (initrd "/gnu/store/nvhkdssz1m1p8xrggi78y8pd7jz4p3ng-raw-initrd/initrd") (bootloader-name grub) (store (device "dayas:/") (mount-point "/")))

But I wouldn't change the serialization format or what fields <boot-parameters> contain.

Just change operating-system-bootcfg to take a "description" parameter.
And change operating-system-boot-parameters to take a "description" parameter and
use it to calculate the label.

And change perform-action to calculate the value for to "description" parameter in this way:
- Taking it from <operating-system> (or the command line?) 
- Falling back to an automatic value (comparing it to the previous generation) otherwise.

That's it.

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

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

* bug#30916: Request: add a short description field for os-configuration
  2018-03-24 16:54         ` Danny Milosavljevic
@ 2018-03-24 17:00           ` ng0
  2018-03-26 13:05           ` Ludovic Courtès
  1 sibling, 0 replies; 8+ messages in thread
From: ng0 @ 2018-03-24 17:00 UTC (permalink / raw)
  To: Danny Milosavljevic; +Cc: 30916, ng0

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

Danny Milosavljevic transcribed 2.9K bytes:
> Hi Martin,
> 
> On Sat, 24 Mar 2018 14:56:03 +0100
> Martin Castillo <castilma@uni-bremen.de> wrote:
> 
> > ng0 wrote:
> > > So basically you want a field in the operating-system declaration where you
> > > can _manually_ set a description of a certain maximum length which will be
> > > added to the GRUB entry of the generated system generation?  
> > yes
> 
> I wonder whether this description can be generated instead - we have all
> the information we need - the packages, the users that are there etc.

But how much space do we have in the GRUB descriptions? I have computers with 800x600 or what
it was resolution for the screen, and while this would be a nice feature I wonder if there's
some implications in GRUB menu readability.
I'm not an expert in GRUB, I can boot without a menu, but GRUB is a small operating system
on its own :)

> Or a description could be generated only if a custom description is not specified.
> 
> In fact it's easy to add this and would be a nice intro project for a
> person interested in Guix development.  I can mentor.
> 
> The thing used to fill the Guix bootloader entries is <boot-parameters>.
> 
> There's a procedure "operating-system-boot-parameters" which is used
> to generate <boot-parameters> instances from an <operating-system>
> declaration.
> 
> (operating-system-bootcfg calls operating-system-boot-parameters)
> (perform-action calls operating-system-bootcfg)
> (perform-action is in the top-level guix script)
> 
> <boot-parameters> are serialized to disk into:
> 
> /var/guix/profiles/system-704-link$ cat parameters 
> (boot-parameters (version 0) (label "GNU with Linux-Libre 4.14.14 (beta)") (root-device "dayas:/") (kernel "/gnu/store/fnk2xhicbrjsvbq082p6x0ch6npkrg0z-linux-libre-4.14.14/bzImage") (kernel-arguments ("crashkernel=256M" "modprobe.blacklist=pcspkr,snd_pcsp" "quiet" "acpi_osi=Linux" "clocksource=acpi_pm" "allow-discards" "root_trim=yes")) (initrd "/gnu/store/nvhkdssz1m1p8xrggi78y8pd7jz4p3ng-raw-initrd/initrd") (bootloader-name grub) (store (device "dayas:/") (mount-point "/")))
> 
> But I wouldn't change the serialization format or what fields <boot-parameters> contain.
> 
> Just change operating-system-bootcfg to take a "description" parameter.
> And change operating-system-boot-parameters to take a "description" parameter and
> use it to calculate the label.
> 
> And change perform-action to calculate the value for to "description" parameter in this way:
> - Taking it from <operating-system> (or the command line?) 
> - Falling back to an automatic value (comparing it to the previous generation) otherwise.
> 
> That's it.



-- 
A88C8ADD129828D7EAC02E52E22F9BBFEE348588
https://n0.is

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

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

* bug#30916: Request: add a short description field for os-configuration
  2018-03-24 16:54         ` Danny Milosavljevic
  2018-03-24 17:00           ` ng0
@ 2018-03-26 13:05           ` Ludovic Courtès
  1 sibling, 0 replies; 8+ messages in thread
From: Ludovic Courtès @ 2018-03-26 13:05 UTC (permalink / raw)
  To: Danny Milosavljevic; +Cc: 30916, ng0

Hello,

What about offering a way to specific the label of the new menu entry,
or otherwise a suffix or prefix of the label?

That is, you’d do:

  guix system reconfigure -m "The config that works just fine, I think." config.scm

and that string would be used as the label itself in the GRUB menu, or
as a prefix/suffix of the automatically generated label.

Thoughts?

Ludo’.

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

end of thread, other threads:[~2018-03-26 13:06 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2018-03-23 12:51 bug#30916: Request: add a short description field for os-configuration Martin Castillo
2018-03-23 14:15 ` ng0
2018-03-24 12:33   ` Martin Castillo
2018-03-24 12:45     ` ng0
2018-03-24 13:56       ` Martin Castillo
2018-03-24 16:54         ` Danny Milosavljevic
2018-03-24 17:00           ` ng0
2018-03-26 13:05           ` 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).