* Soundmodem problems @ 2015-01-31 8:24 Apostolos Kefalas 2015-02-01 4:56 ` David Ranch 0 siblings, 1 reply; 11+ messages in thread From: Apostolos Kefalas @ 2015-01-31 8:24 UTC (permalink / raw) To: linux-hams [-- Attachment #1: Type: text/plain, Size: 520 bytes --] Hello guys, I am using soundmodem version 0.18 on a Debian Jessie system and I have two problems. The first one is that every time I connect or disconnect a usb device, a BJNP probe is sent on all network interfaces including sm0. The second problem is, that soundmodem seems to randomly "freeze" for about 5 minutes, usually with PTT on. After the ~5 min period the PTT is released and soundmodem returns to normal. I would appreciate any help on the above problems. 73 Apostolos, SV1LJJ [-- Attachment #2: This is a digitally signed message part --] [-- Type: application/pgp-signature, Size: 819 bytes --] ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: Soundmodem problems 2015-01-31 8:24 Soundmodem problems Apostolos Kefalas @ 2015-02-01 4:56 ` David Ranch 2015-02-13 14:53 ` conversd-saupp-1.62a Paul Lewis 0 siblings, 1 reply; 11+ messages in thread From: David Ranch @ 2015-02-01 4:56 UTC (permalink / raw) To: linux-hams Hello Apostolos, > I am using soundmodem version 0.18 on a Debian Jessie system and I have > two problems. Though soundmodem should be workable, it hasn't been supported in years and it doesn't work all that well. Fortunately, we have an excellent replacement: Direwolf - https://home.comcast.net/~wb2osz/site/ Download 1.1 and read the User Guide. If you doubt my point on Soundmodem not working well, read the appendix of the User Guide that shows the decode counts on soundmodem vs Direwolf and many other HW TNCs even! Ps. Once you get 1.1 working, it's worth noting there is a known annoyance with an overly long TX-TAIL setting with it. The new 1.2Beta version has fixed this. --David KI6ZHD ^ permalink raw reply [flat|nested] 11+ messages in thread
* conversd-saupp-1.62a 2015-02-01 4:56 ` David Ranch @ 2015-02-13 14:53 ` Paul Lewis 2015-02-13 19:09 ` conversd-saupp-1.62a Thorsten Kranzkowski 0 siblings, 1 reply; 11+ messages in thread From: Paul Lewis @ 2015-02-13 14:53 UTC (permalink / raw) To: linux-hams Hello all I am trying to install conversd-saupp-1.62a.20090622.tar.gz on a new Ubuntu 14.4.1 system and getting the following error. Anyone got any ideas how to over comethis. Still lot of warnings in the make, which I ignore as the binaries are created. Just not able to create the directories and install the binaries using the following command. Anyone have a solution please - may apply to the make install-all as well make install-bin gcc -O2 -s -Wall -Wstrict-prototypes -DWANT_LOG -DFORKPTY -DOWNER=\"daemon\" -DGID=\"daemon\" -DCONF_DIR=\"/usr/local/conversd-saupp/etc\" -DDATA_DIR=\"/usr/local/conversd-saupp/var\" -DCONVERSHOST=\"localhost\" -DSETLOGNAMES -DHAVE_LIBREADLINE -DHAVE_READLINE_READLINE_H \ convers.c ba_stub.o -o convers -lreadline -lncurses convers.c: In function âdo_logâ: convers.c:255:3: warning: format not a string literal and no format arguments [-Wformat-security] fprintf(stderr, fmt); 73 de Paul G4APL -- To unsubscribe from this list: send the line "unsubscribe linux-hams" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: conversd-saupp-1.62a 2015-02-13 14:53 ` conversd-saupp-1.62a Paul Lewis @ 2015-02-13 19:09 ` Thorsten Kranzkowski 2015-02-13 19:18 ` conversd-saupp-1.62a Paul Lewis 2015-02-13 23:59 ` conversd-saupp-1.62a Paul Lewis 0 siblings, 2 replies; 11+ messages in thread From: Thorsten Kranzkowski @ 2015-02-13 19:09 UTC (permalink / raw) To: Paul Lewis; +Cc: linux-hams On Fri, Feb 13, 2015 at 02:53:17PM +0000, Paul Lewis wrote: > > Hello all Hi Paul! > I am trying to install conversd-saupp-1.62a.20090622.tar.gz > on a new Ubuntu 14.4.1 system > > and getting the following error. Anyone got any ideas how to over > comethis. > Still lot of warnings in the make, which I ignore as the binaries are > created. > > Just not able to create the directories and install the binaries using > the following command. > Anyone have a solution please - may apply to the make install-all as > well > > make install-bin > gcc -O2 -s -Wall -Wstrict-prototypes -DWANT_LOG -DFORKPTY > -DOWNER=\"daemon\" -DGID=\"daemon\" > -DCONF_DIR=\"/usr/local/conversd-saupp/etc\" > -DDATA_DIR=\"/usr/local/conversd-saupp/var\" -DCONVERSHOST=\"localhost\" > -DSETLOGNAMES -DHAVE_LIBREADLINE -DHAVE_READLINE_READLINE_H \ > convers.c ba_stub.o -o convers -lreadline -lncurses > convers.c: In function âdo_logâ: > convers.c:255:3: warning: format not a string literal and no format > arguments [-Wformat-security] > fprintf(stderr, fmt); Strictly speaking, it'a a warning, not an error. But anyway, you need to fix convers.c: a) insert a line reading #include <stdarg.h> at the beginning (where the other includes are). b) make the function do_log (around line 255) look like this: int do_log(int level, const char *fmt, ...) { va_list argp; if (level > 2) return 0; va_start(argp, fmt); vfprintf(stderr, fmt, argp); va_end(argp); return fflush(stderr); } This should do the trick. 73 Thorsten DL8BCU > 73 de Paul G4APL > > -- > To unsubscribe from this list: send the line "unsubscribe linux-hams" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html -- | Thorsten Kranzkowski Internet: dl8bcu@dl8bcu.de | | Mobile: ++49 170 1876134 Snail: Kiebitzstr. 14, 49324 Melle, Germany | | Ampr: dl8bcu@db0lj.#rpl.deu.eu, dl8bcu@marvin.dl8bcu.ampr.org [44.130.8.19] | -- To unsubscribe from this list: send the line "unsubscribe linux-hams" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: conversd-saupp-1.62a 2015-02-13 19:09 ` conversd-saupp-1.62a Thorsten Kranzkowski @ 2015-02-13 19:18 ` Paul Lewis 2015-02-13 23:59 ` conversd-saupp-1.62a Paul Lewis 1 sibling, 0 replies; 11+ messages in thread From: Paul Lewis @ 2015-02-13 19:18 UTC (permalink / raw) To: dl8bcu; +Cc: linux-hams Hello Thorsten Many thanks will give that a try 73 de Paul g4apl In message <20150213190950.GA437@ds20.borg.net>, Thorsten Kranzkowski <dl8bcu@dl8bcu.de> writes >On Fri, Feb 13, 2015 at 02:53:17PM +0000, Paul Lewis wrote: >> >> Hello all > >Hi Paul! > >> I am trying to install conversd-saupp-1.62a.20090622.tar.gz >> on a new Ubuntu 14.4.1 system >> >> and getting the following error. Anyone got any ideas how to over >> comethis. >> Still lot of warnings in the make, which I ignore as the binaries are >> created. >> >> Just not able to create the directories and install the binaries using >> the following command. >> Anyone have a solution please - may apply to the make install-all as >> well >> >> make install-bin >> gcc -O2 -s -Wall -Wstrict-prototypes -DWANT_LOG -DFORKPTY >> -DOWNER=\"daemon\" -DGID=\"daemon\" >> -DCONF_DIR=\"/usr/local/conversd-saupp/etc\" >> -DDATA_DIR=\"/usr/local/conversd-saupp/var\" -DCONVERSHOST=\"localhost\" >> -DSETLOGNAMES -DHAVE_LIBREADLINE -DHAVE_READLINE_READLINE_H \ >> convers.c ba_stub.o -o convers -lreadline -lncurses >> convers.c: In function âdo_logâ: >> convers.c:255:3: warning: format not a string literal and no format >> arguments [-Wformat-security] >> fprintf(stderr, fmt); > >Strictly speaking, it'a a warning, not an error. But anyway, you need to >fix convers.c: > >a) insert a line reading > #include <stdarg.h> > at the beginning (where the other includes are). > >b) make the function do_log (around line 255) look like this: > > int do_log(int level, const char *fmt, ...) { > va_list argp; > if (level > 2) > return 0; > va_start(argp, fmt); > vfprintf(stderr, fmt, argp); > va_end(argp); > return fflush(stderr); > } > > >This should do the trick. > >73 Thorsten DL8BCU > >> 73 de Paul G4APL >> >> -- >> To unsubscribe from this list: send the line "unsubscribe linux-hams" in >> the body of a message to majordomo@vger.kernel.org >> More majordomo info at http://vger.kernel.org/majordomo-info.html > -- paul@skywaves.demon.co.uk -- To unsubscribe from this list: send the line "unsubscribe linux-hams" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: conversd-saupp-1.62a 2015-02-13 19:09 ` conversd-saupp-1.62a Thorsten Kranzkowski 2015-02-13 19:18 ` conversd-saupp-1.62a Paul Lewis @ 2015-02-13 23:59 ` Paul Lewis 2015-02-14 11:12 ` conversd-saupp-1.62a Thorsten Kranzkowski 1 sibling, 1 reply; 11+ messages in thread From: Paul Lewis @ 2015-02-13 23:59 UTC (permalink / raw) To: dl8bcu; +Cc: linux-hams Hi all made the changes to convers.c as suggested make clean make install make install-all got a lot of warnings and these look like errors flagged up which I have extracted ## make install user.c:4655:9: warning: ignoring return value of ‚fgets‚, declared with attribute warn_unused_result [-Wunused-result] fgets(lx, 128, fd); ^ In file included from /usr/include/stdio.h:937:0, from user.c:15: In function ‚fgets‚, inlined from ‚last_command‚ at user.c:1192:15: /usr/include/i386-linux-gnu/bits/stdio2.h:261:2: warning: call to ‚__fgets_chk_warn‚ declared with attribute warning: fgets called with bigger size than length of destination buffer [enabled by default] return __fgets_chk_warn (__s, __bos (__s), __n, __stream); ^ gcc -O2 -s -Wall -Wstrict-prototypes -DWANT_LOG -DFORKPTY -DOWNER=\"daemon\" -DGID=\"daemon\" -DCONF_DIR=\"/usr/local/conversd-saupp/etc\" -DDATA_DIR=\"/usr/local/conversd-saupp/var\" -DCONVERSHOST=\"localhost\" -c host.c host.c: In function ‚h_ecmd_command‚: host.c:166:6: warning: variable ‚arglen‚ set but not used [-Wunused-but-set-variable] int arglen; ## host.c:164:21: warning: variable ‚params‚ set but not used [-Wunused-but-set-variable] char *arg, *user, *params; ^ host.c: In function ‚h_netjoin_command‚: host.c:2191:22: warning: the address of ‚buffer‚ will always evaluate as ‚true‚ [-Waddress] lock_j_unlock_other(buffer); ^ host.c:2071:8: note: in definition of macro ‚lock_j_unlock_other‚ if (s) \ ^ host.c: In function ‚h_sysi_command‚: host.c:1394:10: warning: ignoring return value of ‚fscanf‚, declared with attribute warn_unused_result [-Wunused-result] fscanf(fd, "%ld", &uptime); ^ host.c:1398:10: warning: ignoring return value of ‚fgets‚, declared with attribute warn_unused_result [-Wunused-result] fgets(lx, 128, fd); ## In file included from md5_ampr.c:10:0: md5.h:60:6: note: expected ‚unsigned char *‚ but argument is of type ‚char *‚ void MD5Update(MD5_CTX *mdContext, unsigned char *inBuf, unsigned int inLen); ^ gcc -O2 -s -Wall -Wstrict-prototypes -DWANT_LOG -DFORKPTY -DOWNER=\"daemon\" -DGID=\"daemon\" -DCONF_DIR=\"/usr/local/conversd-saupp/etc\" -DDATA_DIR=\"/usr/local/conversd-saupp/var\" -DCONVERSHOST=\"localhost\" -o conversd conversd.o user.o host.o irc.o convert.o config.o tnos.o log.o cfgfile.o access.o noflood.o hmalloc.o ba_stub.o compression.o md5.o md5_ampr.o -lutil gcc -O2 -s -Wall -Wstrict-prototypes -DWANT_LOG -DFORKPTY -DOWNER=\"daemon\" -DGID=\"daemon\" -DCONF_DIR=\"/usr/local/conversd-saupp/etc\" -DDATA_DIR=\"/usr/local/conversd-saupp/var\" -DCONVERSHOST=\"localhost\" -DNO_SHELL -DRAW -DNO_IOCTL \ convers.c ba_stub.o -o rawconvers gcc -O2 -s -Wall -Wstrict-prototypes -DWANT_LOG -DFORKPTY -DOWNER=\"daemon\" -DGID=\"daemon\" -DCONF_DIR=\"/usr/local/conversd-saupp/etc\" -DDATA_DIR=\"/usr/local/conversd-saupp/var\" -DCONVERSHOST=\"localhost\" -DSETLOGNAMES -DHAVE_LIBREADLINE -DHAVE_READLINE_READLINE_H \ convers.c ba_stub.o -o convers -lreadline -lncurses convers.c: In function ‚main‚: convers.c:652:23: error: ‚Function‚ undeclared (first use in this function) rl_event_hook = (Function *) do_select_call; ^ convers.c:652:23: note: each undeclared identifier is reported only once for each function it appears in convers.c:652:33: error: expected expression before ‚)‚ token rl_event_hook = (Function *) do_select_call; ^ convers.c:917:10: warning: ignoring return value of ‚system‚, declared with attribute warn_unused_result [-Wunused-result] system(line + 1); ^ make: *** [convers] Error 1 ################## make install-all make install-all warning: your configuration will be overwritten with the provided example files. do you really want to do this? anyway, a backup is made for every file, as filename~ press ^C to interrupt sleep 10 make install install-conf make[1]: Entering directory `/usr/local/src/ax25/conversd-saupp' gcc -O2 -s -Wall -Wstrict-prototypes -DWANT_LOG -DFORKPTY -DOWNER=\"daemon\" -DGID=\"daemon\" -DCONF_DIR=\"/usr/local/conversd-saupp/etc\" -DDATA_DIR=\"/usr/local/conversd-saupp/var\" -DCONVERSHOST=\"localhost\" -DSETLOGNAMES -DHAVE_LIBREADLINE -DHAVE_READLINE_READLINE_H \ convers.c ba_stub.o -o convers -lreadline -lncurses convers.c: In function ‚main‚: convers.c:652:23: error: ‚Function‚ undeclared (first use in this function) rl_event_hook = (Function *) do_select_call; ^ convers.c:652:23: note: each undeclared identifier is reported only once for each function it appears in convers.c:652:33: error: expected expression before ‚)‚ token rl_event_hook = (Function *) do_select_call; ^ convers.c:917:10: warning: ignoring return value of ‚system‚, declared with attribute warn_unused_result [-Wunused-result] system(line + 1); ^ make[1]: *** [convers] Error 1 make[1]: Leaving directory `/usr/local/src/ax25/conversd-saupp' make: *** [install-all] Error 2 ########### 73 de Paul G4APl -- To unsubscribe from this list: send the line "unsubscribe linux-hams" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: conversd-saupp-1.62a 2015-02-13 23:59 ` conversd-saupp-1.62a Paul Lewis @ 2015-02-14 11:12 ` Thorsten Kranzkowski 2015-02-14 12:15 ` conversd-saupp-1.62a Paul Lewis 2015-02-14 13:18 ` conversd-saupp-1.62a Paul Lewis 0 siblings, 2 replies; 11+ messages in thread From: Thorsten Kranzkowski @ 2015-02-14 11:12 UTC (permalink / raw) To: Paul Lewis; +Cc: linux-hams On Fri, Feb 13, 2015 at 11:59:46PM +0000, Paul Lewis wrote: > Hi all > > made the changes to convers.c as suggested > make clean > make install > make install-all > > > got a lot of warnings and these look like errors flagged up > which I have extracted [warnings snipped] > gcc -O2 -s -Wall -Wstrict-prototypes -DWANT_LOG -DFORKPTY > -DOWNER=\"daemon\" -DGID=\"daemon\" > -DCONF_DIR=\"/usr/local/conversd-saupp/etc\" > -DDATA_DIR=\"/usr/local/conversd-saupp/var\" -DCONVERSHOST=\"localhost\" > -DSETLOGNAMES -DHAVE_LIBREADLINE -DHAVE_READLINE_READLINE_H \ > convers.c ba_stub.o -o convers -lreadline -lncurses > convers.c: In function âmainâ: > convers.c:652:23: error: âFunctionâ undeclared (first use in this > function) > rl_event_hook = (Function *) do_select_call; > ^ > convers.c:652:23: note: each undeclared identifier is reported only once > for each function it appears in > convers.c:652:33: error: expected expression before â)â token > rl_event_hook = (Function *) do_select_call; > ^ replace these two lines (line 651-652 in convers.c) if (RL_READLINE_VERSION < 0x0400) rl_event_hook = (Function *) do_select_call; with these three lines: # if (RL_READLINE_VERSION < 0x0400) rl_event_hook = (Function *) do_select_call; # endif The whole package is in a need of a deep overhaul. There's a lot of unfinished code in it. And support for some really ancient libraries should probably just be removed. 73 Thorsten > > ########### > 73 de Paul G4APl > > > -- | Thorsten Kranzkowski Internet: dl8bcu@dl8bcu.de | | Mobile: ++49 170 1876134 Snail: Kiebitzstr. 14, 49324 Melle, Germany | | Ampr: dl8bcu@db0lj.#rpl.deu.eu, dl8bcu@marvin.dl8bcu.ampr.org [44.130.8.19] | -- To unsubscribe from this list: send the line "unsubscribe linux-hams" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: conversd-saupp-1.62a 2015-02-14 11:12 ` conversd-saupp-1.62a Thorsten Kranzkowski @ 2015-02-14 12:15 ` Paul Lewis 2015-02-14 13:54 ` conversd-saupp-1.62a Thorsten Kranzkowski 2015-02-14 13:18 ` conversd-saupp-1.62a Paul Lewis 1 sibling, 1 reply; 11+ messages in thread From: Paul Lewis @ 2015-02-14 12:15 UTC (permalink / raw) To: dl8bcu; +Cc: linux-hams Hello Thorsten I think you are right about that. Ok on this change here is the result for info. make install-all warning: your configuration will be overwritten with the provided example files. do you really want to do this? anyway, a backup is made for every file, as filename~ press ^C to interrupt sleep 10 make install install-conf make[1]: Entering directory `/usr/local/src/ax25/conversd-saupp' gcc -O2 -s -Wall -Wstrict-prototypes -DWANT_LOG -DFORKPTY -DOWNER=\"daemon\" -DGID=\"daemon\" -DCONF_DIR=\"/usr/local/conv ersd-saupp/etc\" -DDATA_DIR=\"/usr/local/conversd-saupp/var\" -DCONVERSHOST=\"localhost\" -DNO_SHELL -DRAW -DNO_IOCTL \ convers.c ba_stub.o -o rawconvers convers.c: In function âmainâ: convers.c:628:0: error: unterminated #else #ifdef NO_IOCTL ^ convers.c:629:5: error: expected declaration or statement at end of input echo = 0; ^ convers.c:591:11: warning: unused variable âinit_stringâ [-Wunused-variable] char *init_string = 0; ^ convers.c:589:11: warning: unused variable âpâ [-Wunused-variable] char *p; ^ convers.c:585:29: warning: unused variable âdaddrâ [-Wunused-variable] struct sockaddr *addr, *daddr; ^ convers.c:585:22: warning: unused variable âaddrâ [-Wunused-variable] struct sockaddr *addr, *daddr; ^ convers.c:584:18: warning: unused variable âdaddrlenâ [-Wunused-variable] int addrlen, daddrlen; ^ convers.c:584:9: warning: unused variable âaddrlenâ [-Wunused-variable] int addrlen, daddrlen; ^ convers.c:577:9: warning: unused variable âerrflagâ [-Wunused-variable] int errflag = 0; ^ convers.c:576:9: warning: unused variable âchâ [-Wunused-variable] int ch; ^ convers.c:574:10: warning: unused variable âgetopt_optsâ [-Wunused-variable] char getopt_opts[64]; ^ convers.c:573:10: warning: unused variable âbufferâ [-Wunused-variable] char buffer[512]; ^ convers.c:629:5: warning: control reaches end of non-void function [-Wreturn-type] echo = 0; ^ make[1]: *** [rawconvers] Error 1 make[1]: Leaving directory `/usr/local/src/ax25/conversd-saupp' make: *** [install-all] Error 2 73 de Paul g4apl >replace these two lines (line 651-652 in convers.c) > > if (RL_READLINE_VERSION < 0x0400) > rl_event_hook = (Function *) do_select_call; > >with these three lines: > ># if (RL_READLINE_VERSION < 0x0400) > rl_event_hook = (Function *) do_select_call; ># endif > > >The whole package is in a need of a deep overhaul. There's a lot of unfinished >code in it. And support for some really ancient libraries should probably just >be removed. > >73 Thorsten -- paul@skywaves.demon.co.uk -- To unsubscribe from this list: send the line "unsubscribe linux-hams" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: conversd-saupp-1.62a 2015-02-14 12:15 ` conversd-saupp-1.62a Paul Lewis @ 2015-02-14 13:54 ` Thorsten Kranzkowski 2015-02-14 14:47 ` conversd-saupp-1.62a Paul Lewis 0 siblings, 1 reply; 11+ messages in thread From: Thorsten Kranzkowski @ 2015-02-14 13:54 UTC (permalink / raw) To: Paul Lewis; +Cc: linux-hams On Sat, Feb 14, 2015 at 12:15:24PM +0000, Paul Lewis wrote: > Hello Thorsten Hi Paul! > I think you are right about that. > Ok on this change here is the result for info. > make install-all As a rule of thumb, you generally should use a plain 'make' first, and only if you are satisfied with the result (no errors detected) proceed with 'make install' etc. Otherwise with sloppy Makefiles you might end up with a broken installation that possibly overwrote a working previous version.... > warning: your configuration will be overwritten with > the provided example files. > do you really want to do this? > anyway, a backup is made for every file, as filename~ > press ^C to interrupt > sleep 10 > make install install-conf > make[1]: Entering directory `/usr/local/src/ax25/conversd-saupp' > gcc -O2 -s -Wall -Wstrict-prototypes -DWANT_LOG -DFORKPTY > -DOWNER=\"daemon\" -DGID=\"daemon\" -DCONF_DIR=\"/usr/local/conv > ersd-saupp/etc\" -DDATA_DIR=\"/usr/local/conversd-saupp/var\" > -DCONVERSHOST=\"localhost\" -DNO_SHELL -DRAW -DNO_IOCTL \ > convers.c ba_stub.o -o rawconvers > convers.c: In function 창main창: > convers.c:628:0: error: unterminated #else > #ifdef NO_IOCTL > ^ Ok, I suppose you botched the last correction. Did you notice you needed to replace _two_ old lines with _three_ new lines? And two of the new lines have a '#' at the beginning. The whole block should read linke this: 628 #ifdef NO_IOCTL 629 echo = 0; 630 #else 631 signal(SIGTSTP, SIG_IGN); /* suspend */ 632 if (tcgetattr(0, &prev_termios)) stop(convtype); 633 memcpy(&curr_termios, &prev_termios, sizeof(struct termios)); 634 echo = curr_termios.c_lflag & ECHO; 635 curr_termios.c_lflag = 0; 636 curr_termios.c_cc[VMIN] = 1; 637 curr_termios.c_cc[VTIME] = 0; 638 #ifdef HAVE_LIBREADLINE 639 screenwidth = 80; 640 #if defined (TIOCGWINSZ) 641 if (ioctl (0, TIOCGWINSZ, &window_size) == 0) 642 { 643 screenwidth = (int) window_size.ws_col; 644 } 645 #endif 646 curr_termios.c_lflag = curr_termios.c_lflag & ~ECHO; 647 // at the time of writing, rl_event_hook is buggy 648 // http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=144585 649 rl_event_hook = 0; 650 #ifdef RL_READLINE_VERSION /* older version have no RL_READLINE_VERSION :( */ 651 # if (RL_READLINE_VERSION < 0x0400) 652 rl_event_hook = (Function *) do_select_call; 653 # endif 654 #endif 655 #endif 656 if (tcsetattr(0, TCSANOW, &curr_termios)) stop(convtype); 657 #endif Note that in lines 653 to 655 there are three '#endif's in a row. In general each '#if' and '#ifdef' needs a corresponding '#endif'. The compiler (preprocessor, to be precise) tells you, that it cannot find the '#endif' belonging to '#ifdef NO_IOCTL' (line 628) / '#else' (line 630). The matching '#endif' is in line 657, but that only works out, if the other '#if'/'#ifdef's and '#endif's in between are correctly paired. > convers.c:629:5: error: expected declaration or statement at end of > input > echo = 0; > ^ This error is a result from the problem above. > convers.c:591:11: warning: unused variable 창init_string창 > [-Wunused-variable] > char *init_string = 0; > ^ All those 'unused variable' warnings can be more or less safely ignored. They are a sign of sloppy programming. 73 Thorsten > 73 de Paul g4apl > > > > > > >replace these two lines (line 651-652 in convers.c) > > > > if (RL_READLINE_VERSION < 0x0400) > > rl_event_hook = (Function *) do_select_call; > > > >with these three lines: > > > ># if (RL_READLINE_VERSION < 0x0400) > > rl_event_hook = (Function *) do_select_call; > ># endif > > > > > >The whole package is in a need of a deep overhaul. There's a lot of unfinished > >code in it. And support for some really ancient libraries should probably just > >be removed. -- | Thorsten Kranzkowski Internet: dl8bcu@dl8bcu.de | | Mobile: ++49 170 1876134 Snail: Kiebitzstr. 14, 49324 Melle, Germany | | Ampr: dl8bcu@db0lj.#rpl.deu.eu, dl8bcu@marvin.dl8bcu.ampr.org [44.130.8.19] | -- To unsubscribe from this list: send the line "unsubscribe linux-hams" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: conversd-saupp-1.62a 2015-02-14 13:54 ` conversd-saupp-1.62a Thorsten Kranzkowski @ 2015-02-14 14:47 ` Paul Lewis 0 siblings, 0 replies; 11+ messages in thread From: Paul Lewis @ 2015-02-14 14:47 UTC (permalink / raw) To: dl8bcu; +Cc: linux-hams Hello Thorsten MANY THANKS You are correct that endif was missing. Thank you for explaining that error to me and for the revised code. I can now confirmed. It has been compiled and installed including the configuration files. Many thanks for your assistance. I will update the source on my own system, just in case I need to recompile in the future on my own Ubuntu 14.4.1 32bit system 73 de Paul G4APL GB7CIP -- paul@skywaves.demon.co.uk ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: conversd-saupp-1.62a 2015-02-14 11:12 ` conversd-saupp-1.62a Thorsten Kranzkowski 2015-02-14 12:15 ` conversd-saupp-1.62a Paul Lewis @ 2015-02-14 13:18 ` Paul Lewis 1 sibling, 0 replies; 11+ messages in thread From: Paul Lewis @ 2015-02-14 13:18 UTC (permalink / raw) To: dl8bcu; +Cc: linux-hams Hello Thorsten I have just got htppu 1.6 convers server up and running now. So the saupp-convers can be put on the back burner. Happy to continue with any tests if and when required Thanks to you, John and Brian for your Interests and feedback 73 de Paul G4APL GB7CIP -- paul@skywaves.demon.co.uk ^ permalink raw reply [flat|nested] 11+ messages in thread
end of thread, other threads:[~2015-02-14 14:47 UTC | newest] Thread overview: 11+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2015-01-31 8:24 Soundmodem problems Apostolos Kefalas 2015-02-01 4:56 ` David Ranch 2015-02-13 14:53 ` conversd-saupp-1.62a Paul Lewis 2015-02-13 19:09 ` conversd-saupp-1.62a Thorsten Kranzkowski 2015-02-13 19:18 ` conversd-saupp-1.62a Paul Lewis 2015-02-13 23:59 ` conversd-saupp-1.62a Paul Lewis 2015-02-14 11:12 ` conversd-saupp-1.62a Thorsten Kranzkowski 2015-02-14 12:15 ` conversd-saupp-1.62a Paul Lewis 2015-02-14 13:54 ` conversd-saupp-1.62a Thorsten Kranzkowski 2015-02-14 14:47 ` conversd-saupp-1.62a Paul Lewis 2015-02-14 13:18 ` conversd-saupp-1.62a Paul Lewis
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).