<p dir="ltr"><br>
On Jul 1, 2014 8:25 AM, "Sean McGovern" <<a href="mailto:gseanmcg@gmail.com">gseanmcg@gmail.com</a>> wrote:<br>
><br>
><br>
><br>
><br>
> On Tue, Jul 1, 2014 at 9:30 AM, Jean-Paul Saman <<a href="mailto:jpsaman@videolan.org">jpsaman@videolan.org</a>> wrote:<br>
>><br>
>> Yes I saw, thank you.<br>
>><br>
>> Kind regards<br>
>><br>
>> Jean-Paul Saman<br>
>><br>
>><br>
>> On Sun, Jun 29, 2014 at 4:31 PM, Timothy Gu <<a href="mailto:timothygu99@gmail.com">timothygu99@gmail.com</a>> wrote:<br>
>>><br>
>>><br>
>>> On Jun 29, 2014 6:14 AM, "Jean-Paul Saman" <<a href="mailto:jpsaman@videolan.org">jpsaman@videolan.org</a>> wrote:<br>
>>> ><br>
>>> ><br>
>>> > On Fri, Jun 27, 2014 at 7:00 PM, Rafaël Carré <<a href="mailto:funman@videolan.org">funman@videolan.org</a>> wrote:<br>
>>> >><br>
>>> >> On 06/27/14 16:47, Sean McGovern wrote:<br>
>>> >> > On Tuesday, June 24, 2014, Sean McGovern <<a href="mailto:gseanmcg@gmail.com">gseanmcg@gmail.com</a>> wrote:<br>
>>> >> >> ---<br>
>>> >> >>  contrib/src/dvbpsi/dvbpsi-solaris-types.patch |   12 ++++++++++++<br>
>>> >> >>  contrib/src/dvbpsi/rules.mak                  |    1 +<br>
>>> >> >>  2 files changed, 13 insertions(+)<br>
>>> >> >>  create mode 100644 contrib/src/dvbpsi/dvbpsi-solaris-types.patch<br>
>>> >> >><br>
>>> >> >> diff --git a/contrib/src/dvbpsi/dvbpsi-solaris-types.patch<br>
>>> >> > b/contrib/src/dvbpsi/dvbpsi-solaris-types.patch<br>
>>> >> >> new file mode 100644<br>
>>> >> >> index 0000000..51b1cb9<br>
>>> >> >> --- /dev/null<br>
>>> >> >> +++ b/contrib/src/dvbpsi/dvbpsi-solaris-types.patch<br>
>>> >> >> @@ -0,0 +1,12 @@<br>
>>> >> >> +--- libdvbpsi/src/descriptor.c.ORIG    Sun Nov 10 07:13:35 2013<br>
>>> >> >> ++++ libdvbpsi/src/descriptor.c Mon Mar  3 22:30:36 2014<br>
>>> >> >> +@@ -29,6 +29,9 @@<br>
>>> >> >> + #include <stdlib.h><br>
>>> >> >> + #include <stdbool.h><br>
>>> >> >> + #include <string.h><br>
>>> >> >> ++#if defined(__sun)<br>
>>> >> >> ++#include <sys/types.h><br>
>>> >> >> ++#endif<br>
>>> >> >> +<br>
>>> >> >> + #if defined(HAVE_INTTYPES_H)<br>
>>> >> >> + #include <inttypes.h><br>
>>> >> >> diff --git a/contrib/src/dvbpsi/rules.mak b/contrib/src/dvbpsi/rules.mak<br>
>>> >> >> index 454fdbe..5f52809 100644<br>
>>> >> >> --- a/contrib/src/dvbpsi/rules.mak<br>
>>> >> >> +++ b/contrib/src/dvbpsi/rules.mak<br>
>>> >> >> @@ -17,6 +17,7 @@ libdvbpsi: libdvbpsi-$(DVBPSI_VERSION).tar.bz2<br>
>>> >> > .sum-dvbpsi<br>
>>> >> >>         $(UNPACK)<br>
>>> >> >>         $(UPDATE_AUTOCONFIG) && cd $(UNPACK_DIR) && mv config.guess<br>
>>> >> > config.sub .auto<br>
>>> >> >>         $(APPLY) $(SRC)/dvbpsi/dvbpsi-noexamples.patch<br>
>>> >> >> +       $(APPLY) $(SRC)/dvbpsi/dvbpsi-solaris-types.patch<br>
>>> >> >>         $(MOVE)<br>
>>> >> >><br>
>>> >> >>  .dvbpsi: libdvbpsi<br>
>>> >> >> --<br>
>>> >> ><br>
>>> >> > Ping!<br>
>>> >><br>
>>> >> Jean-Paul ?<br>
>>> ><br>
>>> ><br>
>>> > Thanks I'll look into it.<br>
>>><br>
>>> This patch is also needed for MinGW, not only for Sun. See <a href="https://mailman.videolan.org/pipermail/libdvbpsi-devel/2014-June/000737.html">https://mailman.videolan.org/pipermail/libdvbpsi-devel/2014-June/000737.html</a><br>

>>><br>
>>> Timothy<br>
><br>
><br>
> Hi,<br>
><br>
> Is it better to patch the header or the .c file?<br>
><br>
> Can we combine and re-submit this patch? Do you want to do that Timothy or should I?</p>
<p dir="ltr">Definitely the header. The header uses `ssize_t` at line 168 [1]. I wonder how the patch to only the .c even works for you. Log for i686-pc-mingw32 without patch:</p>
<p dir="ltr">libtool: compile: i686-pc-mingw32-gcc -DHAVE_CONFIG_H -I. -I.. -g -O2 -Wall -Werror --std=gnu99 -D_GNU_SOURCE -Wpointer-arith -Wcast-align -Wcast-qual -Wstrict-prototypes -Wshadow -Waggregate-return -Wmissing-prototypes -Wnested-externs -Wsign-compare -MT dvbpsi.lo -MD -MP -MF .deps/dvbpsi.Tpo -c dvbpsi.c -o dvbpsi.o In file included from descriptor.c:42:0: descriptor.h:168:58: error: unknown type name 'ssize_t' void *dvbpsi_DuplicateDecodedDescriptor(void *p_decoded, ssize_t i_size); ^ descriptor.c:162:58: error: unknown type name 'ssize_t' void *dvbpsi_DuplicateDecodedDescriptor(void *p_decoded, ssize_t i_size) ^</p>

<p dir="ltr">Can you submit the patch? I'm a little busy lately.</p>
<p dir="ltr">Timothy</p>
<p dir="ltr">[1] <a href="http://git.videolan.org/?p=libdvbpsi.git;a=blob;f=src/descriptor.h;h=f344a730a06c5d8d0f599fbcffdf12e24f64b508;hb=HEAD#l168">http://git.videolan.org/?p=libdvbpsi.git;a=blob;f=src/descriptor.h;h=f344a730a06c5d8d0f599fbcffdf12e24f64b508;hb=HEAD#l168</a></p>