[Merge] lp:~sergiusens/goget-ubuntu-touch/lock_me_up into lp:goget-ubuntu-touch

Ricardo Salveti rsalveti at rsalveti.net
Tue Nov 4 01:51:28 UTC 2014


Review: Needs Fixing

Just a small typo.

Diff comments:

> === modified file 'ubuntu-device-flash/touch.go'
> --- ubuntu-device-flash/touch.go	2014-11-03 14:07:04 +0000
> +++ ubuntu-device-flash/touch.go	2014-11-03 14:07:04 +0000
> @@ -36,7 +36,7 @@
>  	Bootstrap     bool   `long:"bootstrap" description:"bootstrap the system, do this from the bootloader"`
>  	Wipe          bool   `long:"wipe" description:"Clear all data after flashing"`
>  	Serial        string `long:"serial" description:"Serial of the device to operate"`
> -	DeveloperMode bool   `long:"developer-mode" description:"Enables developer mode after the factory reset"`
> +	DeveloperMode bool   `long:"developer-mode" description:"Enables developer mode after the factory reset, this is meant for automation and makes the device insecure by default"`
>  	DeviceTarball string `long:"device-tarball" description:"Specify a local device tarball to override the one from the server (using official Ubuntu images with custom device tarballs)"`
>  	RunScript     string `long:"run-script" description:"Run a script given by path to finish the flashing process, instead of rebooting to recovery (mostly used during development to work around quirky or incomplete recovery images)"`
>  	Password      string `long:"password" description:"This sets up the default password for the phablet user. This option is meant for CI and not general use"`
> @@ -49,6 +49,10 @@
>  var touchCmd TouchCmd
>  
>  func (touchCmd *TouchCmd) Execute(args []string) error {
> +	if touchCmd.Password || touchCmd.DeveloperMode {
> +		fmt.Println("WARNING --developer-mode and --pasword are dangerous as they remove security features from your device")

typo, password

> +	}
> +
>  	if globalArgs.TLSSkipVerify {
>  		ubuntuimage.TLSSkipVerify()
>  	}
> @@ -198,6 +202,7 @@
>  	var enableList []string
>  	if touchCmd.DeveloperMode {
>  		enableList = append(enableList, "developer_mode")
> +		enableList = append(enableList, "adb_onlock")
>  	}
>  	if touchCmd.Password != "" {
>  		enableList = append(enableList, "default_password "+touchCmd.Password)
> 


-- 
https://code.launchpad.net/~sergiusens/goget-ubuntu-touch/lock_me_up/+merge/240428
Your team Ubuntu Phablet Team is subscribed to branch lp:goget-ubuntu-touch.



More information about the Ubuntu-reviews mailing list