Small change to facilitate compilation in a Linux box

[fexadom]

I am currently testing this code in an Arduino Nano 33 BLE (it has a Nordic nRF52480 MCU) and so far is working fine. Thank you very much for your work, it has been a life saver.

I am working on a Linux machine so I had a compilation problem with the "arduino.h" header file, the compiler could not find it. The easy fix is to change "arduino.h" to "Arduino.h" in every relevant file as the header name is actually "Arduino.h" and in case-sensitive Linux filesystems this matters. I saw in some of the closed issues that some users of this code encountered this problem so to help with a small contribution I propose this pull request that fixes this issue. This shouldn't have any negative effect on Windows systems.

[sergiocntr]

🥰 Il giorno gio 14 mar 2024 alle ore 16:12 Federico Domínguez < ***@***.***> ha scritto:

…

I am currently testing this code in an Arduino Nano 33 BLE (it has a Nordic nRF52480 MCU) and so far is working fine. Thank you very much for your work, it has been a life saver. I am working on a Linux machine so I had a compilation problem with the "arduino.h" header file, the compiler could not find it. The easy fix is to change "arduino.h" to "Arduino.h" in every relevant file as the header name is actually "Arduino.h" and in case-sensitive Linux filesystems this matters. I saw in some of the closed issues that some users of this code encountered this problem so to help with a small contribution I propose this pull request that fixes this issue. This shouldn't have any negative effect on Windows systems. ------------------------------ You can view, comment on, or merge this pull request online at: #39 Commit Summary - 04ebef0 <04ebef0> Changed arduino.h to Arduino.h in all files for compatibility with Linux filesystems where filenames are case sensitive. This change does not affect compilation in a Windows environment. File Changes (4 files <https://github.com/aromring/MAX30102_by_RF/pull/39/files>) - *M* algorithm.cpp <https://github.com/aromring/MAX30102_by_RF/pull/39/files#diff-8e29021a3d987d3ee5934e9e6a94abddfb40f011aadef6d8a308363adc5d2d37> (2) - *M* algorithm.h <https://github.com/aromring/MAX30102_by_RF/pull/39/files#diff-6c833501eb52771977836dc7164c9dff8660c1f82d20d3a20173b637f0f7167f> (2) - *M* algorithm_by_RF.h <https://github.com/aromring/MAX30102_by_RF/pull/39/files#diff-f44035707a9e387c5799319805bc5f6537b5376e423038a39be808bc9be4d5f6> (2) - *M* max30102.h <https://github.com/aromring/MAX30102_by_RF/pull/39/files#diff-281a247395d26b4bbd300fecffb3ea52498841602f4ac1994d6cc115bd8118eb> (2) Patch Links: - https://github.com/aromring/MAX30102_by_RF/pull/39.patch - https://github.com/aromring/MAX30102_by_RF/pull/39.diff — Reply to this email directly, view it on GitHub <#39>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ABW4XQZN4HWUMVMJGWL3KCDYYG44ZAVCNFSM6AAAAABEWJTUSCVHI2DSMVQWIX3LMV43ASLTON2WKOZSGE4DMNRSGU3TONA> . You are receiving this because you are subscribed to this thread.Message ID: ***@***.***>

[aromring]

Thank you very much! Indeed, case does not matter on other platforms I tested it on.