[FFmpeg-devel] [PATCH 6/6] psmf: add FATE tests

2018-01-05 Thread misty
From: Misty De Meo 

---
 tests/Makefile  |  1 +
 tests/fate/psmf.mak | 23 +++
 2 files changed, 24 insertions(+)
 create mode 100644 tests/fate/psmf.mak

diff --git a/tests/Makefile b/tests/Makefile
index fd3713fe81..c569091fcb 100644
--- a/tests/Makefile
+++ b/tests/Makefile
@@ -165,6 +165,7 @@ include $(SRC_PATH)/tests/fate/pcm.mak
 include $(SRC_PATH)/tests/fate/pixlet.mak
 include $(SRC_PATH)/tests/fate/probe.mak
 include $(SRC_PATH)/tests/fate/prores.mak
+include $(SRC_PATH)/tests/fate/psmf.mak
 include $(SRC_PATH)/tests/fate/qt.mak
 include $(SRC_PATH)/tests/fate/qtrle.mak
 include $(SRC_PATH)/tests/fate/real.mak
diff --git a/tests/fate/psmf.mak b/tests/fate/psmf.mak
new file mode 100644
index 00..d1378c8ac7
--- /dev/null
+++ b/tests/fate/psmf.mak
@@ -0,0 +1,23 @@
+#
+# Test detecting ATRAC-3 audio in Sony MPEG files
+#
+
+PSMF_PROBE_COMMAND = \
+   ffprobe$(PROGSSUF)$(EXESUF) -show_entries stream=codec_name \
+   -select_streams a -print_format default=noprint_wrappers=1 -bitexact -v 0
+
+FATE_PSMF_FFPROBE = fate-psmf-probe-6 \
+   fate-psmf-probe-EV01_01_0500D
+
+fate-psmf-probe-6: SRC = $(TARGET_SAMPLES)/psmf/6.MPS
+fate-psmf-probe-6: CMD = run $(PSMF_PROBE_COMMAND) -i "$(SRC)"
+fate-psmf-probe-6: CMP = oneline
+fate-psmf-probe-6: REF = codec_name=atrac3p
+fate-psmf-probe-EV01_01_0500D: SRC = $(TARGET_SAMPLES)/psmf/EV01_01_0500D.PMF
+fate-psmf-probe-EV01_01_0500D: CMD = run $(PSMF_PROBE_COMMAND) -i "$(SRC)"
+fate-psmf-probe-EV01_01_0500D: CMP = oneline
+fate-psmf-probe-EV01_01_0500D: REF = codec_name=atrac3p
+
+FATE_SAMPLES_FFPROBE += $(FATE_PSMF_FFPROBE)
+
+fate-psmf: $(FATE_PSMF_FFPROBE)
-- 
2.15.1

___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel


Re: [FFmpeg-devel] [PATCH 6/6] psmf: add FATE tests

2017-12-31 Thread Michael Niedermayer
On Sun, Dec 31, 2017 at 05:46:07PM +0800, mi...@brew.sh wrote:
> From: Misty De Meo 
> 
> ---
>  tests/Makefile  |  1 +
>  tests/fate/psmf.mak | 23 +++
>  2 files changed, 24 insertions(+)
>  create mode 100644 tests/fate/psmf.mak
> 
> diff --git a/tests/Makefile b/tests/Makefile
> index fd3713fe81..c569091fcb 100644
> --- a/tests/Makefile
> +++ b/tests/Makefile
> @@ -165,6 +165,7 @@ include $(SRC_PATH)/tests/fate/pcm.mak
>  include $(SRC_PATH)/tests/fate/pixlet.mak
>  include $(SRC_PATH)/tests/fate/probe.mak
>  include $(SRC_PATH)/tests/fate/prores.mak
> +include $(SRC_PATH)/tests/fate/psmf.mak
>  include $(SRC_PATH)/tests/fate/qt.mak
>  include $(SRC_PATH)/tests/fate/qtrle.mak
>  include $(SRC_PATH)/tests/fate/real.mak
> diff --git a/tests/fate/psmf.mak b/tests/fate/psmf.mak
> new file mode 100644
> index 00..0d83daad49
> --- /dev/null
> +++ b/tests/fate/psmf.mak
> @@ -0,0 +1,23 @@
> +#
> +# Test detecting ATRAC-3 audio in Sony MPEG files
> +#
> +
> +PROBE_CODEC_NAME_COMMAND = \
> + ffprobe$(PROGSSUF)$(EXESUF) -show_entries stream=codec_name \
> + -select_streams a -print_format default=noprint_wrappers=1 -bitexact -v > 0
> +
> +FATE_PSMF_FFPROBE = fate-psmf-probe-6 \
> + fate-psmf-probe-EV01_01_0500D
> +
> +fate-psmf-probe-6: SRC = $(TARGET_SAMPLES)/psmf/6.MPS
> +fate-psmf-probe-6: CMD = run $(PROBE_CODEC_NAME_COMMAND) -i "$(SRC)"
> +fate-psmf-probe-6: CMP = oneline
> +fate-psmf-probe-6: REF = codec_name=atrac3p
> +fate-psmf-probe-EV01_01_0500D: SRC = $(TARGET_SAMPLES)/psmf/EV01_01_0500D.PMF
> +fate-psmf-probe-EV01_01_0500D: CMD = run $(PROBE_CODEC_NAME_COMMAND) -i 
> "$(SRC)"
> +fate-psmf-probe-EV01_01_0500D: CMP = oneline
> +fate-psmf-probe-EV01_01_0500D: REF = codec_name=atrac3p
> +
> +FATE_SAMPLES_FFPROBE += $(FATE_PSMF_FFPROBE)
> +
> +fate-psmf: $(FATE_PSMF_FFPROBE)

This breaks:
fate-mpegts-probe-latm

i guess the PROBE_CODEC_NAME_COMMAND is what is causing this

[...]
-- 
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

"You are 36 times more likely to die in a bathtub than at the hands of a
terrorist. Also, you are 2.5 times more likely to become a president and
2 times more likely to become an astronaut, than to die in a terrorist
attack." -- Thoughty2



signature.asc
Description: PGP signature
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel


[FFmpeg-devel] [PATCH 6/6] psmf: add FATE tests

2017-12-31 Thread misty
From: Misty De Meo 

---
 tests/Makefile  |  1 +
 tests/fate/psmf.mak | 23 +++
 2 files changed, 24 insertions(+)
 create mode 100644 tests/fate/psmf.mak

diff --git a/tests/Makefile b/tests/Makefile
index fd3713fe81..c569091fcb 100644
--- a/tests/Makefile
+++ b/tests/Makefile
@@ -165,6 +165,7 @@ include $(SRC_PATH)/tests/fate/pcm.mak
 include $(SRC_PATH)/tests/fate/pixlet.mak
 include $(SRC_PATH)/tests/fate/probe.mak
 include $(SRC_PATH)/tests/fate/prores.mak
+include $(SRC_PATH)/tests/fate/psmf.mak
 include $(SRC_PATH)/tests/fate/qt.mak
 include $(SRC_PATH)/tests/fate/qtrle.mak
 include $(SRC_PATH)/tests/fate/real.mak
diff --git a/tests/fate/psmf.mak b/tests/fate/psmf.mak
new file mode 100644
index 00..0d83daad49
--- /dev/null
+++ b/tests/fate/psmf.mak
@@ -0,0 +1,23 @@
+#
+# Test detecting ATRAC-3 audio in Sony MPEG files
+#
+
+PROBE_CODEC_NAME_COMMAND = \
+   ffprobe$(PROGSSUF)$(EXESUF) -show_entries stream=codec_name \
+   -select_streams a -print_format default=noprint_wrappers=1 -bitexact -v 0
+
+FATE_PSMF_FFPROBE = fate-psmf-probe-6 \
+   fate-psmf-probe-EV01_01_0500D
+
+fate-psmf-probe-6: SRC = $(TARGET_SAMPLES)/psmf/6.MPS
+fate-psmf-probe-6: CMD = run $(PROBE_CODEC_NAME_COMMAND) -i "$(SRC)"
+fate-psmf-probe-6: CMP = oneline
+fate-psmf-probe-6: REF = codec_name=atrac3p
+fate-psmf-probe-EV01_01_0500D: SRC = $(TARGET_SAMPLES)/psmf/EV01_01_0500D.PMF
+fate-psmf-probe-EV01_01_0500D: CMD = run $(PROBE_CODEC_NAME_COMMAND) -i 
"$(SRC)"
+fate-psmf-probe-EV01_01_0500D: CMP = oneline
+fate-psmf-probe-EV01_01_0500D: REF = codec_name=atrac3p
+
+FATE_SAMPLES_FFPROBE += $(FATE_PSMF_FFPROBE)
+
+fate-psmf: $(FATE_PSMF_FFPROBE)
-- 
2.15.1

___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel


Re: [FFmpeg-devel] [PATCH 6/6] psmf: add FATE tests

2017-12-26 Thread Michael Niedermayer
On Mon, Dec 25, 2017 at 11:32:50PM +, Rostislav Pehlivanov wrote:
> On 25 December 2017 at 23:27, Michael Niedermayer 
> wrote:
> 
> > On Mon, Dec 25, 2017 at 10:28:34AM +0800, mi...@brew.sh wrote:
> > > From: Misty De Meo 
> > >
> > > ---
> > >  tests/Makefile  |  1 +
> > >  tests/fate/psmf.mak | 23 +++
> > >  2 files changed, 24 insertions(+)
> > >  create mode 100644 tests/fate/psmf.mak
> > >
> > > diff --git a/tests/Makefile b/tests/Makefile
> > > index fd3713fe81..c569091fcb 100644
> > > --- a/tests/Makefile
> > > +++ b/tests/Makefile
> > > @@ -165,6 +165,7 @@ include $(SRC_PATH)/tests/fate/pcm.mak
> > >  include $(SRC_PATH)/tests/fate/pixlet.mak
> > >  include $(SRC_PATH)/tests/fate/probe.mak
> > >  include $(SRC_PATH)/tests/fate/prores.mak
> > > +include $(SRC_PATH)/tests/fate/psmf.mak
> > >  include $(SRC_PATH)/tests/fate/qt.mak
> > >  include $(SRC_PATH)/tests/fate/qtrle.mak
> > >  include $(SRC_PATH)/tests/fate/real.mak
> > > diff --git a/tests/fate/psmf.mak b/tests/fate/psmf.mak
> > > new file mode 100644
> > > index 00..0d83daad49
> > > --- /dev/null
> > > +++ b/tests/fate/psmf.mak
> > > @@ -0,0 +1,23 @@
> > > +#
> > > +# Test detecting ATRAC-3 audio in Sony MPEG files
> > > +#
> > > +
> > > +PROBE_CODEC_NAME_COMMAND = \
> > > + ffprobe$(PROGSSUF)$(EXESUF) -show_entries stream=codec_name \
> > > + -select_streams a -print_format default=noprint_wrappers=1
> > -bitexact -v 0
> > > +
> > > +FATE_PSMF_FFPROBE = fate-psmf-probe-6 \
> > > + fate-psmf-probe-EV01_01_0500D
> > > +
> > > +fate-psmf-probe-6: SRC = $(TARGET_SAMPLES)/psmf/6.MPS
> > > +fate-psmf-probe-6: CMD = run $(PROBE_CODEC_NAME_COMMAND) -i "$(SRC)"
> > > +fate-psmf-probe-6: CMP = oneline
> > > +fate-psmf-probe-6: REF = codec_name=atrac3p
> > > +fate-psmf-probe-EV01_01_0500D: SRC = $(TARGET_SAMPLES)/psmf/EV01_
> > 01_0500D.PMF
> > > +fate-psmf-probe-EV01_01_0500D: CMD = run $(PROBE_CODEC_NAME_COMMAND)
> > -i "$(SRC)"
> > > +fate-psmf-probe-EV01_01_0500D: CMP = oneline
> > > +fate-psmf-probe-EV01_01_0500D: REF = codec_name=atrac3p
> >
> > where are the sample files ?
> > did i miss a upload request ?
> >
> > [...]
> > --
> > Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
> >
> > I have never wished to cater to the crowd; for what I know they do not
> > approve, and what they approve I do not know. -- Epicurus
> >
> > ___
> > ffmpeg-devel mailing list
> > ffmpeg-devel@ffmpeg.org
> > http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
> >
> >
> They're in another email:
> http://ffmpeg.org/pipermail/ffmpeg-devel/2017-December/222885.html

Missed these
one is 512k one 1.7m, these seem a bit large for just testing filetype probing
is this intended ?


[...]
-- 
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

If a bugfix only changes things apparently unrelated to the bug with no
further explanation, that is a good sign that the bugfix is wrong.


signature.asc
Description: PGP signature
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel


Re: [FFmpeg-devel] [PATCH 6/6] psmf: add FATE tests

2017-12-25 Thread Rostislav Pehlivanov
On 25 December 2017 at 23:27, Michael Niedermayer 
wrote:

