Closed epipping closed 7 years ago
Leif Andersen 3:02 PM, Jan 24th OHOHOHOH Okay, yes. I talked to @mflatt and that bug is fixed. That’s actually an internal Racket bug. (oddly enough we only noticed it because of a commit you made a while back. So thanks for helping us find it. :smile: ) Anyway, the bug didn’t exist in racket 6.6. (Its only in 6.7) Alternatively, if you want to use one of the nightly releases, the bug is fixed there. Or 6.8 should come out in a few days. Here are some snapshot builds that are likely going to be very similar to 2.8: https://www.cs.utah.edu/plt/snapshots/
fahree [4:20 PM] I committed something???
leif [4:23 PM] oh, ya, but it wasn’t your fault, it just uncovered a bug in racket. Here is a link to the mailing list post:
[4:24]
https://groups.google.com/forum/#!topic/racket-dev/pSOgcAhQ4zo
[4:25]
And here is the commit that uncovered the bug: https://github.com/fare/acmart/commit/84702ed64d2c2c3a1694e82afeaa673027aa067a
GitHub
Download latest acmart, fix settopmatter. · fare/acmart@84702ed · GitHub
printccs was misspelled in the v1.25 distributed by ACM, but fixed in v1.28. Also use (pretitle ...) so it appears early enough.
[4:25]
(Again, this is not a bug in our package, but in core racket.)
[4:26]
And it only shows up in Racket 6.7
fahree [4:26 PM] is there a workaround? use format instead of string-append?
leif [4:26 PM] If you use 6.8 (just been released) it should be gone
fahree [4:26 PM] not sure I can tell my two co-authors to upgrade their racket...
leif [4:26 PM] Ah, okay
fahree [4:26 PM] oh my own, on nixos
leif [4:27 PM] Ah…hmmph, okay
[4:27]
Well one thing you can do is remove the compiled/ directories in the acmart package.
[4:28]
Because this bug only happens when code has been previously compiled. (Which happens automatically when a package is installed.)
[4:28]
As for format
vs string-append
, I have no idea if that will make a difference.
[4:29]
The other thing you can do is just inline the function by hand. Its ugly, but it doesn’t trigger the optimization that confuses the verifier.
fahree [4:31 PM] should I add acmart/doc to .gitignore?
[4:31]
format looks like it works
leif [4:33 PM] Why do you say that? (I don’t see acmart/doc anywhere in the repo)
[4:33]
Also cool, changing it to that works for me.
fahree [4:34 PM] I did a raco pkg install and it generated stuff there
new messages leif [4:35 PM] very odd…(it didn’t do it for me.)
While I can still install the package, I'm told that there were errors during the installation and in particular, I can no longer create a PDF. Here's what happens:
Here's an example run for fare/asdf2017: