search

uva-cs / pdr

A repo for a Program and Data Representation university-level course
Other
91 stars 227 forks source link

Update Lab 2 #135

Open the-mann opened 2 years ago

the-mann commented 2 years ago

This is needed because the subprocess clang++ needs to be able to read ASAN_OPTIONS and it won't get passed unless it's exported.

I used the conversion tools but it changed a ton of stuff in other .html files, so I only included the relevant change for the .html. Still a little confused on why my setup isn't correctly matching the already generated .html files, and I'd like to continue contributing, so if you could advise that would be helpful. I installed pandoc, astyle, and source-highlight with brew, so maybe it's installing different versions or maybe I missed a setup part.

the-mann commented 2 years ago

@aaronbloomfield when you get the chance to look at this, there are other students running into this issue: https://piazza.com/class/kson5swpsif5qf?cid=456

50Wliu commented 2 years ago

Apparently I never unsubscribed myself from this repository :P.

Anyway,

so if you could advise that would be helpful.

This is most likely because Homebrew installed Pandoc 2.x, but some Linux distributions still ship with Pandoc 1.x. I remember running into that issue as well when I installed Pandoc 2 on Windows. Try downgrading to 1.x and seeing if that helps (and if it does, would be great to update the README to reflect that!).

the-mann commented 2 years ago

@50Wliu I wonder if it would make sense to also update to Homebrew 2... I'll do some testing on a separate PR, but thanks for the workaround in the meantime!