freenode/#clasp - IRC Chatlog
Search
11:23:07
frgo
Hello all - I know it's early for US but, by any chance, are you drmeister or Bike online? I cloned externals-clasp and clasp branch dev and get a build error - is this a known issue?
12:01:27
attila_lendvai
I'm wondering now that I've integrated pump into the build, maybe its output should be deleted from the repo and moved under build/...
12:01:58
attila_lendvai
I'd fancy some opinion on that. python is needed for the build anyways, so there's no reason to keep the pump.py output checked into the repo
12:03:58
frgo
The output are header files. They are needed for C++ level compile. Do you plan to regenerate those every time a compile is done?
12:05:32
frgo
Ok - distclean cleans the build directory, in fact deletes it. So yes, it *should* work.
12:06:51
attila_lendvai
I'll create the PR now anyways to gather feedback, and I'll do that afterwards. then I need to run a full build to test it...
12:10:08
attila_lendvai
frgo: it's black magic... one thing that I needed was bld.add_group(). it's kinda like a separator between two groups of parallel tasks. then there's the ext_out property on tasks, that I don't understand much, but they do affect the order (i.e. tasks with '.h' ext_out must be finished before tasks with '.o' ext_out can start). and then there's new_task(..., after = [ some_other_task_instance ])
12:11:26
attila_lendvai
no, it's waf. it's full of complex DWIM behavior that hurts making progress way more than helps it
12:11:53
frgo
Ok - I figure it's an extension thing. This happens only for the extension I am implementing.
14:18:59
drmeister
We had a host of problems yesterday with the build system. But Shiho and I were able to build at the end of the day - do you have the latest dev commit>
14:22:42
drmeister
We ran into the problem that you are seeing at around 2:00pm in the afternoon and got past it on Shiho's machine and my machine.
14:27:36
drmeister
and then ./waf update_submodules and then go back into clasp/src/lisp/kernel/contrib/sicl
14:28:45
attila_lendvai
I have fixes for that, but I stayed away from dev as we discussed. this is the proper fix: https://github.com/attila-lendvai/clasp/commit/8d1e641e490327e8babe1a4bb07ab019518044ef
14:28:59
drmeister
attila_lendvai: Its something to do with the recent changes to fetch_git_revision
14:30:23
drmeister
I'm not going to pull it into the repo that I'm depending on until I've tested it.
14:34:47
attila_lendvai
I've made some changes locally into those repos and been running fine with those. hope it'll also work in your env!
14:36:59
attila_lendvai
in case of 3 it should merge the news and the local changes. if it doesn't work it should fail (but it may leave merge artifacts in the working copy. not sure which is better...)