-
Notifications
You must be signed in to change notification settings - Fork 5
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
failures running on github.com/tailscale/tailscale #3
Comments
Hi @josharian, thanks for the report, this is very helpful. One thing is since taking the project off pause, I have only been running against a single package at a time, and only recently remembered that it historically supported an actual package pattern like So one thing to try would be just one package at a time. I will look into either making it more robust with a package pattern, or explicitly disallow them with a clear error. Regarding this error:
That might be due to a mismatch between the toolchain and go/packages? I commented here recently on a related older issue: I will look at that as well. Thanks for the report!! |
This works a bit better:
...except that it drops (The |
Oh, and |
Hi @josharian if you are not offended by ugly bash one-liners, this seems to work when run from the root of the tailscale repo (or at least, I started it and it's still running).
I'll look at the results when it is finished. That's slightly tweaked from this one-liner that I was using to stress test against the stdlib: https://github.com/thepudds/fzgen/blob/main/gen/fzgen.go#L30-L31 All that said, it would of course be better to actually properly support package patterns, which I'll look at fixing. |
Also, by default fzgen only targets exported methods/functions, so that one-liner is complaining about several packages that I think don't have anything exported, at least based on brief spot checking. fzgen supports an |
Hi @josharian, FYI, aside from the Setup (using fzgen
Auto-generate fuzz files, and confirm they at least compile with gotip (
Regarding:
That is a good suggestion, and that is now the default behavior when targeting multiple packages at the same time (e.g., if you are a repo owner, and want to do 'fzgen ./...' from repo root). However, I'd still like to make it easy for cursory "passerby fuzzing", where the target might be a dependency or a dependency you are evaluating, and it might only be in your read-only module cache, or the current "hello world" first example in the README of That is probably too subtle, but it is preserving the old behavior for a single target, which I wouldn't mind keeping at least for now... |
Hi @josharian, regarding the I tried to confirm my understanding, and @findleyr replied and expressed some interested in understanding the scenario better:
The most immediate question might be if this still occurs for you using your Go 1.18, and if so, what exact version is that? @findleyr also said clearing the build cache might work around it. That said, if this is worth any additional investigation, maybe clearing the build cache is not the thing to do if @findleyr wants to poke at it... Or, maybe this is not worth investigating more, but I wanted to at least give @findleyr an opportunity to comment. Finally, sorry if there is some mistake here on my part 🙃 |
If you have a chance to look at this, perhaps try copying your cache somewhere and checking if it reproduces with a clear cache. Though I'm not sure there's much value to poking at the cache. Things moved quite quickly over the last few months, and we're mostly just concerned that what we have now (across all importers/exporters) is coherent. |
Closing as resolved. (As of ~January, the only pending mystery was why |
github.com/tailscale/tailscale at 5a9914a92fa72d61510b5f2aacd6665e3f8fa659
Using Go 1.18:
Using the Tailscale Go fork (based on 1.17):
I'm not fussed about this, just reporting it in case it is useful.
The text was updated successfully, but these errors were encountered: