Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1176)

Issue 2829038: [PATCH 3/5] Unify efi and legacy bootloader configuration generation (Closed)

Created:
10 years, 5 months ago by Will Drewry
Modified:
9 years, 7 months ago
Reviewers:
adlr
CC:
chromium-os-reviews_chromium.org, Mandeep Singh Baines, anush, sosa
Visibility:
Public.

Description

[PATCH 3/5] Unify efi and legacy bootloader configuration generation This change unifies the creation of extlinux.conf, syslinux cfgs, and grub efi files. It shouldn't change the existing behavior but does add support for further arguments and future use of syslinux (once it is properly rewritten by an installer or other script). TEST=in progress; manual run BUG=chromium-os:327

Patch Set 1 #

Total comments: 4

Patch Set 2 : replaces common args with variables #

Patch Set 3 : fix line len #

Patch Set 4 : fix typo #

Unified diffs Side-by-side diffs Delta from patch set Stats (+224 lines, -49 lines) Patch
M build_image View 2 chunks +21 lines, -49 lines 0 comments Download
A create_legacy_bootloader_templates.sh View 1 2 3 1 chunk +203 lines, -0 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
Will Drewry
10 years, 5 months ago (2010-07-02 18:19:53 UTC) #1
adlr
LGTM http://codereview.chromium.org/2829038/diff/1/3 File create_legacy_bootloader_templates.sh (right): http://codereview.chromium.org/2829038/diff/1/3#newcode109 create_legacy_bootloader_templates.sh:109: append quiet console=tty2 init=/sbin/init boot=local rootwait root=${FLAGS_usb_disk} ro ...
10 years, 5 months ago (2010-07-02 19:36:36 UTC) #2
Will Drewry
http://codereview.chromium.org/2829038/diff/1/3 File create_legacy_bootloader_templates.sh (right): http://codereview.chromium.org/2829038/diff/1/3#newcode109 create_legacy_bootloader_templates.sh:109: append quiet console=tty2 init=/sbin/init boot=local rootwait root=${FLAGS_usb_disk} ro noresume ...
10 years, 5 months ago (2010-07-02 19:52:03 UTC) #3
adlr
10 years, 5 months ago (2010-07-02 19:53:47 UTC) #4
LGTM thanks

Powered by Google App Engine
This is Rietveld 408576698