search

CondorLang / Condor

A fast, simple, and intelligent new programming language
BSD 3-Clause "New" or "Revised" License
34 stars 11 forks source link

Building the memory tests #9

Closed Berkmann18 closed 7 years ago

Berkmann18 commented 7 years ago

Expected Behavior

Memory test results.

Current Behavior

valgrind --tool=memcheck --leak-check=full --track-origins=yes --dsymutil=yes ./build/linux/condor > log.txt 2>&1
Makefile:112: recipe for target 'mem' failed
make: *** [mem] Error 127

Steps to Reproduce

git clone https://github.com/CondorLang/Condor/
cd Condor
make all
make || make mt
make shell
sudo apt-get install valgrind
make mem #note: runing sudo apt-get update before gives the same result when running this line

Context (Environment)

I was trying to set up the CondorLang dev environment (to use/test/improve it) on Linux Ubuntu 16.04 LTS (64-bit) and I run into this problem.

chaseWillden commented 7 years ago

@Berkmann18 Are you sure Valgrind actually installed on your Linux machine? When I run it on my newly created Ubuntu 16.04 LTS, it works for me. Try just typing valgrind into the terminal.

Berkmann18 commented 7 years ago

@chaseWillden Yes valgrind --version returns valgrind-3.11.0. I reran make mem after restarting my Ubuntu OS (I'm dual booting on that and Win10.3) and it now returns

valgrind --tool=memcheck --leak-check=full --track-origins=yes --dsymutil=yes ./build/linux/condor > log.txt 2>&1

Is that what you got (and perhaps what should be outputed on the screen) ?

chaseWillden commented 7 years ago

Removed from Makefile. Switched to cmake to generate makefile to hopefully prevent bugs.