<br>
<br><font size=2><tt>vdr-bounces@linuxtv.org schrieb am 17.04.2006 22:26:03:<br>
<br>
&gt; <br>
&gt; &gt; I came across this with streamdev earlier on. What I did as a
<br>
&gt; quick bodge was <br>
&gt; &gt; rename the Active functions in streamdev-server to LActive in
<br>
&gt; &gt; streamdev-server.h:24 and streamdev-server.c:58.<br>
&gt; &gt; <br>
&gt; &gt; Not as clean as a proper renaming patch but worked for me!<br>
&gt; &gt; <br>
&gt; <br>
&gt; Actually, that was my first instinct as well.. to rename Active to
<br>
&gt; something else :)<br>
&gt; <br>
&gt; Although, this seems to work to.. (see attachments)<br>
&gt; <br>
&gt; BR.<br>
&gt; <br>
&gt; --- burn/burn.c.orig &nbsp; 2005-11-12 10:17:11.000000000 -0800<br>
&gt; +++ burn/burn.c &nbsp; 2006-04-17 13:10:40.000000000 -0700<br>
&gt; @@ -34,7 +34,7 @@<br>
&gt; &nbsp; &nbsp; &nbsp;virtual bool Start(void);<br>
&gt; &nbsp; &nbsp; &nbsp;virtual void Stop(void);<br>
&gt; &nbsp; &nbsp; &nbsp;virtual void Housekeeping(void);<br>
&gt; - &nbsp; &nbsp;virtual bool Active(void);<br>
&gt; + &nbsp; &nbsp;virtual cString Active(void);<br>
&gt; &nbsp; &nbsp; &nbsp;virtual const char *MainMenuEntry(void) { return
tr(MAINMENUENTRY); }<br>
&gt; &nbsp; &nbsp; &nbsp;virtual cOsdObject *MainMenuAction(void);<br>
&gt; &nbsp; &nbsp; &nbsp;virtual cMenuSetupPage *SetupMenu(void);<br>
&gt; @@ -193,9 +193,11 @@<br>
&gt; &nbsp; &nbsp; &nbsp;cBurnMenuBurn::Housekeeping();<br>
&gt; &nbsp;}<br>
&gt; &nbsp;<br>
&gt; -bool cPluginBurn::Active(void)<br>
&gt; +cString cPluginBurn::Active(void)<br>
&gt; &nbsp;{<br>
&gt; - &nbsp; &nbsp;return cBurnMenuBurn::JobActive();<br>
&gt; + &nbsp; &nbsp;if(cBurnMenuBurn::JobActive())<br>
&gt; + &nbsp; &nbsp; &nbsp; return tr(&quot;Burn DVD job active&quot;);<br>
&gt; + &nbsp; &nbsp; &nbsp; return NULL;<br>
&gt; &nbsp;}<br>
</tt></font>
<br><font size=2><tt>Rename would be quite wrong as burn and streamdev
are already using this new funktion.</tt></font>
<br><font size=2><tt>It was previously hidden in the patch section of both
plugins.</tt></font>
<br><font size=2><tt>Klaus just opened up the idea somewhat more to allow
for reporting the busy-reason by the plugin.</tt></font>
<br><font size=2><tt>The original idea of the burn/streamdev designer was
to just report it's busy.</tt></font>
<br>
<br><font size=2><tt>Active() had previously no real function when VDR
was not patched. </tt></font>
<br><font size=2><tt>VDR was ignoring the busy plugin and just killed it
:)</tt></font>
<br><font size=2><tt>Now VDR should politely ask wether to kill burn and
streamdev while they are busy.</tt></font>
<br>
<br><font size=2><tt>To make VDR master again you may have to remove some
double checks in your shutdown scripts for burn and streamdev.</tt></font>
<br><font size=2><tt>VDR can still overrule a busy plugin by manually aknoledging
a forced shutdown.</tt></font>
<br><font size=2><tt>Scripts do mostly not allow this.</tt></font>
<br><font size=2><tt>If again some script kiddy continously tries to hack
into my VDRs ssh port, my shutdown script complain always about some still
open ports. Would be nice to replace this with some mini-plugin still allowing
a forced shutdown without having to kill sshd before VDR accepts shutting
down ;-))</tt></font>
<br><font size=2><tt>May be it's a good idea now to move all the shutdown
scripts to some mini-plugin which still allowes a manual forced shutdown.</tt></font>
<br>
<br><font size=2><tt>regards &nbsp; &nbsp; &nbsp; &nbsp;Peter</tt></font>
<br>
<br><font size=2><tt>*********************************************************************************</tt></font>
<br><font size=2><tt>It still surprises me that the &quot;patch&quot; was
so quitely taken over by Klaus ;-)</tt></font>