> On Mon, Dec 25, 2017 at 10:28:34AM +0800, mi...@brew.sh wrote:
> > From: Misty De Meo 
> >
> > ---
> >  tests/Makefile  |  1 +
> >  tests/fate/psmf.mak | 23 +++
> >  2 files changed, 24 insertions(+)
> >  create mode 100644 tests/fate/psmf.mak
> >
> > diff --git a/tests/Makefile b/tests/Makefile
> > index fd3713fe81..c569091fcb 100644
> > --- a/tests/Makefile
> > +++ b/tests/Makefile
> > @@ -165,6 +165,7 @@ include $(SRC_PATH)/tests/fate/pcm.mak
> >  include $(SRC_PATH)/tests/fate/pixlet.mak
> >  include $(SRC_PATH)/tests/fate/probe.mak
> >  include $(SRC_PATH)/tests/fate/prores.mak
> > +include $(SRC_PATH)/tests/fate/psmf.mak
> >  include $(SRC_PATH)/tests/fate/qt.mak
> >  include $(SRC_PATH)/tests/fate/qtrle.mak
> >  include $(SRC_PATH)/tests/fate/real.mak
> > diff --git a/tests/fate/psmf.mak b/tests/fate/psmf.mak
> > new file mode 100644
> > index 00..0d83daad49
> > --- /dev/null
> > +++ b/tests/fate/psmf.mak
> > @@ -0,0 +1,23 @@
> > +#
> > +# Test detecting ATRAC-3 audio in Sony MPEG files
> > +#
> > +
> > +PROBE_CODEC_NAME_COMMAND = \
> > + ffprobe$(PROGSSUF)$(EXESUF) -show_entries stream=codec_name \
> > + -select_streams a -print_format default=noprint_wrappers=1
> -bitexact -v 0
> > +
> > +FATE_PSMF_FFPROBE = fate-psmf-probe-6 \
> > + fate-psmf-probe-EV01_01_0500D
> > +
> > +fate-psmf-probe-6: SRC = $(TARGET_SAMPLES)/psmf/6.MPS
> > +fate-psmf-probe-6: CMD = run $(PROBE_CODEC_NAME_COMMAND) -i "$(SRC)"
> > +fate-psmf-probe-6: CMP = oneline
> > +fate-psmf-probe-6: REF = codec_name=atrac3p
> > +fate-psmf-probe-EV01_01_0500D: SRC = $(TARGET_SAMPLES)/psmf/EV01_
> 01_0500D.PMF
> > +fate-psmf-probe-EV01_01_0500D: CMD = run $(PROBE_CODEC_NAME_COMMAND)
> -i "$(SRC)"
> > +fate-psmf-probe-EV01_01_0500D: CMP = oneline
> > +fate-psmf-probe-EV01_01_0500D: REF = codec_name=atrac3p
>
> where are the sample files ?
> did i miss a upload request ?
>
> [...]
> --
> Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
>
> I have never wished to cater to the crowd; for what I know they do not
> approve, and what they approve I do not know. -- Epicurus
>
> ___
> ffmpeg-devel mailing list
> ffmpeg-devel@ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
>
They're in another email:
http://ffmpeg.org/pipermail/ffmpeg-devel/2017-December/222885.html
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel


Re: [FFmpeg-devel] [PATCH 6/6] psmf: add FATE tests

