Commit 479b17f6 authored by StefanBruens's avatar StefanBruens Committed by Bastien Nocera

Allow *.spx glob for audio/x-speex+ogg

Currently, *.spx is associated with "raw" speex, i.e. without an ogg
container, while it is also often used for audio/x-speex+ogg. The latter
currently only allows the *.oga and *.ogg globs.

See also
https://bugs.freedesktop.org/show_bug.cgi?id=5299
https://bugs.freedesktop.org/show_bug.cgi?id=53376

The libspeex speexenc/speexdec reference implementations use ogg
containers, and use the *.spx extension, see e.g.
https://linux.die.net/man/1/speexdec

"raw" speex and speex+ogg are covered by their content matches, i.e.
'OggS'@0 + 'Speex   '@28 vs. 'Speex   '@0.
Apparently the only application capable of creating "raw" speex is the xiph
sampleenc, see https://github.com/xiph/speex/blob/master/doc/sampleenc.c

Fixes #103
parent c97f1c1e
Pipeline #142720 passed with stage
in 2 minutes and 27 seconds
......@@ -3249,6 +3249,7 @@ command to generate the output files.
</magic>
<glob pattern="*.oga"/>
<glob pattern="*.ogg"/>
<glob pattern="*.spx"/>
</mime-type>
<mime-type type="audio/x-speex">
<comment>Speex audio</comment>
......
......@@ -236,6 +236,8 @@ sample2.amr audio/AMR
sample2.amr audio/amr
# Truncated file from https://bugzilla.gnome.org/show_bug.cgi?id=792325
raw-mjpeg.mjpeg video/x-mjpeg oxo
# From https://cgit.kde.org/kfilemetadata.git/tree/autotests/samplefiles
x_speex_ogg.spx audio/x-speex+ogg
# text
text-iso8859-15.txt text/plain
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment