valgrind-macos icon indicating copy to clipboard operation
valgrind-macos copied to clipboard

not compiling with xcode 12.1 and command line tools 12

Open windev92 opened this issue 4 years ago • 3 comments

cd valgrind-macos ./autogen.sh ./configure --prefix=/where/you/want/it/installed --enable-only64bit make sudo make install

/usr/local/bin/mpicc -I../include -g -O -fno-omit-frame-pointer -Wall -dynamic -arch x86_64 -Wno-deprecated-declarations -MT libmpiwrap_amd64_darwin_so-libmpiwrap.o -MD -MP -MF .deps/libmpiwrap_amd64_darwin_so-libmpiwrap.Tpo -c -o libmpiwrap_amd64_darwin_so-libmpiwrap.o test -f 'libmpiwrap.c' || echo './'libmpiwrap.c libmpiwrap.c:281:19: error: expected ')' else if (ty == MPI_UB) fprintf(f,"UB");

windev92 avatar Nov 06 '20 19:11 windev92

This seems to be fixed when using the patch here that removes the use of deprecated MPI symbols (I don't know if there are replacement symbols that should be added to replace the removed lines). I have implemented this patch on my fork (it also includes the addition of a handler for the mkdirat syscall which causes valgrind to fail when used for R).

It seems this might be fixed upstream at some point, but I can create a PR here if desired.

langmm avatar Feb 24 '21 19:02 langmm

@windev92, Which version of macOS were you using?

LouisBrunner avatar Apr 18 '21 15:04 LouisBrunner

bigsur


From: Louis Brunner @.> Sent: Sunday, April 18, 2021 5:45 PM To: LouisBrunner/valgrind-macos @.> Cc: windev92 @.>; Mention @.> Subject: Re: [LouisBrunner/valgrind-macos] not compiling with xcode 12.1 and command line tools 12 (#20)

@windev92https://github.com/windev92, Which version of macOS were you using?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHubhttps://github.com/LouisBrunner/valgrind-macos/issues/20#issuecomment-822012521, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ACIH2NC3Q4R7VIWUONKUID3TJL5DBANCNFSM4TNB3W7A.

windev92 avatar Apr 19 '21 16:04 windev92

Looks like this has been fixed upstream, so I am closing this.

@langmm I think mkdirat has been added a while back (I will do a second pass to be sure) but feel free to open a PR if there is anything you are missing.

LouisBrunner avatar Jan 02 '23 22:01 LouisBrunner