[mythtv] [mythtv-commits] mythtv branch master updated by pbennett. v29-pre-305-gef7e594

[Peter Bennett]

On 02/10/2017 02:57 PM, Peter Bennett wrote:
> On 02/10/2017 12:57 PM, Piotr Oniszczuk wrote:
>> Peter,
>> I think empty field in setup screen should mean standard port. This 
>> is IIRC stated in help text on setup screen (leave blank is standard 
>> port is used).
>> It looks like new setup screen don’t touches port if port=0.
>> I suspect old setup screen puts port=0 when field is blank.
>> For new setup screen it is logical to not touch port if config.xml 
>> has it non-empty - but for smooth migrations from old to new I would 
>> add condition like:
>> if port=0 then port=3306
>>   I do test with deleting config.xml and restarting FE.
>> Result: when there is no config.xml - FE starts with country/language 
>> screen.
>> Selecting country/language and pressing save gives country/language 
>> screen again.
> Hi Piotr
>
> I know it will not work with blank port or port zero. That is an easy 
> fix to default to 3306. I will fix that.
Hi Piotr
I tested with version MythTV v0.28 (none of the startup changes). If I 
blank the port in config.xml
     <Port></Port>
Then when I start the frontend it goes immediately to the database setup 
screen, it does not default to the correct port. I don't think I should 
change it. It likely has always been this way.
Peter
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.mythtv.org/pipermail/mythtv-dev/attachments/20170210/2486a810/attachment.html>