Hi Alok,

Alok Aggarwal wrote:
> Hi Jan,
>
> Here are my comments:
>
> auto_install.c: lines 920-933: Why was this removed?

Actually, that code is there twice (see lines 891-906) - it was likely
caused by unintentional copy-paste step. It took me a while to determine
why related message appears in log file twice :-)

>
> perform_slim_install.c: line 240: be_attrs need to be
> freed here as well.

Agreed - will be added.

>
> perform_slim_install.c: lines 662, 667, 675: Should these say "pkg(5) 
> packages" instead?

I agree - I will modify this.

>
> live-fs-root: line 396: s/server/install server

I will change this.

Thank you very much for review !
Jan


Reply via email to