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

allow to compile Slave mode #19

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

kgawda
Copy link

@kgawda kgawda commented Apr 27, 2016

some corrections in er9x Makefile to fix issues with compiling with SLAVE=YES

kgawda added 2 commits April 27, 2016 23:45
do not add source files specific for Extensions, if Slave mode is selected
@MikeBland
Copy link
Owner

I've corrected the caps in the filename on my local copy.
The "extra files" is not really a problem since if you have "SLAVE = YES" you shouldn't have something like "EXT=JETI" as it is meaningless for the slave (used by the 9Xtreme). I have, however, also moved the lines in the makefile as suggested.
I have other changes in the 'next' branch (locally) so that is where I've made these changes.

@kgawda
Copy link
Author

kgawda commented May 1, 2016

Hi

Just to explain: I have modified the lines regarding EXT because: I have
the FRSKY modification in my Turnigy 9X hardware (and I guess it is
necessary because you otherwise RX/TX is unavailable) and I see that
presence of FRSKY is checked in slave.c code. So I use make EXT=FRSKY SLAVE=YES.

Thanks for taking this into account. Looking forward for seeing the 'next'
branch published :)

Best Regards
Konrad

2016-04-30 22:36 GMT+02:00 MikeBland [email protected]:

I've corrected the caps in the filename on my local copy.
The "extra files" is not really a problem since if you have "SLAVE = YES"
you shouldn't have something like "EXT=JETI" as it is meaningless for the
slave (used by the 9Xtreme). I have, however, also moved the lines in the
makefile as suggested.
I have other changes in the 'next' branch (locally) so that is where I've
made these changes.


You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
#19 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants