[Buildroot] [PATCH v3] dvbsnoop: new package

[Buildroot] [PATCH v3] dvbsnoop: new package

[Carsten Schoenert]

Hello,

this is the 3rd version of the new package dvbsnoop. I've done more
rework as suggested by Thomas. If still something wron just let me
know.

Regards
Carsten

[Buildroot] [PATCH] dvbsnoop: new package dvbsnoop

[Carsten Schoenert]

From: Carsten Schoenert <c.schoenert@t-online.de>

dvbsnoop is a DVB/MPEG stream analyzer. It's designed to debug, dump or
view the digital stream info available via

    satellite   -> DVB-S, DVB-S2
    cable       -> DVB-C, DVB-C2
    terrestrial -> DVB-T, DVB-T2

dvbsnoop can be used on any digital settopbox that running on linux and
provides the DVB APIv3. It's also usable on classical PCs that contains
a DVB hardware (PCI Card or USB Plug).

Signed-off-by: Carsten Schoenert <c.schoenert@gmail.com>
---
Changes for v3: 
    - reworking the helping text for the package as requested by
      Thomas Petazzoni
    - change _LICENSE_FILE -> _LICENSE_FILES

Changes for v2: 
    - moved from 'Audio and video ...' to 'Hardware Handling' as  
      suggested by Yann E. Morin
    - removing _SITE_METHOD from the Makefile

 package/Config.in            |    1 +
 package/dvbsnoop/Config.in   |   21 +++++++++++++++++++++
 package/dvbsnoop/dvbsnoop.mk |   12 ++++++++++++
 3 files changed, 34 insertions(+)
 create mode 100644 package/dvbsnoop/Config.in
 create mode 100644 package/dvbsnoop/dvbsnoop.mk

diff --git a/package/Config.in b/package/Config.in
index fcc2480..126f450 100644
--- a/package/Config.in
+++ b/package/Config.in
@@ -218,6 +218,7 @@ endif
 source "package/dmidecode/Config.in"
 source "package/dmraid/Config.in"
 source "package/dvb-apps/Config.in"
+source "package/dvbsnoop/Config.in"
 source "package/eeprog/Config.in"
 source "package/evtest/Config.in"
 source "package/flashrom/Config.in"
diff --git a/package/dvbsnoop/Config.in b/package/dvbsnoop/Config.in
new file mode 100644
index 0000000..a0ba834
--- /dev/null
+++ b/package/dvbsnoop/Config.in
@@ -0,0 +1,21 @@
+config BR2_PACKAGE_DVBSNOOP
+	bool "dvbsnoop"
+	help
+	  Dvbsnoop is just a simple tool to analyze, view or debug a transport
+	  stream (TS), program elementary stream (PES) or even a program stream
+	  (PS). Dvbsnoop is trying to get input mostly direct from a frontend
+	  (satellite or cable tuner inside a set-top box for example) via DVB
+	  APIv3 to view into a TS, or a demux to view PES or PS (also mostly
+	  inside the set-top box or even PCI Card or USB Plug) but also from a
+	  file (recorded TS).
+	  Dvbsnoop is working well on SD transport streams but may have some
+	  limited functionality on HD transport streams (DVB-{C,S,T}2 because
+	  of missing knowledge how to handle it.
+
+	  Dvbsnoop itself doesn?t do the tuning on the frontend, this has to be
+	  done by external helpers or manually driver call.
+	  The dvbsnoop tool was written while developing the software Neutrino on
+	  the set-top box DBox2.
+
+	  http://dvbsnoop.sourceforge.net
+	  http://en.wikipedia.org/wiki/DBox2
diff --git a/package/dvbsnoop/dvbsnoop.mk b/package/dvbsnoop/dvbsnoop.mk
new file mode 100644
index 0000000..dd62e8c
--- /dev/null
+++ b/package/dvbsnoop/dvbsnoop.mk
@@ -0,0 +1,12 @@
+#############################################################
+#
+# dvbsnoop
+#
+#############################################################
+
+DVBSNOOP_VERSION        = 1.4.50
+DVBSNOOP_SITE           = http://downloads.sourceforge.net/project/dvbsnoop/dvbsnoop/dvbsnoop-$(DVBSNOOP_VERSION)
+DVBSNOOP_LICENSE        = GPLv2
+DVBSNOOP_LICENSE_FILES  = COPYING
+
+$(eval $(autotools-package))
--
1.7.10.4

[Buildroot] [PATCH] dvbsnoop: new package dvbsnoop

[Peter Korsgaard]

>>>>> "Carsten" == Carsten Schoenert <c.schoenert@gmail.com> writes:

Hi,

 Carsten> From: Carsten Schoenert <c.schoenert@t-online.de>

 Carsten> dvbsnoop is a DVB/MPEG stream analyzer. It's designed to
 Carsten> debug, dump or view the digital stream info available via

 Carsten>     satellite   -> DVB-S, DVB-S2
 Carsten>     cable       -> DVB-C, DVB-C2
 Carsten>     terrestrial -> DVB-T, DVB-T2


 Carsten> +config BR2_PACKAGE_DVBSNOOP
 Carsten> +	bool "dvbsnoop"

It needs largefile support, so I've added the needed dependency /
comment on BR2_LARGEFILE.


 Carsten> +	  Dvbsnoop itself doesn?t do the tuning on the frontend, this has to be

Please stick to ascii for Config.in, menuconfig doesn't like utf-8.

Committed with those fixes, thanks.

-- 
Bye, Peter Korsgaard

[Buildroot] [PATCH] packages/dvbsnoop: new package dvbsnoop

[Yann E. MORIN]

Carsten, All,

On Wednesday 09 January 2013 Carsten Schoenert wrote:
> From: Carsten Schoenert <c.schoenert@t-online.de>
> 
> dvbsnoop is a DVB/MPEG stream analyzer. It's designed to debug, dump or
> view the digital stream info available via

Hmm. Interesting! :-)

> diff --git a/package/multimedia/Config.in b/package/multimedia/Config.in
> index 273d9bc..07b4f1a 100644
> --- a/package/multimedia/Config.in
> +++ b/package/multimedia/Config.in
> @@ -2,6 +2,7 @@ menu "Audio and video applications"
>  source "package/multimedia/alsa-utils/Config.in"
>  source "package/multimedia/aumix/Config.in"
>  source "package/multimedia/bellagio/Config.in"
> +source "package/multimedia/dvbsnoop/Config.in"

There's already dvb-apps that deals with DVB stuff, and it is not in the
multimedia/ sub-dir, and is also in the "Hardware handling" sub-menu.

I think dvb-apps and dvbsnoop should be in the same sub-menu, and
"hardware handling" seems (to me) a better place than "Multimedia".

[--SNIP--]
> diff --git a/package/multimedia/dvbsnoop/dvbsnoop.mk b/package/multimedia/dvbsnoop/dvbsnoop.mk
> new file mode 100644
> index 0000000..a3ff468
> --- /dev/null
> +++ b/package/multimedia/dvbsnoop/dvbsnoop.mk
> @@ -0,0 +1,13 @@
> +#############################################################
> +#
> +# dvbsnoop
> +#
> +#############################################################
> +
> +DVBSNOOP_VERSION        = 1.4.50
> +DVBSNOOP_SITE           = http://downloads.sourceforge.net/project/dvbsnoop/dvbsnoop/dvbsnoop-$(DVBSNOOP_VERSION)
> +DVBSNOOP_SITE_METHOD    = gz

'gz' is not a valid _SITE_METHOD.
Since you're downloading a tarball, then you should not set _SITE_METHOD.

Regards,
Yann E. MORIN.

-- 
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 223 225 172 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'

[Buildroot] [PATCH v2] packages/dvbsnoop: new package dvbsnoop

[Carsten Schoenert]

Hello Yann, All,

thanks for your reply.
I was searching for the right place and after some thinking about Video
and Audio I had it placed there, but yes, it was not the best place, and
after it's a system tool it's o.k. to put in the same section there
dvb-apps are placed.

The _SITE_METHOD comes just from copy & paste :) , don't know which
package i was using, thougt it was a generic-package but dvbsnoop is a
autotool package, but that's reviewing are for. :) Thx.

So here is the reworked version of the patch.

Regards
Carsten

[Buildroot] [PATCH] dvbsnoop: new package dvbsnoop

[Carsten Schoenert]

From: Carsten Schoenert <c.schoenert@t-online.de>

dvbsnoop is a DVB/MPEG stream analyzer. It's designed to debug, dump or
view the digital stream info available via

    satellite   -> DVB-S, DVB-S2
    cable       -> DVB-C, DVB-C2
    terrestrial -> DVB-T, DVB-T2

dvbsnoop can be used on any digital settopbox that running on linux and
provides the DVB APIv3. It's also usable on classical PCs that contains
a DVB hardware (PCI Card or USB Plug).

Signed-off-by: Carsten Schoenert <c.schoenert@gmail.com>
---
Changes for v2:
    - moved from 'Audio and video ...' to 'Hardware Handling' as 
      suggested by Yann
    - Removing from _SITE_METHOD from the Makefile

 package/Config.in            |    1 +
 package/dvbsnoop/Config.in   |    9 +++++++++
 package/dvbsnoop/dvbsnoop.mk |   12 ++++++++++++
 3 files changed, 22 insertions(+)
 create mode 100644 package/dvbsnoop/Config.in
 create mode 100644 package/dvbsnoop/dvbsnoop.mk

diff --git a/package/Config.in b/package/Config.in
index bd1db6b..84a53a2 100644
--- a/package/Config.in
+++ b/package/Config.in
@@ -218,6 +218,7 @@ endif
 source "package/dmidecode/Config.in"
 source "package/dmraid/Config.in"
 source "package/dvb-apps/Config.in"
+source "package/dvbsnoop/Config.in"
 source "package/eeprog/Config.in"
 source "package/evtest/Config.in"
 source "package/flashrom/Config.in"
diff --git a/package/dvbsnoop/Config.in b/package/dvbsnoop/Config.in
new file mode 100644
index 0000000..afd0bc5
--- /dev/null
+++ b/package/dvbsnoop/Config.in
@@ -0,0 +1,9 @@
+config BR2_PACKAGE_DVBSNOOP
+	bool "dvbsnoop"
+	help
+	  DVB analyzer / MPEG analyzer program to monitor, analyze, debug,
+	  dump or view dvb/mpeg/dsm-cc/mhp stream information (TS, PES,
+	  SECTION) (e.g. digital television) send via satellite, cable or
+	  terrestrial.
+
+	  http://dvbsnoop.sourceforge.net/
diff --git a/package/dvbsnoop/dvbsnoop.mk b/package/dvbsnoop/dvbsnoop.mk
new file mode 100644
index 0000000..e987dc6
--- /dev/null
+++ b/package/dvbsnoop/dvbsnoop.mk
@@ -0,0 +1,12 @@
+#############################################################
+#
+# dvbsnoop
+#
+#############################################################
+
+DVBSNOOP_VERSION        = 1.4.50
+DVBSNOOP_SITE           = http://downloads.sourceforge.net/project/dvbsnoop/dvbsnoop/dvbsnoop-$(DVBSNOOP_VERSION)
+DVBSNOOP_LICENSE        = GPLv2
+DVBSNOOP_LICENSE_FILE   = COPYING
+
+$(eval $(autotools-package))
-- 
1.7.10.4

[Buildroot] [PATCH] dvbsnoop: new package dvbsnoop

[Thomas Petazzoni]

Dear Carsten Schoenert,

On Wed,  9 Jan 2013 19:56:12 +0100, Carsten Schoenert wrote:

> +	  DVB analyzer / MPEG analyzer program to monitor, analyze, debug,
> +	  dump or view dvb/mpeg/dsm-cc/mhp stream information (TS, PES,
> +	  SECTION) (e.g. digital television) send via satellite, cable or
> +	  terrestrial.

Could you include a bit more details here? You have a lot of nice
details in your commit log, what about copy/pasting them here?


> +DVBSNOOP_LICENSE_FILE   = COPYING

Should be _LICENSE_FILES even if there is only one file.

Thanks,

Thomas
-- 
Thomas Petazzoni, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com

[Buildroot] [PATCH] dvbsnoop: new package dvbsnoop

[Carsten Schoenert]

Hello Thomas,

Am 09.01.2013 21:07, schrieb Thomas Petazzoni:
>> +	  DVB analyzer / MPEG analyzer program to monitor, analyze, debug,
>> +	  dump or view dvb/mpeg/dsm-cc/mhp stream information (TS, PES,
>> +	  SECTION) (e.g. digital television) send via satellite, cable or
>> +	  terrestrial.
> 
> Could you include a bit more details here? You have a lot of nice
> details in your commit log, what about copy/pasting them here?

I don't know what you exactly mean. Do you want more details in the
commit message or more explanation here on the mailing list?

I thought while everybody can read the commit message I don't have to
explain it twice. :-/
And the webpage of dvbsnoop is written very well and the author can
explain it's work better than me (I think).

So what is dvbsnoop?
Dvbsnoop is just a simple tool to analyze, view or debug a transport
stream (TS), program elementary stream (PES) or even a program stream
(PS). This input comes mostly direct from a frontend (satellite or cable
tuner inside a set-top box for example) via DVB APIv3 to view into a TS,
a demux to view PES or PS (also mostly inside the set-top box or even
PCI Card) or even from a file (recorded TS).
The device files are differ from hardware to hardware this can be given
as parameter to dvbsnoop.

Dvbsnoop itself doesn?t do the tuning on the frontend, this has to be
done by external helpers or manually driver call. It just takes the data
from it an do the parsing.

The dvbsnoop tool was written while developing the software Neutrino on
the set-top box DBox2 [1]. It was useful to see if the GUI Neutrino was
working correct with the digital data from the frontend or from a NFS Share.
In the last time Rainer Scherg doesn't has worked on dvbsnoop because
it's doing what it should. Some parts are re workable, the complete
latest standards for integrating the MPEG4 standards (DVB-[S,C,T]*2*)
are just "it works". Some flags for this dvbsnoop don?t know right now.

> Should be _LICENSE_FILES even if there is only one file.

O.K. Will rework this.

[1] http://en.wikipedia.org/wiki/DBox2

Regards
Carsten

[Buildroot] [PATCH] dvbsnoop: new package dvbsnoop

[Thomas Petazzoni]

Dear Carsten Schoenert,

On Thu, 10 Jan 2013 20:27:03 +0100, Carsten Schoenert wrote:

> I don't know what you exactly mean. Do you want more details in the
> commit message or more explanation here on the mailing list?

Sorry, I explained improperly my request. Your commit log is absolutely
perfect and gives a very nice description of what dvbsnoop is.

The only minor comment I had is that the help text of your
BR2_PACKAGE_DVBSNOOP option (which corresponds to the package you're
adding) doesn't have as many details as your commit log as.

So I was simply suggesting to copy/paste a few more details from your
commit log into the help text of the BR2_PACKAGE_DVBSNOOP option.
That's all!

> I thought while everybody can read the commit message I don't have to
> explain it twice. :-/

Sure, sure. See my explanation above. I was probably unclear in my
previous e-mail.

Thanks a lot for your contribution, and sorry for my previous unclear
e-mail.

Best regards,

Thomas
-- 
Thomas Petazzoni, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com

[Buildroot] [PATCH] dvbsnoop: new package dvbsnoop

[Carsten Schoenert]

Hello Thomas,

Am 10.01.2013 20:45, schrieb Thomas Petazzoni:
> Sure, sure. See my explanation above. I was probably unclear in my
> previous e-mail.
> 
> Thanks a lot for your contribution, and sorry for my previous unclear
> e-mail.

No problem at all. I understand what you want to say and I also like to
see not just "BR_PACKAGE_BLUE  --> It lights blue" (in backing to a
famous scene in one of the Rambo films :P ).
I will grabbing your suggestions and add a more precise help text later.
Unfortunately many other target needs this rework also. :-)

Regards
Carsten