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

Error: Process completed with exit code 249 for substrate #60

Closed
mutantcornholio opened this issue Feb 5, 2025 · 3 comments
Closed
Assignees

Comments

@mutantcornholio
Copy link
Contributor

example: https://github.com/paritytech/polkadot-sdk/actions/runs/13160144256/job/36726562373

Works for ./polkadot/**/*.rs and ./cumulus/**/*.rs, but doesn't work for ./substrate/**/*.rs
We also see that node.js process doesn't even start, as the ExperimentalWarning doesn't show there.
Possibly, we're hitting some kind of argument amount limit, or command line size limit, as substrate glob matches much more files, in comparison to polkadot and cumulus

Image

@mutantcornholio mutantcornholio self-assigned this Feb 5, 2025
@mutantcornholio
Copy link
Contributor Author

Yeah, a quick test here shows that current master works fine, while adding a couple of .rs files to PR leads to failure

@mutantcornholio
Copy link
Contributor Author

Which brings us back to paritytech/polkadot-sdk#5776, which removes globbing and enables cargo manifest checks

@mutantcornholio
Copy link
Contributor Author

Okay, it wasn't that hard to update the PR, should've done it months ago...
Fix is merged, closing the issue.

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

No branches or pull requests

1 participant