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

New build system #25

Open
wants to merge 8 commits into
base: master
Choose a base branch
from
Open

New build system #25

wants to merge 8 commits into from

Conversation

shigma
Copy link
Collaborator

@shigma shigma commented Nov 10, 2018

  • Use mathematica to generate symbol lists and inserted them into syntax file automatically. Currently support variables built_in_functions, built_in_options, built_in_constants and named_characters. A .mx file will be generated to collect data for all this, which is git-ignored and can be regenerated by running namespace.wl.
  • Change the language name into Wolfram Language. (Is that necessary?)
  • Add suffix .wolfram to some of the scopes.
  • Bump Mathematica version into 11.3.

@shigma shigma requested review from chere005 and batracos November 10, 2018 06:54
@shigma shigma self-assigned this Nov 10, 2018
@chere005
Copy link
Collaborator

How do you get the usages from sublime? Is the syntax settings etc on this branch already built, or does it need to be built in order to test? How are the usages pulled up in sublime? Is there documentation for the build system committed yet that I just missed?

}], {"Cells", "GuideText"}];


DumpSave[NotebookDirectory[] <> "wldata.mx", "wl`"];
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should there be a notebook committed if this file expects one to be saved?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No, the wldata.mx will only be used by classify.wl, not by sublime. Anyone who wants the file can generate it by himself. Given that the file is big and binary, a better practice is to leave it ignored.

@shigma
Copy link
Collaborator Author

shigma commented Nov 10, 2018

How do you get the usages from sublime? Is the syntax settings etc on this branch already built, or does it need to be built in order to test? How are the usages pulled up in sublime? Is there documentation for the build system committed yet that I just missed?

I haven't done all the jobs for the usages before they can be seen in sublime (current jobs can be seen at plugin-initiate branch), so the usage file is currently useless. If you mind, I can remove it now and wait for the plugin coming up.

@chere005
Copy link
Collaborator

I might take a crack at a build system myself, this seems a bit hard to follow and lacks proper documentation of the build process. Also need to update for 12 now.

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

Successfully merging this pull request may close these issues.

2 participants