Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

soundmodem #108545

Closed
danwdart opened this issue Jan 6, 2021 · 4 comments
Closed

soundmodem #108545

danwdart opened this issue Jan 6, 2021 · 4 comments

Comments

@danwdart
Copy link

danwdart commented Jan 6, 2021

Project description
Audio based modem for ham radio supporting ax.25

Metadata

@ymarkus
Copy link
Contributor

ymarkus commented Jan 6, 2021

I'm trying to build it right now (on x86_64-linux), but the builder throws following error:

In file included from gendf9icfilt.cc:39:
../matlib/mat.hh: In function 'void mdet(const T*, unsigned int)':
../matlib/mat.hh:101:24: error: return-statement with a value, in function returning 'void' [8;;https://gcc.gnu.org/onlinedocs/gcc/Warning-Options.html#index-fpermissive-fpermissive8;;]
  101 |                 return 0;
      |                        ^
make[2]: *** [Makefile:349: gendf9icfilt.o] Error 1
make[2]: Leaving directory '/build/soundmodem-0.18/fsk'
make[1]: *** [Makefile:348: all-recursive] Error 1
make[1]: Leaving directory '/build/soundmodem-0.18'
make: *** [Makefile:288: all] Error 2
builder for '/nix/store/pjmpzcm01191h6ic75afa6lrd28wb2n3-soundmodem.drv' failed with exit code 2

As I haven't worked with C++, I don't know if this is a gcc issue or an error in the source code, could you please check?
I guess changing the return value might fix it.

@ymarkus
Copy link
Contributor

ymarkus commented Jan 6, 2021

Ok, looks like I've managed to fix that but there's more errors popping up. I'm sorry, but I don't have time and the knowledge to fix a package that also has issues on the ArchLinux AUR according to the last comment there from 2015.
If you have the time and knowledge of C++, please fix it and I'll be happy to continue with the package. I won't close the issue for now, but I won't continue working on it until these issues are fixed.

Found a 0.20 version by looking at the PKGBUILD and it seems to build now. The binary runs but complains about a missing config file, so this will still need a module.

@ymarkus ymarkus mentioned this issue Jan 6, 2021
10 tasks
@danwdart
Copy link
Author

That's right though, it is configured by running soundmodemconfig first.

@ymarkus
Copy link
Contributor

ymarkus commented Jan 17, 2021

Ok, I guess this is working now, feel free to open a new issue or reopen if there are any other issues popping up!

@ymarkus ymarkus closed this as completed Jan 17, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants