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

refactor: replace libb64 with custom base64 implementation #150

Merged
merged 1 commit into from
Nov 3, 2023

Conversation

edubart
Copy link
Contributor

@edubart edubart commented Oct 27, 2023

This is to simplify our dependencies, so the cartesi machine is easier to build on more platforms.

The base64 implementation was copied from LuaSocket code which @diegonehab himself wrote.

@edubart edubart added the refactor Restructuring code, while not changing its original functionality label Oct 27, 2023
@edubart edubart requested a review from diegonehab October 27, 2023 13:26
@edubart edubart self-assigned this Oct 27, 2023
@edubart edubart requested a review from vfusco October 27, 2023 13:38
Copy link
Contributor

@diegonehab diegonehab left a comment

Choose a reason for hiding this comment

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

See comments.

@edubart edubart force-pushed the feature/replace-b64 branch 2 times, most recently from c954ce8 to 14850b3 Compare October 30, 2023 19:23
@edubart edubart requested a review from diegonehab October 30, 2023 19:43
@edubart edubart force-pushed the feature/replace-b64 branch from 14850b3 to 08331eb Compare October 30, 2023 21:45
@edubart edubart force-pushed the feature/replace-b64 branch from 08331eb to fddca22 Compare October 30, 2023 21:53
@vfusco vfusco added this to the v0.16.0 milestone Nov 3, 2023
@edubart edubart merged commit fddca22 into main Nov 3, 2023
7 checks passed
@edubart edubart deleted the feature/replace-b64 branch November 3, 2023 19:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
refactor Restructuring code, while not changing its original functionality
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants