Bioconductor / SparseArray

High-performance sparse data representation and manipulation in R
8 stars 2 forks source link

SparseArray 1.1.5–1.1.6 are broken on macOS with Clang #3

Closed barracuda156 closed 1 year ago

barracuda156 commented 1 year ago
/opt/local/bin/clang-mp-15 -I"/opt/local/Library/Frameworks/R.framework/Resources/include" -DNDEBUG  -I'/opt/local/Library/Frameworks/R.framework/Versions/4.3/Resources/library/S4Vectors/include' -I'/opt/local/Library/Frameworks/R.framework/Versions/4.3/Resources/library/IRanges/include' -I'/opt/local/Library/Frameworks/R.framework/Versions/4.3/Resources/library/XVector/include' -I/opt/local/include -isysroot/Library/Developer/CommandLineTools/SDKs/MacOSX13.sdk    -fPIC  -pipe -Os -isysroot/Library/Developer/CommandLineTools/SDKs/MacOSX13.sdk -arch x86_64  -c Rvector_summarization.c -o Rvector_summarization.o
  Rvector_summarization.c:849:3: error: expected expression
                  double center = summarize_op->center;
                  ^
  Rvector_summarization.c:850:32: error: use of undeclared identifier 'center'
                  res->outbuf.one_double[0] += center * center * zerocount;
                                               ^
  Rvector_summarization.c:850:41: error: use of undeclared identifier 'center'
                  res->outbuf.one_double[0] += center * center * zerocount;
                                                        ^
  3 errors generated.
  make: *** [Rvector_summarization.o] Error 1

https://github.com/macports/macports-ports/actions/runs/5031655856/jobs/9024858826

P. S. Build with GCC seems to be fine.

hpages commented 1 year ago

Thanks. Commit 191979fadb2fd76635e59e86e9f9ec837f010599 should fix that.

What a sensitive compiler! :wink:

BTW the link you provide above does not work.

barracuda156 commented 1 year ago

Thanks. Commit 191979fadb2fd76635e59e86e9f9ec837f010599 should fix that.

What a sensitive compiler! :wink:

BTW the link you provide above does not work.

@hpages Thank you!

P. S. I expected GH to keep logs for a while, but apparently rebasing killed them.