search

divx118 / crouton-packages

Kernel-headers packages to use with crouton
56 stars 16 forks source link

Accept default option, as indicated. #21

Closed JoshMcCullough closed 8 years ago

JoshMcCullough commented 8 years ago

After running sudo sh change-kernel-flags, I'm prompted to confirm my action and presented with the options 'Y' or 'n'. Since 'Y' is uppercase, indicating the default value, I hit enter but then the script simply exited. This is nit-picky, but the 'Y' should be lower case or correctly default when enter is pressed.

Do you want to apply those changes (Y/n)? [enter] Exiting with no changes

divx118 commented 8 years ago

@JoshMcCullough You are totally right, sorry overlooked it. I should make 'n' uppercase. I will change it ASAP. Initially I had y as default, but decided that it is better to make n the default, so you are sure you want to go ahead.

divx118 commented 8 years ago

Fixed https://github.com/divx118/crouton-packages/commit/acc9cd74b4e0dfce9730f69557d5d0dba76e239a thanks.

JoshMcCullough commented 8 years ago

NP, it's a small bug! Thanks for making this available, I've now got VirtualBox up and running on my Pixel2. :+1: