All of lore.kernel.org
 help / color / mirror / Atom feed
* Soundmodem problems
@ 2015-01-31  8:24 Apostolos Kefalas
  2015-02-01  4:56 ` David Ranch
  0 siblings, 1 reply; 12+ 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] 12+ 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; 12+ 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] 12+ 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; 12+ 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] 12+ 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; 12+ 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] 12+ 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; 12+ 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] 12+ 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; 12+ 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] 12+ 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; 12+ 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] 12+ 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; 12+ 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] 12+ 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; 12+ 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] 12+ 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; 12+ 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] 12+ 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; 12+ 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] 12+ messages in thread

* Soundmodem problems
@ 2015-01-31  8:02 Apostolos Kefalas
  0 siblings, 0 replies; 12+ messages in thread
From: Apostolos Kefalas @ 2015-01-31  8:02 UTC (permalink / raw)
  To: linux-hams

[-- Attachment #1: Type: text/plain, Size: 518 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] 12+ messages in thread

end of thread, other threads:[~2015-02-14 14:47 UTC | newest]

Thread overview: 12+ 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
  -- strict thread matches above, loose matches on Subject: below --
2015-01-31  8:02 Soundmodem problems Apostolos Kefalas

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.