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

make: *** [dfa.o] Error 1 #12

Closed
rexdouglass opened this issue Dec 19, 2016 · 2 comments
Closed

make: *** [dfa.o] Error 1 #12

rexdouglass opened this issue Dec 19, 2016 · 2 comments

Comments

@rexdouglass
Copy link

Hi all,

I'm having trouble installing re2r on Ubuntu 16.10 (Yakkety Yak). The error is below and appears to be related to a gcc problem (google/re2#102)
I've been unable to resolve it by updating to the latest gcc (version 6.2.0 20161018 (Ubuntu 6.2.0-7ubuntu11) or including flags in the src/Makevars to try to force it to use clang instead (always defaults to g++)
"CC = clang
CXX = clang++"

Error below:

g++ -I/usr/share/R/include -DNDEBUG -I../inst/include -DNDEBUG -I"/home/skynet2/R/x86_64-pc-linux-gnu-library/3.3/Rcpp/include" -I"/home/skynet2/R/x86_64-pc-linux-gnu-library/3.3/RcppParallel/include" -fpic -g -O2 -fdebug-prefix-map=/build/r-base-qk3a9o/r-base-3.3.2=. -fstack-protector-strong -Wformat -Werror=format-security -Wdate-time -D_FORTIFY_SOURCE=2 -g -c dfa.cc -o dfa.o
dfa.cc: In constructor ‘re2::DFA::State::State()’:
dfa.cc:95:10: error: unknown array size in delete
struct State {
^~~~~
dfa.cc: In member function ‘re2::DFA::State* re2::DFA::CachedState(int*, int, re2::uint)’:
dfa.cc:703:9: note: synthesized method ‘re2::DFA::State::State()’ first required here
State state;
^~~~~
/usr/lib/R/etc/Makeconf:139: recipe for target 'dfa.o' failed
make: *** [dfa.o] Error 1
ERROR: compilation failed for package ‘re2r’

  • removing ‘/home/skynet2/R/x86_64-pc-linux-gnu-library/3.3/re2r’
    Error: Command failed (1)
@qinwf
Copy link
Owner

qinwf commented Dec 19, 2016

Thanks for your report. Yes, this is a upstream related issue. I will push the workaround commit as in stefanv/re2@86503cb

qinwf pushed a commit that referenced this issue Dec 19, 2016
@rexdouglass
Copy link
Author

Confirmed working, thanks for the quick help.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants