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

Patch for #834 #839

Merged
merged 9 commits into from
Apr 24, 2022
Merged

Patch for #834 #839

merged 9 commits into from
Apr 24, 2022

Conversation

2bndy5
Copy link
Member

@2bndy5 2bndy5 commented Apr 15, 2022

solves #834

Using the RPi OS lite image doesn't ship with pigpio, and the configure script wasn't properly detecting if pigpio is installed.

  • adjust configure script to check for pigpio in
    1. user installed location (when fetched from git src - typically in /usr/local/lib)
    2. apt installed location (typically in /usr/lib)
    3. and /usr/arm-linux-gnueabihf (for 32bit cross-compiling purposes)
  • adjust Makefile for lib and linux examples
  • augment install docs with info about pigpio as req for IRQ (and remind reader that RPi OS lite doesn't come with pigpio).

I tested this with the Linux CI: When using the SPIDEV driver (with the configure script), pigpio is no longer installed.

@2bndy5 2bndy5 linked an issue Apr 15, 2022 that may be closed by this pull request
@2bndy5 2bndy5 added the bugfix label Apr 15, 2022
@2bndy5 2bndy5 requested a review from TMRh20 April 24, 2022 02:44
@TMRh20 TMRh20 merged commit c3d1897 into master Apr 24, 2022
@2bndy5 2bndy5 deleted the patch-for-#834 branch April 24, 2022 13:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

pigpio.h: No such file or directory
2 participants