Remove default paramater for Command

TODO.md

@@ -7,9 +7,17 @@
 * [x] Error handling in binary
 * [x] Find a nice logger 
 * [ ] Create linux and macos configuration
+    * macOS
+        * sw_vers
+        * softwareupdate -l
+        * http://www.brendangregg.com/USEmethod/use-macosx.html
+    * Linux
+        * https://www.cyberciti.biz/tips/top-linux-monitoring-tools.html
+        * http://www.brendangregg.com/USEmethod/use-linux.html
 
 ## Before 1.0
 
+* Profiles, Categories, 
 * Config: Use defaults
 * Use a abstract type to hold results
     * Include Host information etc. that are currently in Report (maybe)

contrib/osx.conf

@@ -7,22 +7,19 @@ title = "Current Load"
 description = "Current load and uptime"
 command = "/usr/bin/uptime"
 timeout = 1
-default_run = true
 
 [[command]]
 name = "vm_stat"
 title = "Virtual Memory statistics"
 description = "Current memory usage and statistics in pages"
 command = "vm_stat -c 5 1"
 timeout = 5
-default_run = true
 
 [[command]]
 name = "iostat"
 title = "Kernel I/O statistics"
 description = "Current I/O statistics per dev in KB"
 command = "/usr/sbin/iostat -c 5 -K"
 timeout = 5
-default_run = true
 
 # vim: set ft=toml:

src/command.rs

@@ -29,8 +29,7 @@ pub type Result<T, E = Error> = std::result::Result<T, E>;
 /// ```
 /// # use usereport::command::{Command, CommandResult};
 /// let command = Command::new("uname", r#"/usr/bin/uname -a"#, 5)
-///     .title("Host OS")
-///     .run_by_default(false);
+///     .title("Host OS");
 /// match command.exec() {
 ///     Ok(CommandResult::Success {
 ///         command: _,
@@ -50,7 +49,6 @@ pub struct Command {
     pub(crate) args:        Vec<String>,
     #[serde(rename = "timeout")]
     pub(crate) timeout_sec: u64,
-    pub(crate) default_run: bool,
 }
 
 fn de_ser_args<'de, D>(deserializer: D) -> ::std::result::Result<Vec<String>, D::Error>
@@ -90,7 +88,6 @@ impl Command {
             description: None,
             args,
             timeout_sec,
-            default_run: true,
         }
     }
 
@@ -110,14 +107,6 @@ impl Command {
         }
     }
 
-    /// Set whether to run this command by default
-    pub fn run_by_default(self, value: bool) -> Command {
-        Command {
-            default_run: value,
-            ..self
-        }
-    }
-
     /// Execute this command
     pub fn exec(self) -> Result<CommandResult> {
         let mut p = Popen::create(

src/config.rs

@@ -88,7 +88,6 @@ default_run = true
             Command::new("uname", "/usr/bin/uname -a", 1)
                 .title("Host OS")
                 .description("Basic host OS information")
-                .run_by_default(true),
         );
         let expected = Config { defaults, commands };