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

Migrate to comrak/Rust #185

Merged
merged 15 commits into from
Aug 5, 2022
Merged

Migrate to comrak/Rust #185

merged 15 commits into from
Aug 5, 2022

Conversation

gjtorikian
Copy link
Owner

Will close #107

pure fyi to @kivikakk for demo of how to integrate Rust into Ruby. I need to study more on how to package the gem itself as a precompiled artifact for various OSes (which Nokogiri recently did).

For anyone else reading this: node iteration/manipulation will probably not be supported during the major release cut over off of C and onto Rust. I'd like to incorporate https://github.com/gjtorikian/selma into https://github.com/gjtorikian/html-pipeline and encourage using a post-processor. Part of this stems from the fact that implementing an AST parser in Ruby is slow; selma also uses a Rust create to more efficiently change multiple nodes at once (via CSS selectors).

So, basically, I'm working on all three of these projects at once, but they won't be released until I can manage to make precompiled gems happen.

@gjtorikian gjtorikian mentioned this pull request Jul 15, 2022
1 task
@gjtorikian gjtorikian changed the base branch from main to v1.0 July 15, 2022 19:10
Was only used to assist in bringing upstream cmark sources in.
@kivikakk
Copy link
Collaborator

kivikakk commented Aug 5, 2022

Looking awesome so far. rake test working well locally.

Is it ridiculously presumptuous of me to add commits to this branch? Drop them if you have any reservations at all, just acting on my compulsions a little bit; removed the out of date Makefile in a76379a, and added some casts for rb_hash_foreach calls to build without warnings in aa4afde (on arm64-darwin anyway).

@gjtorikian
Copy link
Owner Author

Is it ridiculously presumptuous of me to add commits to this branch?

Not at all! I usually have only a few hours a week free to work on this so any help is appreciated.

end

# Why is this so long?
tarball_extract_path = File.join("tmp", recipe.host, "ports", recipe.name, recipe.version, "#{recipe.name}-#{recipe.version}")
Copy link
Owner Author

@gjtorikian gjtorikian Aug 5, 2022

Choose a reason for hiding this comment

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

@kivikakk I'm real embarrassed you had to see all this, hah

@gjtorikian
Copy link
Owner Author

PR looks great. Truly, thanks so much for your help.

In a subsequent (final?) PR, I'm going to use https://github.com/rake-compiler/rake-compiler-dock to precompile native gems.

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.

Should this move to be Rust-backed?
2 participants