[pve-devel] [PATCH docs 2/3] boot: mention pve-efiboot-tool earlier

[Fabian Grünbichler]

On Mon, Jul 15, 2019 at 02:37:45PM +0200, Aaron Lauterer wrote:
> Found a missing possible missing comma
> 
> On 7/15/19 11:35 AM, Fabian Grünbichler wrote:
> > Signed-off-by: Fabian Grünbichler <f.gruenbichler at proxmox.com>
> > ---
> >   system-booting.adoc | 3 ++-
> >   1 file changed, 2 insertions(+), 1 deletion(-)
> > 
> > diff --git a/system-booting.adoc b/system-booting.adoc
> > index c7f6534..fc8d896 100644
> > --- a/system-booting.adoc
> > +++ b/system-booting.adoc
> > @@ -79,7 +79,8 @@ system boots even if the first boot device fails.  The ESPs are kept in sync by
> >   a kernel postinstall hook script `/etc/kernel/postinst.d/zz-pve-efiboot`. The
> >   script copies certain kernel versions and the initrd images to `EFI/proxmox/`
> >   on the root of each ESP and creates the appropriate config files in
> > -`loader/entries/proxmox-*.conf`.
> > +`loader/entries/proxmox-*.conf`. The `pve-efiboot-tool` script assists in
> > +managing both synced ESPs themselves, and their contents.
> 
> This might be nitpicky, but shouldn't there be a comma?
> "managing both, synced ESPs themselves, and their contents."

or no comma at all? :-P

"managing both synced ESPs themselves and their contents."

never been my strong suite, always miss adding them in German as well..

> 
> >   The following kernel versions are configured by default:
> > 
> 
> _______________________________________________
> pve-devel mailing list
> pve-devel at pve.proxmox.com
> https://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel