samth
2017-1-30 17:27:29

@mflatt If I cast a _pointer to (_bytes o N) for some appropriate length, will the collector potentially move the storage for that value?


samth
2017-1-30 17:30:01

@pnwamk Did we have errors in the math library at some point about Sequenceof vs Listof?


samth
2017-1-30 17:30:43

That’s what seems to have happened to @asumu for the PPA build



mflatt
2017-1-30 18:05:47

Yes, the value produced for (_bytes o N) will have a copy of the content of the original _pointer, and that copy can move


sabauma
2017-1-30 18:10:56

@mflatt Then mutating the resulting bytes object will not alter the storage backing the _pointer object?


mflatt
2017-1-30 18:11:04

Right


mflatt
2017-1-30 18:11:27

If you want that, use make-sized-byte-string


pnwamk
2017-1-30 18:11:57

nothing with those two together comes to mind…


pnwamk
2017-1-30 18:12:46

@asumu is this a consistent build error? Or an intermittent (possible nondeterministic?) one?


asumu
2017-1-30 18:14:21

@pnwamk: it happened for two builds (precise and trusty) but I haven’t tried to reproduce it otherwise. It didn’t happen on any of the other three builds.


pnwamk
2017-1-30 18:14:44

off the top of our heads, is there something that is different for those builds than for others?


asumu
2017-1-30 18:14:45

(Going to be AFK for a while)


pnwamk
2017-1-30 18:14:47

k


asumu
2017-1-30 18:15:11

Might be a 32/64 bit difference but not sure.


pnwamk
2017-1-30 18:15:31

okay, thanks.


pnwamk
2017-1-30 21:35:26

I’ve got a 32-bit Ubuntu install and I think I’ve reproduced the issue… will report more later


pnwamk
2017-1-30 23:13:09

it appears to be a non-deterministic failure


samth
2017-1-30 23:35:11

Can you open an issue to track it?


samth
2017-1-30 23:37:15

@mflatt will the lack of null termination be a problem?


samth
2017-1-30 23:37:43

It can’t really be helped since this pointer is the result of mmap


mflatt
2017-1-30 23:39:10

@samth Probably not, as long as you don’t try to pass it to a C function as a string


samth
2017-1-30 23:39:25

OK, great


mflatt
2017-1-30 23:39:29

It’s difficult to make any guarantees, which is why the docs for make-sized-byte-string don’t really try


samth
2017-1-30 23:41:00

I just wasn’t sure if the vagueness meant that it will probably work but we don’t make any promises, or that it probably won’t work except by accident


pnwamk
2017-1-30 23:58:49

its fixed



pnwamk
2017-1-30 23:59:05

my bad, guys


asumu
2017-1-31 01:52:09

@pnwamk thanks! Packages built successfully this time.