reproduce-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[task #15643] Add valgrind and patch to maneage


From: Raul Infante-Sainz
Subject: [task #15643] Add valgrind and patch to maneage
Date: Fri, 22 May 2020 04:33:01 -0400 (EDT)
User-agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.13; rv:76.0) Gecko/20100101 Firefox/76.0

Follow-up Comment #3, task #15643 (project reproduce):

I tried but it didn't work. My laptop is a macOS High Sierra, v10.13.6. You
can have a look at the error in the attached file
`valgrind-crash-macOS.tar.gz'.

I also make some changes in order to try to fix it, but at the end it didn't
work. In short, I went to Homebrew formulae for installing valgrind in macOS
(https://github.com/Homebrew/homebrew-core/blob/master/Formula/valgrind.rb).
There, in the configure step there are some options:


   --disable-dependency-tracking
   --enable-only64bit
   --build=amd64-darwin


With these options, it also crashed. So, I investigated a bit more and it
seems that there are some hard coded paths into `coregrind/Makefile.in`, that
had to be replaced by the proper path for macOS systems (with Xcode). To
resume, I did the following changes (showed as Git diff):


diff --git a/reproduce/software/make/high-level.mk
b/reproduce/software/make/high-level.mk
index efaa3c6..14eab09 100644
--- a/reproduce/software/make/high-level.mk
+++ b/reproduce/software/make/high-level.mk
@@ -761,7 +761,8 @@ $(ibidir)/valgrind: $(ibidir)/patch \
             patch --verbose -p1 < $(patchdir)/valgrind-3.15.0-mpi-fix2.patch; \
           fi \
        && autoreconf \
-       && ./configure --prefix=$(idir) \
+       && sed -i
-e's|/usr/include/mach|/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX.sdk/usr/include/mach|g'
$(ddir)/valgrind-$(valgrind-version)/coregrind/Makefile.in \
+       && ./configure --prefix=$(idir) --disable-dependency-tracking
--enable-only64bit --build=amd64-darwin \
        && make -j$(numthreads) \
        && if ! make check -j$(numthreads); then \
             echo; echo "Valgrind's 'make check' failed!"; echo; \


But, again, it didn't work :-(((
At this point, I think if we want to merge with the master branch, we should
put a warning for noticing the user that Valgrind is not ready for macOS
systems yet. What do you think? I will try to fix this if I have time, but let
me know if you have any idea.

    _______________________________________________________

Reply to this item at:

  <https://savannah.nongnu.org/task/?15643>

_______________________________________________
  Message sent via Savannah
  https://savannah.nongnu.org/




reply via email to

[Prev in Thread] Current Thread [Next in Thread]