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

introspect: include choices for array options #9395

Merged
merged 1 commit into from
Oct 14, 2021

Conversation

bonzini
Copy link
Contributor

@bonzini bonzini commented Oct 13, 2021

There was even a test covering this, but it did not fail due to a typo.

@bonzini
Copy link
Contributor Author

bonzini commented Oct 13, 2021

Adding @mensinda too.

@codecov
Copy link

codecov bot commented Oct 13, 2021

Codecov Report

Merging #9395 (2d7ded8) into master (220d5aa) will increase coverage by 0.00%.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #9395   +/-   ##
=======================================
  Coverage   67.29%   67.29%           
=======================================
  Files         396      396           
  Lines       85412    85416    +4     
  Branches    17481    17483    +2     
=======================================
+ Hits        57475    57480    +5     
- Misses      23243    23246    +3     
+ Partials     4694     4690    -4     
Impacted Files Coverage Δ
mesonbuild/mintro.py 90.25% <100.00%> (+0.05%) ⬆️
mesonbuild/scripts/vcstagger.py 87.50% <0.00%> (-4.17%) ⬇️
mesonbuild/dependencies/ui.py 68.45% <0.00%> (-1.35%) ⬇️
mesonbuild/dependencies/dev.py 68.04% <0.00%> (-1.19%) ⬇️
mesonbuild/minstall.py 67.88% <0.00%> (-0.77%) ⬇️
mesonbuild/dependencies/base.py 85.02% <0.00%> (-0.60%) ⬇️
mintro.py 90.25% <0.00%> (+0.05%) ⬆️
mesonlib/universal.py 75.84% <0.00%> (+0.08%) ⬆️
mesonbuild/mesonlib/universal.py 82.03% <0.00%> (+0.16%) ⬆️
msubprojects.py 55.82% <0.00%> (+0.40%) ⬆️
... and 3 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 220d5aa...2d7ded8. Read the comment docs.

There was even a test covering this, but it did not fail due to a typo.
@bonzini
Copy link
Contributor Author

bonzini commented Oct 13, 2021

The failures are due to internal issues in GitHub and Azure.

Copy link
Member

@mensinda mensinda left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@mensinda
Copy link
Member

I am adding this to 0.60.0 since this looks more like a bug fix since there was/is a broken test case.

@mensinda mensinda added this to the 0.60.0 milestone Oct 13, 2021
@xclaesse xclaesse merged commit 5e96730 into mesonbuild:master Oct 14, 2021
bonzini added a commit to qemu/qemu that referenced this pull request Oct 14, 2021
Manually patch the introspection data to include the tracing backends.
This works around a deficiency in Meson that will be fixed by
mesonbuild/meson#9395.

Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
@bonzini bonzini deleted the introspect-array-choices branch October 14, 2021 19:58
@nirbheek nirbheek modified the milestones: 0.60.0, 0.59.3 Oct 19, 2021
duskmoon314 pushed a commit to duskmoon314/qemu that referenced this pull request Oct 23, 2021
Manually patch the introspection data to include the tracing backends.
This works around a deficiency in Meson that will be fixed by
mesonbuild/meson#9395.

Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
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

Successfully merging this pull request may close these issues.

4 participants