Bug 256 - Add sync/unsync command to CLI
: Add sync/unsync command to CLI
Status: RESOLVED FIXED
Product: Logitech Media Server
Classification: Unclassified
Component: CLI
: unspecified
: All All
: P4 enhancement (vote)
: ---
Assigned To: KDF
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2004-04-13 20:14 UTC by Jason Holtzapple
Modified: 2009-09-08 09:15 UTC (History)
1 user (show)

See Also:
Category: ---


Attachments
basic sync CLI (3.08 KB, patch)
2004-07-18 23:05 UTC, KDF
Details | Diff
new patch (3.08 KB, patch)
2004-07-19 17:55 UTC, KDF
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Jason Holtzapple 2004-04-13 20:14:58 UTC
It would be great if there was a way to sync/unsync a player via the CLI.
Comment 1 KDF 2004-07-18 23:05:55 UTC
Created attachment 74 [details]
basic sync CLI

simply allows one player to sync to another, or unsync.  added the docs to
explain it as well.  this cannot sync up more than 2, or allow joining an
existing sync group yet.
Comment 2 Blackketter Dean 2004-07-19 05:13:36 UTC
Looks reasonable, but one thing comes to mind... The "?" returns a human readable list, which may not 
be the most useful thing for the CLI.  

Also, this should allow, through multiple commands, syncing of more than two players, no?
Comment 3 KDF 2004-07-19 08:51:48 UTC
eventually, yes.
what woudl you prefer be returned? existing routines have human readable, or an
array of client object references.
Comment 4 KDF 2004-07-19 13:40:51 UTC
I think perhaps I was putting the two clients in the wrong order for the sync
call.  This was causing a subsequent sync command to unsync the previous.  The
CLI allows referencing client by the ID via getClient or by the index.  As this
is used more than once, do we think it might break something to allow index
reference to return from getClient?
Comment 5 KDF 2004-07-19 17:55:37 UTC
Created attachment 75 [details]
new patch

this one does allow multiple sync. 
? uses syncname to return the query, which is properly escaped. The only other
thing I could think of would be to parse that and return a list of ID's or
indexes.
Comment 6 KDF 2004-07-21 19:06:12 UTC
committed July 21, 2004 with updates to CLI docs. 

   <playerID> sync <playerindex|playerID> to sync
               <playerID> sync - to remove <playerID> from its group
               <playerID> sync ? to list playerIDs synced with that player
Comment 7 Chris Owens 2006-06-16 14:42:14 UTC
There are 536 bugs in the database with targets of '---' that were fixed prior to new year 2006.  I am setting them to targets of 6.2.1 to keep them from showing up in my queries.