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

Question: What branch is stable? #109

Closed
kvishnivetsky opened this issue Jul 16, 2021 · 8 comments
Closed

Question: What branch is stable? #109

kvishnivetsky opened this issue Jul 16, 2021 · 8 comments

Comments

@kvishnivetsky
Copy link

Hi!

What branch contains stable code?
What branch contains development code?

@dragos-oancea
Copy link

master branch is the branch you want.

@baloian
Copy link
Contributor

baloian commented Jul 31, 2021

I am getting the following error on master branch. @kvishnivetsky did you get the same?

  CCLD     stirshaken
/usr/bin/ld: util/src/stirshaken-stir_shaken_tool.o:/home/artiom/libstirshaken/util/include/stir_shaken_tool.h:231: multiple definition of `options'; util/src/stirshaken-stir_shaken_tool_command.o:/home/artiom/libstirshaken/util/include/stir_shaken_tool.h:231: first defined here
/usr/bin/ld: util/src/stirshaken-stir_shaken_tool.o:/home/artiom/libstirshaken/util/include/stir_shaken_tool.h:197: multiple definition of `sp'; util/src/stirshaken-stir_shaken_tool_command.o:/home/artiom/libstirshaken/util/include/stir_shaken_tool.h:197: first defined here
/usr/bin/ld: util/src/stirshaken-stir_shaken_tool.o:/home/artiom/libstirshaken/util/include/stir_shaken_tool.h:185: multiple definition of `pa'; util/src/stirshaken-stir_shaken_tool_command.o:/home/artiom/libstirshaken/util/include/stir_shaken_tool.h:185: first defined here
/usr/bin/ld: util/src/stirshaken-stir_shaken_tool.o:/home/artiom/libstirshaken/util/include/stir_shaken_tool.h:174: multiple definition of `ca'; util/src/stirshaken-stir_shaken_tool_command.o:/home/artiom/libstirshaken/util/include/stir_shaken_tool.h:174: first defined here
collect2: error: ld returned 1 exit status
make: *** [Makefile:1260: stirshaken] Error 1

@slackr31337
Copy link

slackr31337 commented Aug 4, 2021

@baloian I'm getting that same output when attempting to build on Debian 11.

@baloian
Copy link
Contributor

baloian commented Aug 4, 2021

@slackr31337 I solved. The problem is that libstirshaken/util/include/stir_shaken_tool.h file is included in two different .c files: stir_shaken_tool.c and stir_shaken_tool_command.c. You need do copy-paste code from stir_shaken_tool_command.c to stir_shaken_tool.c and comment source code inside of stir_shaken_tool_command.c.

This is not the best solution but it works.

@piotrgregor
Copy link
Contributor

SignalWire doesn't seem to maintain this library. I do maintain my fork here. It has this issue fixed and this PR merged. Enjoy!

@crienzo
Copy link
Member

crienzo commented Oct 17, 2021

This is untrue. All of our open source projects are community supported. Please submit your PR here for review.

@crienzo
Copy link
Member

crienzo commented Oct 18, 2021

@piotrgregor this will be your first and only warning to participate constructively and positively with signalwire's and freeswitch's open source projects.

@andywolk
Copy link
Contributor

@baloian The error you mentioned is fixed in master. Answering your question: you should use master as it's our default branch. Open another issue if there is any problem still. I'm closing this for now.

@signalwire signalwire locked and limited conversation to collaborators Oct 18, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

7 participants