<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 />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On June 7th, 2016, 11:14 a.m. UTC, <b>Mauro Carvalho Chehab</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<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>
</blockquote>
</blockquote>
<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;">Change done for Kaffeine 2.0.3: Kaffeine will now download the scanfile from: https://linuxtv.org/downloads/dtv-scan-tables/kaffeine/</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">There is a script at linuxtv.org that automatically re-generate the file, once by day, when new patches are applied at the dtv-scan-tables git tree.</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">So, all the user needs to do, when either new channels are added on some City, or when new Satellite beams are added, is to run either dvbv5-scan or w_scan, to generate the new scan file and submit to linux-media@vger.kernel.org. After reviewed by the maintainer, the new patch will be added and will be ready to be upload on the next day, by clicking at the "Update scan data over Internet" button at the "Configure Television" menu.</p></pre>
<br />
<p>- Mauro</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>