New Ticket     Tickets     Wiki     Browse Source     Timeline     Roadmap     Ticket Reports     Search

Ticket #26787 (closed defect: fixed)

Opened 3 years ago

Last modified 3 years ago

R: libR.pc doesn't include path to all includes

Reported by: nigel.stanger@… Owned by: ryandesign@…
Priority: Normal Milestone:
Component: ports Version: 1.9.1
Keywords: Cc: kjell.konis@…
Port: R

Description

graphviz @2.26.3_1+java+macosx+perl+php+python26+r+rsvg+tcl to be precise ☺, but it’s +r that it fails on. It fails during the build phase trying to find Rconfig.h (log attached):

:info:build In file included from /opt/local/lib/R/include/Rdefines.h:26,
:info:build                  from gv_R.cpp:743:
:info:build /opt/local/lib/R/include/R_ext/RS.h:27:54: error: Rconfig.h: No such file or directory

It looks like the R includes are not being set up correctly. Out of the box it sets R_CFLAGS to “-I/opt/local/lib/R/include”, but Rconfig.h is in /opt/local/lib/R/include/x86_64. I manually ./configured with R_CFLAGS set to “-I/opt/local/lib/R/include -I/opt/local/lib/R/include/x86_64” and it appears to build correctly.

Attachments

main.log (582.6 KB) - added by nigel.stanger@… 3 years ago.
Portfile-R.diff (842 bytes) - added by kjell.konis@… 3 years ago.

Change History

Changed 3 years ago by nigel.stanger@…

comment:1 Changed 3 years ago by jmr@…

  • Owner changed from macports-tickets@… to ryandesign@…

Please remember to cc the maintainer.

comment:2 Changed 3 years ago by ryandesign@…

  • Status changed from new to assigned
  • Port changed from graphviz to graphviz, graphviz-devel
  • Summary changed from Graphviz @2.26.3_2 +r fails to build to graphviz, graphviz-devel: +r variant: Rconfig.h: No such file or directory

Yes, R seems to have moved where it puts its headers; now everything that uses R is broken. Hooray for R. R's libR.pc pkgconfig file makes no mention of this extra directory. Sounds like an R bug.

Changed 3 years ago by kjell.konis@…

comment:3 Changed 3 years ago by kjell.konis@…

The pkg-config information was overlooked when R switched to architecture-dependant subdirectories. I've told the developers so this should be fixed in an upcoming release. In the meantime I've modified the R port so that the path to Rconfig.h is also included in the pkg-config info.

comment:4 Changed 3 years ago by ryandesign@…

  • Status changed from assigned to closed
  • Resolution set to fixed
  • Port changed from graphviz, graphviz-devel to R
  • Summary changed from graphviz, graphviz-devel: +r variant: Rconfig.h: No such file or directory to R: libR.pc doesn't include path to all includes

Thank you, fixed in r72437.

comment:5 Changed 3 years ago by ryandesign@…

  • Cc kjell.konis@… added
Note: See TracTickets for help on using tickets.