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

Fix macos build #121

Merged
merged 3 commits into from
Apr 18, 2022
Merged

Fix macos build #121

merged 3 commits into from
Apr 18, 2022

Conversation

PikachuHy
Copy link
Contributor

see #120

@sp1ritCS
Copy link
Contributor

Thanks, we were stuck on this one and didn't know how to fix it. Also see #116

@sp1ritCS
Copy link
Contributor

In ragrds to the failing mingw build, for some reason mirrorbrain picks a mirror that doesn't provide repos in the home: namespace. Maybe it's tagged incorrectly, I'll ask the oS heros later.

@PikachuHy
Copy link
Contributor Author

maybe we can add a guard 😭
like this

#if defined(__linux__) || defined(_WIN32)
// do nothing
#else
#include <clocale>
#endif

@sp1ritCS
Copy link
Contributor

no, why would we? it seems to work on linux at least. The buils failures aren't due to your commit. (or doesn't it work on windows?)

@PikachuHy
Copy link
Contributor Author

In ragrds to the failing mingw build

sorry, I misunderstood what you said.

@PikachuHy
Copy link
Contributor Author

can this PR be merged?

@sp1ritCS
Copy link
Contributor

sp1ritCS commented Apr 2, 2022

I've asked about it, since I want this merged too. Development is quite slow, unless something affects @blackhole89 himself :)

Screenshot from 2022-04-02 18-51-25

@blackhole89
Copy link
Owner

Argh, sorry! I'm merging it for now, but both the MinGW and the Darwin CI build seem to fail with something pertaining to clatexmath.

@blackhole89 blackhole89 merged commit 9ca3430 into blackhole89:master Apr 18, 2022
@PikachuHy
Copy link
Contributor Author

both the MinGW and the Darwin CI build seem to fail with something pertaining to clatexmath.

use openmath branch

@sp1ritCS
Copy link
Contributor

@PikachuHy

use openmath branch

no, that wasn't the issue. Darwin CI was failing with linuxbrew and incorrect libstdc++, #124 fixes that.

newer OS X versions seem to fail with cLaTeXMath however (according to one iMac NoteKit user in the chatroom), so I have set up sp1ritCS/tap/notekit-sp1rit and sp1ritCS/tap/microtex that use MicroTeX/openmath. However (likely due to homebrew wierdness) notekit-sp1rit seems to fail to install the datadir and charter fonts property (maybe only when installing the bottle), but I haven't looked further into that.

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.

3 participants