Menu ▾ ▴

#282 include path missing in .pc file

0.10.1
closed
nobody
None
fixed
2017-08-24
2017-04-11
Alec Leamas
No

Discussion

  • Alec Leamas

    Alec Leamas - 2017-04-12
    • status: open --> closed
    • Resolution: na --> fixed
     
  • Alec Leamas

    Alec Leamas - 2017-04-12

    Fixed in [712f35], closing

     

    Related

    Commit: [712f35]

  • Alec Leamas

    Alec Leamas - 2017-04-13
    • status: closed --> open
    • Resolution: fixed --> na
     
  • Alec Leamas

    Alec Leamas - 2017-04-13

    Re-opening: this is not the correct fix. A driver should only access what's available using lirc-driver.h; having all sors of private headers available is opening a can of worms.

     
    • Alec Leamas

      Alec Leamas - 2017-04-13

      The real problem here is a bad include path in media/lirc.h; this is already fixed in [87c021] without changing the .pc file include paths. Ditching clumsy patch which makes all sorts of private headers visible in the driver.

      Users not using master running into this issue could change

         #include  <config.h>
      

      to

         #include <lirc/config.h>
      

      in include/media/lirc.h. Downstream patches will take this approach.

       

      Related

      Commit: [87c021]


      Last edit: Alec Leamas 2017-05-08
  • Alec Leamas

    Alec Leamas - 2017-08-24
    • status: open --> closed
    • Resolution: na --> fixed
    • Milestone: 0.10 --> 0.10.1
     

Log in to post a comment.