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

Noisy warnings are printed #8124

Closed
PierrickVoulet opened this issue Dec 5, 2020 · 3 comments
Closed

Noisy warnings are printed #8124

PierrickVoulet opened this issue Dec 5, 2020 · 3 comments
Labels

Comments

@PierrickVoulet
Copy link

What version of protobuf and what language are you using?

Version: 3.14.0, C implementation
Language: PHP 7.2, PHP 7.3 , PHP 7.4

What operating system (Linux, Windows, ...) and version?
Linux 64

What did you do?

Run any script that uses the library (e.g. PHPUnit tests)

What did you expect to see

No WARNING: file was already added

What did you see instead?

Several WARNING: file was already added

@witchakorn1999
Copy link

@blowmage

@gmatkowski
Copy link

gmatkowski commented Dec 24, 2020

Yes i have a same issue which prevents me from running tests :-(

@haberman
Copy link
Member

I was able to reproduce this, thanks to help from @PierrickVoulet.

This warning was due to the same proto descriptor being loaded twice. I believe this was due to it being included in multiple aggregated metadata bundles.

The warning message was very unhelpful, which made it very difficult to track down the root cause. I've fixed it to include the name of the file that was being loaded a second time, and where it is being loaded from.

This change is in #8276.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants