github2-x
2018-10-25 13:33:00

github2-x
2018-10-25 14:31:23

github2-x
2018-10-25 14:55:47

github2-x
2018-10-25 15:36:17

github2-x
2018-10-25 15:36:22

github2-x
2018-10-25 15:39:04

github2-x
2018-10-25 16:43:52

github2-x
2018-10-25 16:43:53

github2-x
2018-10-25 20:52:09

github2-x
2018-10-25 20:52:10

cadr
2018-10-25 21:08:29

@samth I think we’re writing into “compiled” dir instead of “pycket-compiled”


samth
2018-10-25 21:08:45

you need to update racket


github2-x
2018-10-25 21:09:06

cadr
2018-10-25 21:09:28

I did, I’m using 1.7.0.2


cadr
2018-10-25 21:09:41

7.1.0.2 that is


samth
2018-10-25 21:10:28

do you have aa6ec1f0cd51ce630c57c2bf343eb6f911077ffc ?


cadr
2018-10-25 21:12:03

Yes, I do have that commit from 3 days ago


cadr
2018-10-25 21:12:12

(have to leave now but I’ll look into it)


cadr
2018-10-25 21:14:44

Ah I think it’s because we’re not setting the use-compiled-file-paths in compile-file-pycket.rkt module


samth
2018-10-25 21:16:53

Shouldn’t it happen automatically when we run under pycket?


cadr
2018-10-25 21:17:57

But we’re resetting the namespace in compile-file-pycket


samth
2018-10-25 21:19:58
                                                                        │··················
[samth@huor:~/sw/plt (master) plt] pycket-c-linklets                                                    │··················
Welcome to Pycket v7.1.0.1                                                                              │··················
> (use-compiled-file-paths)                                                                             │··················
(#<path:pycket-compiled>)                                                                               │··················
> (current-namespace (make-base-namespace))                                                             │··················
> (use-compiled-file-paths)                                                                             │··················
(#<path:pycket-compiled>)                                                                               │··················
>

dan
2018-10-25 21:56:53

@dan has left the channel


cadr
2018-10-26 00:41:51

@samthYes, it’s not about the namespace, it’s about that parameter being overloaded to contain two pieces of information. The recompile target in our Makefile clears the zo files and build them again. And we run the clean with -c flag (to avoid getting version errors when we bump up the expander version), and -c sets the parameter to null to basically say “don’t use compiled code”, so the rebuilding uses the new value null and compile-file defaults to writing into "compiled"


samth
2018-10-26 00:42:14

Ah


cadr
2018-10-26 00:42:23

We basically need to reset the parameters value after clean


samth
2018-10-26 00:42:26

So we need to use parameterize more


samth
2018-10-26 00:43:15

Or really just make the regular Racket tools work


samth
2018-10-26 00:43:34

Did your change fix the unbound variable errors?


cadr
2018-10-26 00:43:58

It seems so, but due to this confusion I’m not entirely sure


cadr
2018-10-26 00:44:12

I’ll fix this parameter issue and run everything again


github2-x
2018-10-26 01:32:15