Skip to content

Convert project into workspaces #1747

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

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

ryansuhartanto
Copy link

Should resolve #1746

@ryansuhartanto
Copy link
Author

ryansuhartanto commented May 13, 2025

Test is failing with this error for some reason. Wait, the upstream/main is failing by itself.

RuntimeError: To use Array#pack, you must first require 'corelib/array/pack'.

I'll try find the relative node_modules accessor later. Also it's a good idea to change commit 833e4b4 into the project directory.

Edit: This is good enough, I'll commit some few missing relative node_modules paths and let me know if there's any trouble running any of the examples.

@@ -93,7 +93,7 @@ const applyPatches = (asciidoctorCoreDependency) => {
const patchOpalCompiler = () => {
// revert https://github.com/opal/opal/commit/d46792d2160e4f524c3add711f6424dd99187d1c
// see: https://github.com/opal/opal/issues/2099
const sourceFile = path.join(__dirname, '..', '..', 'node_modules', 'opal-compiler', 'src', 'opal-builder.js')
const sourceFile = path.join(__dirname, '..', '..', '..', '..', 'node_modules', 'opal-compiler', 'src', 'opal-builder.js')
const source = fs.readFileSync(sourceFile, 'utf8')
fs.writeFileSync(sourceFile, source.replace(/(Opal\.modules\["opal\/parser\/patch"].*?)(\s+\(function\([^)]+\) {\n\s+var self = \$klass\(\$base, \$super, 'Lexer'\);.*},\s\$Lexer_source_buffer\$eq\$1\.\$\$arity = 1\), nil\) && 'source_buffer='\n\s+}\)\(\$\$\(\$nesting, 'Parser'\), null, \$nesting\);)/gs, '$1'))
}
Copy link
Author

Choose a reason for hiding this comment

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

Should this be deleted? opal/opal#2099 is merged after all.

@ryansuhartanto ryansuhartanto marked this pull request as ready for review May 13, 2025 07:51
@ggrossetie ggrossetie changed the title Covert project into workspaces Convert project into workspaces May 19, 2025
@ggrossetie
Copy link
Member

@ryansuhartanto I rebased on main but there are a few dependency issues on the CI :(

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.

Convert the project into a workspace
2 participants