Improve build system & Update dependency
Merge request reports
Activity
Added 3 commits:
@jung You maybe want to look at the Makefile changes. The rest is trivial. If Makefile is good then you can merge this ASAP :) or we can discuss further improvement of this "build system".
Why do you want to split
iris-local
like this? Notice that the way I wrote it, it serves as bothinit
andupdate
; the--init
arguments just means "initialize if necessary" and the-nf
passed toln
means "if the link already exists, delete it". So right now,make iris-local
is "initialize Iris if necessary, or update it if necessary, and then build it if necessary". I feel if you (for some reason, but that should be rare) want to only do one of these two things, that's just a single command and you can just do it yourself.Edited by Ralf JungSo why did you add
iris-local-update
?make iris-local
(the way it is currently in master) already does an update :)Edited by Ralf JungRight... my definition of "update" is "update the contents of the folder to the commit given by the submodule".
I just realized that you probably meant "change the submodule to point to the latest upstream commit"? Hm, I see. I didn't add a target for that because it was my expectation that this is only done by the developer of the repo, so they can just do it manually. Also, I didn't know
git submodule
can actually do that ;)Notice however that to do the update, one also has to do a
git commit iris -m "Updating Iris"
(or so), and a re-build of the local repo (not just the remote one). So I am not sure if it is worth having this as a make target. Notice that--merge
is probably wrong, we don't want any additional merge commits there, we want it to essentially hard-reset to the neworigin/master
.Added 2 commits:
Added 2 commits:
Mentioned in commit 0a54775b