[mythtv] Ticket #10722: mythshutdown -l isn't recognised

[John Veness]

Hi,

Regarding ticket 10722 <http://code.mythtv.org/trac/ticket/10722> which 
has been Won't Fixed, I'd like to ask the some further discussion, and 
opinion of other devs.

 From the user point of view, "mythshutdown -l" always used to work to 
lock shutdown and no longer does in 0.25. I don't think this was an 
intentional change, based on the unchanged code in 
mythtv/programs/mythshutdown/commandlineparser.cpp which still lists 
"-l" as a valid alias to "--lock".

As far as I can tell, the only reason it no longer works in because of 
the blanket prohibition of -l/--logfile added in 
mythtv/libs/libmythbase/mythcommandlineparser.cpp. This blanket change 
makes sense for many of the myth commands but doesn't make sense for 
mythshutdown which never used -l to mean --logfile.

I realise that "mythshutdown --lock" still works, but years of muscle 
memory and user scripts make it annoying to change, especially as there 
seems to be no good reason to change.

Can there not be an exception made such the "-l" can still work for 
mythshutdown (where it never meant "--logfile") but prohibit it for 
other myth commands (where it did used to mean "--logfile")? This may 
mean moving code out of 
mythtv/libs/libmythbase/mythcommandlineparser.cpp and into individual 
command parsers, where it is appropriate.

Cheers,

John

-- 
John Veness, MythTV user, UK, DVB-T