<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="12" style="border: 1px #c9c399 solid; border-radius: 6px; -moz-border-radius: 6px; -webkit-border-radius: 6px;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="https://git.reviewboard.kde.org/r/128096/">https://git.reviewboard.kde.org/r/128096/</a>
</td>
</tr>
</table>
<br />
<p>
Ship it!
</p>
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Sounds OK. I guess it is ready for addition at Kaffeine.</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Answering your questions:</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;"><!--<member>qt5options(1)</member> only for application with a GUI FIXME use if we have an updated man page qt5options--></p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Kaffeine does accept qt5options. It doesn't do anything special to them, just whatever Qt5 implements.</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;"><!--<member>foo(1)</member> FIXME any ref to an other man page here ? vlc? or an utility to detect a tv card and gives outpout necessary to unse kaffeine?--></p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">For now, I don't think we need to point to VLC manpage, as there's nothing there that would help the user. Yet, we can mention at the manpage that it uses libVLC as video/audio play backend.</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">The only thing external that Kaffeine indirectly uses is this tree: https://git.linuxtv.org/dtv-scan-tables.git/</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">It contains the list of known channels per locality and per Satellite (for DVB-S). There's a developer's utility that generates a Kaffeine-specific file (scanfile.dvb) with uses such information. We run such tool from time to time, and we store the scanfile on a public repository. There's a button at the "Configura Television" screen that says: "Update scan data over Internet". When the user clicks on it, it downloads the latest file stored at: http://kaffeine.kde.org/scanfile.dvb.qz (I'm actually thinking that maybe the best would be to store it at linuxtv.org, and add there some automation to auto-create it when the upstream dtv-scan-tree changes).</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Unfortunately, we don't have a manfile for dtv-scan-tables yet - patches are welcome.</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Anyway, if the user has the need for a channel that it is not listed on Kaffeine, he needs to either use dvbv5-scan or w_scan to generate a new file (in the latter case, converted to the proper format using dvb-format-convert), and submit it to linux-media@vger.kernel.org, with "[PATCH] dtv-scan-tables: <some description>" at the subject, for us to add it at the tree and create a new scanfile.dvb.qz.</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">So, I guess we should mention the above procedure and point to the manpages for dvbv5-scan and w_scan.</p></pre>
<br />
<p>- Mauro Carvalho Chehab</p>
<br />
<p>On June 4th, 2016, 11:29 a.m. UTC, Burkhard Lück wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="12" style="border: 1px #888a85 solid; border-radius: 6px; -moz-border-radius: 6px; -webkit-border-radius: 6px;">
<tr>
<td>
<div>Review request for Documentation and Mauro Carvalho Chehab.</div>
<div>By Burkhard Lück.</div>
<p style="color: grey;"><i>Updated June 4, 2016, 11:29 a.m.</i></p>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt;">Repository: </b>
kaffeine
</div>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">add new man page
debian policy requires it anyway
this should replace the outdated sect in the handbook</p></pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Testing </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">passes checkXML5</p></pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>doc/CMakeLists.txt <span style="color: grey">(06f05fb)</span></li>
<li>doc/man-kaffeine.1.docbook <span style="color: grey">(PRE-CREATION)</span></li>
</ul>
<p><a href="https://git.reviewboard.kde.org/r/128096/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>