Remove misfiring IP address presence check.

This check assumed that IP addresses are only given for debugging with
NetCore, but since @hawkw's improvements recently, that is no longer the
case.

Now, this check is actually preventing anyone from using the gimlet
subcommand, which is a Detached command.

I glanced over the commands that use this, and I don't think this check
is pulling its weight -- I don't think it's worth introducing another
piece of command metadata to specify "I'm detached but also I'd like an
IP address." So, I've removed it.

Author: cbiffle

Cargo.lock

@@ -17,7 +17,7 @@ name = "humility"
 #
 # Be sure to check in and push all of the files that change.  Happy versioning!
 #
-version = "0.10.38"
+version = "0.10.39"
 authors = ["Bryan Cantrill <[email protected]>"]
 edition = "2018"
 license = "MPL-2.0"

Cargo.toml

@@ -106,10 +106,6 @@ pub fn subcommand(
                 bail!("cannot specify a dump for {} command", cmd);
             }
 
-            if context.cli.ip.is_some() {
-                bail!("cannot specify IP address for {} command", cmd);
-            }
-
             if context.cli.probe.is_some() {
                 bail!("cannot specify probe for {} command", cmd);
             }

src/cmd.rs

@@ -13,7 +13,7 @@ For more information try --help
 
 ```
 $ humility --chip this-can-be-anything -V
-humility 0.10.38
+humility 0.10.39
 
 ```
 
@@ -28,7 +28,7 @@ For more information try --help
 
 ```
 $ humility -c apx432 -V
-humility 0.10.38
+humility 0.10.39
 
 ```
 

tests/cmd/chip.trycmd

@@ -2,14 +2,14 @@ Long version flag:
 
 ```
 $ humility --version
-humility 0.10.38
+humility 0.10.39
 
 ```
 
 Short version flag:
 
 ```
 $ humility -V
-humility 0.10.38
+humility 0.10.39
 
 ```