dh1tw / remoteAudio

Audio streaming application for Amateur Radio purposes
MIT License
166 stars 24 forks source link

Compilation issues #26

Closed crazybyte closed 3 years ago

crazybyte commented 3 years ago

Hello,

when compiling the following error is encountered

make all
...
go: finding github.com/micro/mdns v0.3.0
go: github.com/micro/mdns@v0.3.0: unknown revision v0.3.0
go: error loading module requirements
protoc --proto_path=./icd --gofast_out=./sb_audio ./icd/audio.proto
protoc --proto_path=./icd --micro_out=./sb_audio ./icd/audio.proto
protoc-gen-micro: program not found or is not executable
--micro_out: protoc-gen-micro: Plugin failed with status code 1.
make: *** [Makefile:14: build] Error 1

I figured perhaps it needs some dependencies so I ran make install-deps hoping that it will solve the issue. Unfortunately, I get a very similar error related to github.com/micro/mdns

go: finding github.com/micro/mdns v0.3.0
go: github.com/micro/mdns@v0.3.0: unknown revision v0.3.0
go: error loading module requirements
protoc --proto_path=./icd --gofast_out=./sb_audio ./icd/audio.proto
protoc --proto_path=./icd --micro_out=./sb_audio ./icd/audio.proto
protoc-gen-micro: program not found or is not executable
--micro_out: protoc-gen-micro: Plugin failed with status code 1.
make: *** [Makefile:14: build] Error 1

It seems that is related to some dependencies in micro. The github repo referenced is archived since April 2020. Any fix to circumvent and use something else as a dependency to be able to compile the application?

Thank you

dh1tw commented 3 years ago

Hi @crazybyte, go-micro is the underlying communication / micro-services framework which is used by remoteAudio. The go-micro project suffered during the last months from several moves, re-branding, reverted re-branding and merges caused by the alternating mood of the author. By today, go-micro:

I think it's just a matter of sorting out the new locations and updating the dependencies.

I'm right now pretty busy, but hopefully on the weekend or next week I should find some time to update the dependencies. Otherwise a PR would also be greatly appreciated.

crazybyte commented 3 years ago

Hello,

I see. Thank you for your reply. I though that probably you are pretty are busy, but I thought that I ask before making a mess of it :) It's not a pressing issue. I will also try my hand in making a fix and a PR, hopefully with some success. I will leave the issue open (I hope you agree but you can also close it if you want to) so that I can reference it if I manage to make a fix and PR. Thank you again.

dh1tw commented 3 years ago

Hey @crazybyte,

I'm happy to report that the dependency issues have been resolved. The Code compiles now again on all supported platforms. Feel free to re-open this issue if you still encounter a problem.