Michael Fink authored 7 years ago and Ralph Giles committed 7 years ago

added missing exported functions ogg_stream_pageout_fill and ogg_stream_flush_fill to compile and use libogg under Visual Studio

Signed-off-by: Ralph Giles <giles@thaumas.net>

James Ross-Gowan authored 8 years ago and Ralph Giles committed 7 years ago

On 64-bit Windows, sizeof(unsigned char *) is 8, sizeof(long) is 4 and
alignof(ogg_int64_t) is 8. This results in a 4-byte hole in ogg_packet
after "long e_o_s", which means ogg_packet structs cannot be reliably
compared by memcmp. Compare ogg_packet structs member-by-member instead.

This fixes `make check` for me on mingw-w64/GCC.

Signed-off-by: Ralph Giles <giles@thaumas.net>

Move off the end-of-life precise base image.

Copied from the vorbis implementation by Marcel Metz.

The previous commit changed the generated source file location
without updating the installed-file list or include directories,
breaking installation and downstream integration tests.

Adding the include directory doesn't seem to be necessary on macOS,
but even the initial build step fails on Linux without it.

A false continued packet occurs if the previous page ended at the
 end of a packet, with no sequence number gap and no continued data
 on that page, while the current page has the continued packet flag
 set.
Previously, if you drained all of the buffered packet data by
 repeatedly calling ogg_stream_packetout() after submitting the
 previous page but before submitting the current page, libogg would
 discard the continued data at the beggining of the current page.
However, if you left the previous page's data in the buffer, then
 libogg would happily return the continued data at the start of the
 current page as a separate packet when you eventually did call
 ogg_stream_packetout().

This patch makes libogg consistently discard this data.

Tomas Maly authored 8 years ago and Ralph Giles committed 8 years ago

Generate config_types in the build directory instead of
the source directory.

Signed-off-by: Ralph Giles <giles@thaumas.net>

Chocobo1 authored 9 years ago and Ralph Giles committed 8 years ago

Signed-off-by: Ralph Giles <giles@thaumas.net>

Chocobo1 authored 9 years ago and Ralph Giles committed 8 years ago

Signed-off-by: Ralph Giles <giles@thaumas.net>

Ricardo Constantino (:RiCON) authored 9 years ago and Ralph Giles committed 9 years ago

Signed-off-by: Ralph Giles <giles@thaumas.net>

Ricardo Constantino (:RiCON) authored 9 years ago and Ralph Giles committed 9 years ago

Signed-off-by: Ralph Giles <giles@thaumas.net>

Visual Studio 2015 has been out for some time and has a free
version. It doesn't make since to try to maintain all these
old variants.

Users should be able to install the MacOSX Framework by pointing
CMAKE_INSTALL_PREFIX to the corresponding directory. For example to
install into `/Library/Frameworks` configure with

cmake -DCMAKE_INSTALL_PREFIX=/Library/Frameworks <build dir>

This prevents the duplicate installation of header inside the framework
and in the regular include directory on MacOSX.

autoreconf looks for ACLOCAL_*FLAGS in Makefile.am and will
pass that on when it invokes aclocal. Furthermore, setting
a local like this doesn't make it available to subcommands,
so the line has no effect.

George Gerdin authored 10 years ago and Ralph Giles committed 10 years ago

Minor change to CMakeLists.txt to make sure config_types.h and ogg.pc get
generated in the source tree as expected.

Signed-off-by: Ralph Giles <giles@thaumas.net>

stopiccot authored 10 years ago and Ralph Giles committed 10 years ago

Signed-off-by: Ralph Giles <giles@thaumas.net>

stopiccot authored 10 years ago and Ralph Giles committed 10 years ago

Signed-off-by: Ralph Giles <giles@thaumas.net>

stopiccot authored 10 years ago and Ralph Giles committed 10 years ago

Signed-off-by: Ralph Giles <giles@thaumas.net>

Signed-off-by: Ralph Giles <giles@mozilla.com>

Signed-off-by: Ralph Giles <giles@mozilla.com>

Also update build instructions for autogen.sh change.

Signed-off-by: Ralph Giles <giles@mozilla.com>

Call configure and distcheck which should verify all the
things we have automation for.

Fixes 'make dist'.

Newer versions prefer AM_CPPFLAGS to INCLUDES.

Copy the autogen.sh script from the newer daala project. This
just calls autoreconf instead of doing its own tool and version
detection.

Note that this now means one must explicitly invoke ./configure
like in the opus and daala projects. ./autogen.sh will no longer
do so.

Signed-off-by: Ralph Giles <giles@thaumas.net>

Signed-off-by: Erik de Castro Lopo <erikd@mega-nerd.com>

Signed-off-by: Erik de Castro Lopo <erikd@mega-nerd.com>