[mythtv-users] Updated dct-channel for controlling DCT-2000 via serial port

[Ian Forde]

On Sat, 2003-10-18 at 22:07, Jim Paris wrote:
> > > No, nothing else changed.  Maybe configure was picking a bad gcc or
> > > giving it bad options.
> > 
> > I just ran the builds again - under 1.3 there weren't any preprocessor
> > options.  Under 1.2 there were a few...
> 
> Yeah, the configure script adds a ton.  What were they for you?  I'd
> say it's bad code if it still compiles but doesn't work, so I'd like
> to fix it.

Okay - for 1.3, running the make yields:

cc -Wall   -c -o channel.o channel.c
cc -Wall   -c -o crc.o crc.c
cc -Wall   -c -o opt.o opt.c
cc -Wall   -c -o packet.o packet.c
cc -Wall   -c -o serial.o serial.c
cc   channel.o crc.o opt.o packet.o serial.o   -o channel

But for 1.2, it's a ton of stuff.  Here goes...

if gcc -DPACKAGE_NAME=\"\" -DPACKAGE_TARNAME=\"\" -DPACKAGE_VERSION=\"\"
-DPACKAGE_STRING=\"\" -DPACKAGE_BUGREPORT=\"\" -DPACKAGE=\"dct-channel\"
-DVERSION=\"1.2\" -DSTDC_HEADERS=1 -DHAVE_SYS_TYPES_H=1
-DHAVE_SYS_STAT_H=1 -DHAVE_STDLIB_H=1 -DHAVE_STRING_H=1
-DHAVE_MEMORY_H=1 -DHAVE_STRINGS_H=1 -DHAVE_INTTYPES_H=1
-DHAVE_STDINT_H=1 -DHAVE_UNISTD_H=1 -DHAVE_ERR_H=1 -DHAVE_FCNTL_H=1
-DHAVE_STDINT_H=1 -DHAVE_STDIO_H=1 -DHAVE_STDLIB_H=1 -DHAVE_STRING_H=1
-DHAVE_STRINGS_H=1 -DHAVE_SYS_POLL_H=1 -DHAVE_TERMIOS_H=1
-DHAVE_UNISTD_H=1 -DHAVE_POLL=1 -DHAVE_STRTOD=1  -I. -I.     -Wall -MT
channel.o -MD -MP -MF ".deps/channel.Tpo" \
  -c -o channel.o `test -f 'channel.c' || echo './'`channel.c; \
then mv -f ".deps/channel.Tpo" ".deps/channel.Po"; \
else rm -f ".deps/channel.Tpo"; exit 1; \
fi
if gcc -DPACKAGE_NAME=\"\" -DPACKAGE_TARNAME=\"\" -DPACKAGE_VERSION=\"\"
-DPACKAGE_STRING=\"\" -DPACKAGE_BUGREPORT=\"\" -DPACKAGE=\"dct-channel\"
-DVERSION=\"1.2\" -DSTDC_HEADERS=1 -DHAVE_SYS_TYPES_H=1
-DHAVE_SYS_STAT_H=1 -DHAVE_STDLIB_H=1 -DHAVE_STRING_H=1
-DHAVE_MEMORY_H=1 -DHAVE_STRINGS_H=1 -DHAVE_INTTYPES_H=1
-DHAVE_STDINT_H=1 -DHAVE_UNISTD_H=1 -DHAVE_ERR_H=1 -DHAVE_FCNTL_H=1
-DHAVE_STDINT_H=1 -DHAVE_STDIO_H=1 -DHAVE_STDLIB_H=1 -DHAVE_STRING_H=1
-DHAVE_STRINGS_H=1 -DHAVE_SYS_POLL_H=1 -DHAVE_TERMIOS_H=1
-DHAVE_UNISTD_H=1 -DHAVE_POLL=1 -DHAVE_STRTOD=1  -I. -I.     -Wall -MT
crc.o -MD -MP -MF ".deps/crc.Tpo" \
  -c -o crc.o `test -f 'crc.c' || echo './'`crc.c; \
then mv -f ".deps/crc.Tpo" ".deps/crc.Po"; \
else rm -f ".deps/crc.Tpo"; exit 1; \
fi
if gcc -DPACKAGE_NAME=\"\" -DPACKAGE_TARNAME=\"\" -DPACKAGE_VERSION=\"\"
-DPACKAGE_STRING=\"\" -DPACKAGE_BUGREPORT=\"\" -DPACKAGE=\"dct-channel\"
-DVERSION=\"1.2\" -DSTDC_HEADERS=1 -DHAVE_SYS_TYPES_H=1
-DHAVE_SYS_STAT_H=1 -DHAVE_STDLIB_H=1 -DHAVE_STRING_H=1
-DHAVE_MEMORY_H=1 -DHAVE_STRINGS_H=1 -DHAVE_INTTYPES_H=1
-DHAVE_STDINT_H=1 -DHAVE_UNISTD_H=1 -DHAVE_ERR_H=1 -DHAVE_FCNTL_H=1
-DHAVE_STDINT_H=1 -DHAVE_STDIO_H=1 -DHAVE_STDLIB_H=1 -DHAVE_STRING_H=1
-DHAVE_STRINGS_H=1 -DHAVE_SYS_POLL_H=1 -DHAVE_TERMIOS_H=1
-DHAVE_UNISTD_H=1 -DHAVE_POLL=1 -DHAVE_STRTOD=1  -I. -I.     -Wall -MT
opt.o -MD -MP -MF ".deps/opt.Tpo" \
  -c -o opt.o `test -f 'opt.c' || echo './'`opt.c; \
then mv -f ".deps/opt.Tpo" ".deps/opt.Po"; \
else rm -f ".deps/opt.Tpo"; exit 1; \
fi
if gcc -DPACKAGE_NAME=\"\" -DPACKAGE_TARNAME=\"\" -DPACKAGE_VERSION=\"\"
-DPACKAGE_STRING=\"\" -DPACKAGE_BUGREPORT=\"\" -DPACKAGE=\"dct-channel\"
-DVERSION=\"1.2\" -DSTDC_HEADERS=1 -DHAVE_SYS_TYPES_H=1
-DHAVE_SYS_STAT_H=1 -DHAVE_STDLIB_H=1 -DHAVE_STRING_H=1
-DHAVE_MEMORY_H=1 -DHAVE_STRINGS_H=1 -DHAVE_INTTYPES_H=1
-DHAVE_STDINT_H=1 -DHAVE_UNISTD_H=1 -DHAVE_ERR_H=1 -DHAVE_FCNTL_H=1
-DHAVE_STDINT_H=1 -DHAVE_STDIO_H=1 -DHAVE_STDLIB_H=1 -DHAVE_STRING_H=1
-DHAVE_STRINGS_H=1 -DHAVE_SYS_POLL_H=1 -DHAVE_TERMIOS_H=1
-DHAVE_UNISTD_H=1 -DHAVE_POLL=1 -DHAVE_STRTOD=1  -I. -I.     -Wall -MT
packet.o -MD -MP -MF ".deps/packet.Tpo" \
  -c -o packet.o `test -f 'packet.c' || echo './'`packet.c; \
then mv -f ".deps/packet.Tpo" ".deps/packet.Po"; \
else rm -f ".deps/packet.Tpo"; exit 1; \
fi
if gcc -DPACKAGE_NAME=\"\" -DPACKAGE_TARNAME=\"\" -DPACKAGE_VERSION=\"\"
-DPACKAGE_STRING=\"\" -DPACKAGE_BUGREPORT=\"\" -DPACKAGE=\"dct-channel\"
-DVERSION=\"1.2\" -DSTDC_HEADERS=1 -DHAVE_SYS_TYPES_H=1
-DHAVE_SYS_STAT_H=1 -DHAVE_STDLIB_H=1 -DHAVE_STRING_H=1
-DHAVE_MEMORY_H=1 -DHAVE_STRINGS_H=1 -DHAVE_INTTYPES_H=1
-DHAVE_STDINT_H=1 -DHAVE_UNISTD_H=1 -DHAVE_ERR_H=1 -DHAVE_FCNTL_H=1
-DHAVE_STDINT_H=1 -DHAVE_STDIO_H=1 -DHAVE_STDLIB_H=1 -DHAVE_STRING_H=1
-DHAVE_STRINGS_H=1 -DHAVE_SYS_POLL_H=1 -DHAVE_TERMIOS_H=1
-DHAVE_UNISTD_H=1 -DHAVE_POLL=1 -DHAVE_STRTOD=1  -I. -I.     -Wall -MT
serial.o -MD -MP -MF ".deps/serial.Tpo" \
  -c -o serial.o `test -f 'serial.c' || echo './'`serial.c; \
then mv -f ".deps/serial.Tpo" ".deps/serial.Po"; \
else rm -f ".deps/serial.Tpo"; exit 1; \
fi
gcc  -Wall   -o channel  channel.o crc.o opt.o packet.o serial.o  

Sooo... there it is...

> (Plus, 1.3 installs the manpage in the wrong place, so I'll be fixing
> stuff anyway :) )

Oh yeah - I forgot to mention that bit!  You have a dependency upon
help2man, which wasn't on my system before... why not just have a static
man page?  (I'd like to say that I'm one who doesn't update the software
on my main myth box often, but since I've just spent the last 45 minutes
updating rpms via apt and am now building 0.12... heh heh heh...)

But you might want to add something to the README - as it is now, the
build will fail if channel.1 won't build.  You could always reverse the
order of the "all:" line...

	-I
-- 
                       __________________________________
                       Ian Forde, RHCE, CCSE, SCNA, SCDME
                       CYTBeN, Inc.
                       ian at duckland.org / ian at cytben.com