2017-12-25 Thread Michael Niedermayer
On Mon, Dec 25, 2017 at 10:28:34AM +0800, mi...@brew.sh wrote:
> From: Misty De Meo 
> 
> ---
>  tests/Makefile  |  1 +
>  tests/fate/psmf.mak | 23 +++
>  2 files changed, 24 insertions(+)
>  create mode 100644 tests/fate/psmf.mak
> 
> diff --git a/tests/Makefile b/tests/Makefile
> index fd3713fe81..c569091fcb 100644
> --- a/tests/Makefile
> +++ b/tests/Makefile
> @@ -165,6 +165,7 @@ include $(SRC_PATH)/tests/fate/pcm.mak
>  include $(SRC_PATH)/tests/fate/pixlet.mak
>  include $(SRC_PATH)/tests/fate/probe.mak
>  include $(SRC_PATH)/tests/fate/prores.mak
> +include $(SRC_PATH)/tests/fate/psmf.mak
>  include $(SRC_PATH)/tests/fate/qt.mak
>  include $(SRC_PATH)/tests/fate/qtrle.mak
>  include $(SRC_PATH)/tests/fate/real.mak
> diff --git a/tests/fate/psmf.mak b/tests/fate/psmf.mak
> new file mode 100644
> index 00..0d83daad49
> --- /dev/null
> +++ b/tests/fate/psmf.mak
> @@ -0,0 +1,23 @@
> +#
> +# Test detecting ATRAC-3 audio in Sony MPEG files
> +#
> +
> +PROBE_CODEC_NAME_COMMAND = \
> + ffprobe$(PROGSSUF)$(EXESUF) -show_entries stream=codec_name \
> + -select_streams a -print_format default=noprint_wrappers=1 -bitexact -v > 0
> +
> +FATE_PSMF_FFPROBE = fate-psmf-probe-6 \
> + fate-psmf-probe-EV01_01_0500D
> +
> +fate-psmf-probe-6: SRC = $(TARGET_SAMPLES)/psmf/6.MPS
> +fate-psmf-probe-6: CMD = run $(PROBE_CODEC_NAME_COMMAND) -i "$(SRC)"
> +fate-psmf-probe-6: CMP = oneline
> +fate-psmf-probe-6: REF = codec_name=atrac3p
> +fate-psmf-probe-EV01_01_0500D: SRC = $(TARGET_SAMPLES)/psmf/EV01_01_0500D.PMF
> +fate-psmf-probe-EV01_01_0500D: CMD = run $(PROBE_CODEC_NAME_COMMAND) -i 
> "$(SRC)"
> +fate-psmf-probe-EV01_01_0500D: CMP = oneline
> +fate-psmf-probe-EV01_01_0500D: REF = codec_name=atrac3p

where are the sample files ?
did i miss a upload request ?

[...]
-- 
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

I have never wished to cater to the crowd; for what I know they do not
approve, and what they approve I do not know. -- Epicurus


signature.asc
Description: PGP signature
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel


[FFmpeg-devel] [PATCH 6/6] psmf: add FATE tests

2017-12-24 Thread misty
From: Misty De Meo 

---
 tests/Makefile  |  1 +
 tests/fate/psmf.mak | 23 +++
 2 files changed, 24 insertions(+)
 create mode 100644 tests/fate/psmf.mak

diff --git a/tests/Makefile b/tests/Makefile
index fd3713fe81..c569091fcb 100644
--- a/tests/Makefile
+++ b/tests/Makefile
@@ -165,6 +165,7 @@ include $(SRC_PATH)/tests/fate/pcm.mak
 include $(SRC_PATH)/tests/fate/pixlet.mak
 include $(SRC_PATH)/tests/fate/probe.mak
 include $(SRC_PATH)/tests/fate/prores.mak
+include $(SRC_PATH)/tests/fate/psmf.mak
 include $(SRC_PATH)/tests/fate/qt.mak
 include $(SRC_PATH)/tests/fate/qtrle.mak
 include $(SRC_PATH)/tests/fate/real.mak
diff --git a/tests/fate/psmf.mak b/tests/fate/psmf.mak
new file mode 100644
index 00..0d83daad49
--- /dev/null
+++ b/tests/fate/psmf.mak
@@ -0,0 +1,23 @@
+#
+# Test detecting ATRAC-3 audio in Sony MPEG files
+#
+
+PROBE_CODEC_NAME_COMMAND = \
+   ffprobe$(PROGSSUF)$(EXESUF) -show_entries stream=codec_name \
+   -select_streams a -print_format default=noprint_wrappers=1 -bitexact -v 0
+
+FATE_PSMF_FFPROBE = fate-psmf-probe-6 \
+   fate-psmf-probe-EV01_01_0500D
+
+fate-psmf-probe-6: SRC = $(TARGET_SAMPLES)/psmf/6.MPS
+fate-psmf-probe-6: CMD = run $(PROBE_CODEC_NAME_COMMAND) -i "$(SRC)"
+fate-psmf-probe-6: CMP = oneline
+fate-psmf-probe-6: REF = codec_name=atrac3p
+fate-psmf-probe-EV01_01_0500D: SRC = $(TARGET_SAMPLES)/psmf/EV01_01_0500D.PMF
+fate-psmf-probe-EV01_01_0500D: CMD = run $(PROBE_CODEC_NAME_COMMAND) -i 
"$(SRC)"
+fate-psmf-probe-EV01_01_0500D: CMP = oneline
+fate-psmf-probe-EV01_01_0500D: REF = codec_name=atrac3p
+
+FATE_SAMPLES_FFPROBE += $(FATE_PSMF_FFPROBE)
+
+fate-psmf: $(FATE_PSMF_FFPROBE)
-- 
2.15.1

___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel