<div dir="ltr">I think MythTV has always been welcoming of code contributions however I think the best method to do so is:<br>Create the PR at github: <a href="https://github.com/MythTV/mythtv/pulls">https://github.com/MythTV/mythtv/pulls</a><div>Create the corresponding ticket: <a href="https://code.mythtv.org/trac">https://code.mythtv.org/trac</a></div><div><br></div><div>This is largely what is outlined at: <a href="https://github.com/MythTV/mythtv">https://github.com/MythTV/mythtv</a></div><div>Sample: <a href="https://github.com/MythTV/mythtv/pull/121">https://github.com/MythTV/mythtv/pull/121</a></div><div><br></div><div>Note it does sometimes take awhile, but github at least makes sure it can patch cleanly into master.</div></div><div class="gmail_extra"><br clear="all"><div><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><br><div>~mitch</div></div></div></div>
<br><div class="gmail_quote">On Fri, Oct 13, 2017 at 7:44 AM, Piotr Oniszczuk <span dir="ltr"><<a href="mailto:piotr.oniszczuk@gmail.com" target="_blank">piotr.oniszczuk@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Guys,<br>
<br>
I see ongoing process of opening of myth to new code contributions (which is finally great!)<br>
<br>
Will You consider to back port some of MiniMyth2 patches to myth tv master code?<br>
<br>
I’m using mm2 patch set in my production system without any issues since years while enjoying greater functionality&flexibility.<br>
Also whole mm2 community runs them with great success I believe.<br>
<br>
If You think it is worth to do - we can discuss patch-by-patch here and dev team can decide - is it worth to merge or not.<br>
<br>
So lets try with first candidate:<br>
<br>
<a href="https://github.com/warpme/minimyth2/blob/master/script/myth-master/mythtv/files/0324-disable-caches-pruning.patch" rel="noreferrer" target="_blank">https://github.com/warpme/<wbr>minimyth2/blob/master/script/<wbr>myth-master/mythtv/files/0324-<wbr>disable-caches-pruning.patch</a><br>
<br>
Patch allows to disable cache pruning by setting UIDiskCacheDays to -1.<br>
<br>
Why useful:<br>
If frontend is used as appliance with sleep/resume regime with mode where mythfrontend process is exited at sleep and started at appliance resume - cache pruning can considerable increase mythfrontend start time (on my ION2 it is 11->19..21sec).<br>
<br>
Sure - this we can overcome by moving cache pruning to separate thread (which is also good idea IMHO)<br>
<br>
But I have another argument to turn-off pruning some cases:<br>
<br>
lets see 2 cases:<br>
<br>
a\ no pruning:<br>
(-)bigger disk usage<br>
(+)no issue with lagging some images on UI (no cache)<br>
(+)quick, predictable start of frontend<br>
<br>
b\pruning<br>
(+)space on hdd saved<br>
(-)noticeable UE impact when playing with UI<br>
(-)mythfrontend startup sometimes so longer(*) than user thinks it hanged<br>
<br>
For me in mm2 b\ wins clearly.<br>
<br>
Patch allows to user to decide a\ or b\<br>
<br>
(*) I have appliances where mythfrontend wasn’t restarted by weeks. Any restart triggers long (10-20sec) of cache pruning.<br>
<br>
br<br>
<br>
______________________________<wbr>_________________<br>
mythtv-dev mailing list<br>
<a href="mailto:mythtv-dev@mythtv.org">mythtv-dev@mythtv.org</a><br>
<a href="http://lists.mythtv.org/mailman/listinfo/mythtv-dev" rel="noreferrer" target="_blank">http://lists.mythtv.org/<wbr>mailman/listinfo/mythtv-dev</a><br>
<a href="http://wiki.mythtv.org/Mailing_List_etiquette" rel="noreferrer" target="_blank">http://wiki.mythtv.org/<wbr>Mailing_List_etiquette</a><br>
MythTV Forums: <a href="https://forum.mythtv.org" rel="noreferrer" target="_blank">https://forum.mythtv.org</a></blockquote></div><br></div>