Skip to content

Update Lab 2 #135

Open
the-mann wants to merge 1 commit into
uva-cs:masterfrom
the-mann:master
Open

Update Lab 2 #135
the-mann wants to merge 1 commit into
uva-cs:masterfrom
the-mann:master

Conversation

@the-mann

@the-mann the-mann commented Oct 4, 2021

Copy link
Copy Markdown

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.

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.
@the-mann

the-mann commented Oct 4, 2021

Copy link
Copy Markdown
Author

@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

@winstliu

winstliu commented Oct 4, 2021

Copy link
Copy Markdown

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

the-mann commented Oct 5, 2021

Copy link
Copy Markdown
Author

@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!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants