-
Notifications
You must be signed in to change notification settings - Fork 8
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
adding httptrace package #26
Comments
to implement this, I was thinking of doing following:
There is a possibility that this might not work, in which case, I was thinking we can just stub out the exported functions. This will atleast allow the compilation to succeed. Also happy to hear if there are other approaches we can take for making progress on this issue. thank you |
This does introduce the problem that code that works with Big Go will compile but fail to actually work with TinyGo, which seems more dangerous than having to reimplement bits of httputil/httptrace in your own code though 😅 |
I would not mind a stubbed version of httptrace. we have other stubs, and so far I have also run into it as a dependency in packages that would otherwise work, such as upgrading to the latest paho mqtt package. |
hello folks,
Thank you for your wonderful work on TinyGo.
I am trying to update one of my app from TinyGo 1.29 to 1.31.x, and running into a problem because package
net/http/httptrace
no longer exists. an issue in TinyGo repo has been reported for this here: tinygo-org/tinygo#4186is there any objection (or obvious known blockers) to adding the package in this repo? I am happy to try to submit a PR for that if this is ok to add.
many thanks for considering
Rajat Jindal
The text was updated successfully, but these errors were encountered: