#66012 closed defect (fixed)

Can we use legacysupport with R packages?

Reported by: barracuda156 Owned by: i0ntempest
Priority: Normal Milestone:
Component: ports Version: 2.7.2
Keywords: powerpc, leopard, snowleopard Cc:
Port: R

Description

Trying to build vroom package fails on 10.5.8:

* installing *source* package ‘vroom’ ...
** package ‘vroom’ successfully unpacked and MD5 sums checked
** using staged installation
** libs
/opt/local/bin/g++-mp-11 -std=gnu++11 -I"/opt/local/Library/Frameworks/R.framework/Resources/include" -DNDEBUG -Imio/include -DWIN32_LEAN_AND_MEAN -Ispdlog/include -DFMT_HEADER_ONLY -I'/Users/svacchanda/Library/R/Power Macintosh/4.2/library/cpp11/include' -I'/Users/svacchanda/Library/R/Power Macintosh/4.2/library/progress/include' -I'/Users/svacchanda/Library/R/Power Macintosh/4.2/library/tzdb/include' -I/opt/local/include   -fPIC  -pipe -Os -arch ppc  -c Iconv.cpp -o Iconv.o
Iconv.cpp: In function 'SEXPREC* safeMakeChar(const char*, size_t, bool)':
Iconv.cpp:70:23: error: 'strnlen' was not declared in this scope; did you mean 'strlen'?
   70 | #define readr_strnlen strnlen
      |                       ^~~~~~~
Iconv.cpp:76:24: note: in expansion of macro 'readr_strnlen'
   76 |   size_t m = hasNull ? readr_strnlen(start, n) : n;
      |                        ^~~~~~~~~~~~~
make: *** [Iconv.o] Error 1
ERROR: compilation failed for package ‘vroom’

However legacysupport PG has this definition. Can we set it to be used with R packages?

Change History (4)

comment:1 Changed 19 months ago by barracuda156

Keywords: powerpc leopard snowleopard added

comment:2 Changed 19 months ago by i0ntempest

There is a .r/Makevars file for you to define additional build settings like CFLAGS and LDFLAGS. Maybe try putting stuff in there? Or would it need R to be compiled with legacysupport first?

comment:3 in reply to:  2 Changed 14 months ago by barracuda156

Replying to i0ntempest:

There is a .r/Makevars file for you to define additional build settings like CFLAGS and LDFLAGS. Maybe try putting stuff in there? Or would it need R to be compiled with legacysupport first?

Thank you.

The ticket can be closed, it works now.

comment:4 Changed 14 months ago by reneeotten (Renee Otten)

Resolution: fixed
Status: assignedclosed
Note: See TracTickets for help on using tickets.