rotctl: only support long quit command

gpredict actually sends either q or Q when disconnecting. This is not
actually a supported command according to my reading of the rotctld
documentation. `q`/`Q` for quitting is limited to the interactive
rotctl prompt.

For autoparking, we don't want to quit when gpredict disconnects. Also
in general, we probably don't want to quit when gpredict disconnects.
I still want to have a quit command when using this via netcat or
whatever, so make them a form gpredict probably does not send.

src/RotCtl.zig

@@ -37,7 +37,8 @@ pub fn run(allocator: std.mem.Allocator) !void {
         const client = try server.accept();
         defer {
             log.info("disconnecting client", .{});
-            interface.rotator.stop();
+            if (!config.rotctl.autopark)
+                interface.rotator.stop();
             client.stream.close();
         }
 
@@ -60,10 +61,11 @@ pub fn run(allocator: std.mem.Allocator) !void {
         }
 
         // loop ended due to client disconnect
-        if (interface.running and config.rotctl.autopark) {
+        if (interface.running and config.rotctl.autopark)
             interface.rotator.startPark();
-        }
     }
+
+    interface.rotator.control_thread.join();
 }
 
 fn write(self: *RotCtl, buf: []const u8) !void {
@@ -175,7 +177,6 @@ fn handleHamlibCommand(
     if (first.len == 1 or first[0] == '\\') {
         switch (first[0]) {
             // NOTE: this is not technically supported by rotctld.
-            'q', 'Q' => try self.quit(first, &tokens),
             'S' => try self.stop(first, &tokens),
             'K' => try self.park(first, &tokens),
             'p' => try self.getPosition(first, &tokens),
@@ -246,8 +247,8 @@ const HamlibCommand = struct {
 };
 
 const rotctl_commands = [_]HamlibCommand{
-    .{ .short = 'q', .callback = quit }, // quit
-    .{ .short = 'Q', .callback = quit }, // quit
+    .{ .long = "quit", .callback = quit },
+    .{ .long = "exit", .callback = quit },
     .{ .long = "AOS", .callback = blindAck },
     .{ .long = "LOS", .callback = blindAck },
     .{ .short = 'P', .long = "set_pos", .callback = setPosition }, // azimuth: f64, elevation: f64