Closed edwinb closed 4 years ago
So much for reverting being simple. I guess I'll leave it for now.
- Even if
OPT
isn't set, it buildsidris2.c
with-O2
due to a setting indist/Makefile
. Ideally, by default, the code indist/rts
should be built with-O2
since its stable rts code, but during development we don't want-O2
for the generatedidris2.c
- Most worryingly, when I do
make -C dist/rts clean
I get an error:make: Entering directory '/home/edwin/Research/Yaffle/dist/rts' ../config.mk:1: /config.mk: No such file or directory make: *** No rule to make target '/config.mk'. Stop.
@edwinb I have fixed those issues in PR #383
Now make -C dist/rts clean
works as expected.
Reverts edwinb/Idris2#370
This turned out to cause a few problems, and I can't immediately see how to fix them so I'll need to revert for now. They are:
config.mk
, settingCC ?= clang
doesn't do anything, sinceCC
is set by default, so it should probably be:=
OPT
isn't set, it buildsidris2.c
with-O2
due to a setting indist/Makefile
. Ideally, by default, the code indist/rts
should be built with-O2
since its stable rts code, but during development we don't want-O2
for the generatedidris2.c
Most worryingly, when I do
make -C dist/rts clean
I get an error:I'm no
Makefile
expert, as you can probably tell, so I'd rather not get tied up in knots trying to work this one out.