-
Notifications
You must be signed in to change notification settings - Fork 6
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
crate directory is missing a Cargo.toml
file; is native
the wrong directory?
#10
Comments
We are using this patch in flutter web app. Please guide. |
If you just use the flutter patch, you can skip building the rust stuff. Besides, I don't know why you get the Cargo.toml error in your environment. What build_web.sh does is just using |
Understood. We just want to build a flutter web app and apply the patch for performance improvements. Should we just run the patch script after flutter build web command? |
Yeah, you are right. |
I am getting this error ./build_tools/patch_web.sh |
I downloaded your code and copied build_tools and native folder into root of my flutter project, Did I missed something? |
Maybe flutter version, this patch needs flutter master channel. You can view the patch files in build_tools and add changes to your generated |
I'm already on master channel btw
|
Besides, flutter changes recently. Replace those code in your patched flutter.js.
|
I was able to run it but at one place got error .```
|
Can you send us the new patch files? |
Sorry, it's not ready to share. You can check your patch result to verify it's patched correct or not. It's know that flutter master channel changed flutter.js flutter_servicework.js frequently. |
should I downgrade to specific master commit where I can apply your patch? Also when will you be able to share new patch? |
Do I need to update your patch file with this code? |
try to downgrade flutter to a commit at March 1. I think they don't change at that time. |
I have tried to update flutter.js.patch but getting below error
|
I found this flutter commit release tag - https://github.com/flutter/flutter/releases/tag/3.7.6 . |
I have no experience about downgrading the flutter. |
@prashantjdgit I add the new patch in master branch. The patch don't need flutter nightly anymore. |
Theodore Manzanares |
No description provided.
The text was updated successfully, but these errors were encountered: