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

make px4 should configure_px4fmu first #37

Closed
wants to merge 5 commits into from
Closed

make px4 should configure_px4fmu first #37

wants to merge 5 commits into from

Conversation

monkeypants
Copy link
Contributor

make clean removes the nuttx configuration for px4fmu/nsh, which prevents make px4 from working. Same with a fresh clone - make won't work unless nuttx has been configured first (using /nuttx/tools/configure.sh).

Reconfiguring nuttx on recompilation doesn't seem to do any harm.

@monkeypants
Copy link
Contributor Author

oops - silly typo. s/PX$/PX4/ in my changed line.

pchickey pushed a commit that referenced this pull request Jan 19, 2013
Pull Request #37
Ammended: squashed fixups down into a single commit

Signed-off-by: Pat Hickey <pat@moreproductive.org>
@pchickey
Copy link
Contributor

Squashed this down and pushed as 76ebcb9. Thanks!

@pchickey pchickey closed this Jan 19, 2013
stephendade pushed a commit to stephendade/ardupilot that referenced this pull request Mar 3, 2013
Pull Request ArduPilot#37
Ammended: squashed fixups down into a single commit

Signed-off-by: Pat Hickey <pat@moreproductive.org>
rayjanwilson referenced this pull request Jul 9, 2013
* added comment
* replaced "~0" with the appropriate macro
* resolved -Wconversion
* saved some space by modifying AP_Param::add_vector_suffix (writing
behind buffer could happen, if buffer_size is almost uint16_max but this
can surely be ignored)

further reading:
http://stackoverflow.com/questions/809227/is-it-safe-to-use-1-to-set-all-bits-to-true
IamPete1 pushed a commit to IamPete1/ardupilot that referenced this pull request Jun 14, 2021
AP_Parachute: refactor releae logic to start at and return to min PWM
shellixyz added a commit to shellixyz/ardupilot that referenced this pull request Jul 5, 2022
…ions (ArduPilot#37)

* Add OSD input throttle element (ArduPilot#31)

* Plane: add arming switch safety (ArduPilot#30)

* use positive throttle range for tecs throttle respose calculations

Co-authored-by: Michel Pastor <shellixyz@users.noreply.github.com>
shellixyz added a commit to shellixyz/ardupilot that referenced this pull request Apr 20, 2023
…ions (ArduPilot#37)

* Add OSD input throttle element (ArduPilot#31)

* Plane: add arming switch safety (ArduPilot#30)

* use positive throttle range for tecs throttle respose calculations

Co-authored-by: Michel Pastor <shellixyz@users.noreply.github.com>
abasahmedd pushed a commit to abasahmedd/ardupilot that referenced this pull request May 20, 2024
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.

None yet

2 participants