Soundmodem problems

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

Hello Apostolos,

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

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 [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html

Hi Paul!

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

| Thorsten Kranzkowski        Internet: [email protected]                      |
| Mobile: ++49 170 1876134       Snail: Kiebitzstr. 14, 49324 Melle, Germany  |
| Ampr: dl8bcu@db0lj.#rpl.deu.eu, [email protected] [44.130.8.19] |
--
To unsubscribe from this list: send the line "unsubscribe linux-hams" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Hello Thorsten

Many thanks will give that a try
73 de Paul g4apl

In message <[email protected]>, Thorsten Kranzkowski <[email protected]> writes

[email protected]
--
To unsubscribe from this list: send the line "unsubscribe linux-hams" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

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 [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html

[warnings snipped]

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

| Thorsten Kranzkowski        Internet: [email protected]                      |
| Mobile: ++49 170 1876134       Snail: Kiebitzstr. 14, 49324 Melle, Germany  |
| Ampr: dl8bcu@db0lj.#rpl.deu.eu, [email protected] [44.130.8.19] |
--
To unsubscribe from this list: send the line "unsubscribe linux-hams" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

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

[email protected]
--
To unsubscribe from this list: send the line "unsubscribe linux-hams" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Hi Paul!

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....

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.

This error is a result from the problem above.

All those 'unused variable' warnings can be more or less safely ignored. They are a sign of sloppy programming.

73 Thorsten

| Thorsten Kranzkowski        Internet: [email protected]                      |
| Mobile: ++49 170 1876134       Snail: Kiebitzstr. 14, 49324 Melle, Germany  |
| Ampr: dl8bcu@db0lj.#rpl.deu.eu, [email protected] [44.130.8.19] |
--
To unsubscribe from this list: send the line "unsubscribe linux-hams" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

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

[email protected]

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

[email protected]