Code review comment for lp:~cyphermox/debian-cd/nomodeset

Revision history for this message
Anthony Wong (anthonywong) wrote :

Thanks for working on this stop gap solution.
I have two comments:
1. Should we omit adding this new boot entry when $PROJECT = ubuntu-server? I admit I am not familiar with server image, but results from google show they are text based and so I think server image doesn't need it.
2. I don't want to clutter up the boot menu with safe graphics options. Those who use "OEM install" option should be experienced enough to figure out what cmdline parameters they need. And I think our OEM images do not use this option.

« Back to merge proposal