search

parallaxinc / Flight-Controller

Quadcopter Flight Controller
Other
24 stars 20 forks source link

Filename cases are wrong for Firmware-C #6

Closed electrodude closed 8 years ago

electrodude commented 8 years ago

Most operating systems other than Windows (e.g. OSX and Linux.) have case sensitive filesystems. The C firmware does not compile on these systems because the cases of the filenames on disk and the filenames in #include directives do not agree.

Fixing this would be as simple as either renaming the files or changing the #includes so that they agree with each other.

electrodude commented 8 years ago

I would submit a pull request to do this for you, but I'm not sure whether you'd rather rename the files or change the #includes.

JasonDorie commented 8 years ago

I'll take care of it - thanks for letting me know. I think I've done that for all the GroundStation files already, but I've never tried to build the firmware on anything but a PC.

J

On Wednesday, April 6, 2016, electrodude notifications@github.com wrote:

I would submit a pull request to do this for you, but I'm not sure whether you'd rather rename the files or change the #includes.

— You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub https://github.com/parallaxinc/Flight-Controller/issues/6#issuecomment-206506046

PropGit commented 8 years ago

Thanks for taking care of it, @JasonDorie.

JasonDorie commented 8 years ago

Electrodude - If you can easily test this, sync and let me know if I missed anything. I believe this should be correct now.

On Wed, Apr 6, 2016 at 12:50 PM, Parallax Git Administrator < notifications@github.com> wrote:

Thanks for taking care of it, @JasonDorie https://github.com/JasonDorie.

— You are receiving this because you were mentioned. Reply to this email directly or view it on GitHub https://github.com/parallaxinc/Flight-Controller/issues/6#issuecomment-206530724

electrodude commented 8 years ago

It compiles, but I didn't actually test it yet. I'm closing it now. Thanks.