Skip to content
This repository was archived by the owner on Nov 26, 2025. It is now read-only.

Conversation

@hovsater
Copy link
Collaborator

The argument passed to require_cmd did not respect the TM_ZIG
environment variable.

The argument passed to `require_cmd` did not respect the TM_ZIG
environment variable.
@hovsater
Copy link
Collaborator Author

hovsater commented Jul 22, 2021

@emekoi would you mind taking a look at this one, #52 and #53? Further, I would be happy assist in maintaining this bundle. I'm already doing something similar for the Crystal TextMate bundle. 🙂

@emekoi emekoi merged commit 8ccee6e into ziglang:master Jul 23, 2021
@emekoi
Copy link
Contributor

emekoi commented Jul 23, 2021

@kevinsjoberg i see we now have https://github.com/ziglang/Zig.tmbundle. is the plan split the TextMate and Sublime Text stuff? since it seems you are already helping out with TextMate library, i would be glad to have you.

@hovsater
Copy link
Collaborator Author

Yeah, we split yesterday and I'll maintain the TextMate version. 🙂

@emekoi
Copy link
Contributor

emekoi commented Jul 24, 2021

so is the plan to have github's lignuist continue to point here, or will it point at the TextMate version? also i suppose this means i could remove all the TextMate-specific stuff from this repo and vice versa for the TextMate one. i have no real preference as to whether that happens though.

@hovsater
Copy link
Collaborator Author

hovsater commented Jul 24, 2021

I think we can keep Linguist pointing here still. I've already updated the README in the other repo.

Feel free to remove TextMate specifics. 🙂

@emekoi
Copy link
Contributor

emekoi commented Jul 24, 2021

the issue is that linguist relies on TextMate syntax files, but the newer sublime text syntax features aren't compatible with them iiirc.

@hovsater
Copy link
Collaborator Author

hovsater commented Jul 24, 2021

Oh, in that case I think it makes sense to keep it in the TextMate bundle. I'll submit a PR to linguist itself to change the repo. 👍

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants