Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update Lab 2 #135

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Update Lab 2 #135

wants to merge 1 commit into from

Conversation

the-mann
Copy link

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

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
Copy link
Author

the-mann commented Oct 4, 2021

@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
Copy link

winstliu commented Oct 4, 2021

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
Copy link
Author

the-mann commented Oct 5, 2021

@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