Opened 10 years ago

Closed 10 years ago

Last modified 10 years ago

#26787 closed defect (fixed)

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

Reported by: nstanger (Nigel Stanger) Owned by: ryandesign (Ryan Schmidt)
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 (2)

main.log (582.6 KB) - added by nstanger (Nigel Stanger) 10 years ago.
Portfile-R.diff (842 bytes) - added by kjell.konis@… 10 years ago.

Download all attachments as: .zip

Change History (7)

Changed 10 years ago by nstanger (Nigel Stanger)

Attachment: main.log added

comment:1 Changed 10 years ago by jmroot (Joshua Root)

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

Please remember to cc the maintainer.

comment:2 Changed 10 years ago by ryandesign (Ryan Schmidt)

Port: graphviz-devel added
Status: newassigned
Summary: Graphviz @2.26.3_2 +r fails to buildgraphviz, 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 10 years ago by kjell.konis@…

Attachment: Portfile-R.diff added

comment:3 Changed 10 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 10 years ago by ryandesign (Ryan Schmidt)

Port: R added; graphviz graphviz-devel removed
Resolution: fixed
Status: assignedclosed
Summary: graphviz, graphviz-devel: +r variant: Rconfig.h: No such file or directoryR: libR.pc doesn't include path to all includes

Thank you, fixed in r72437.

comment:5 Changed 10 years ago by ryandesign (Ryan Schmidt)

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