Skip to content

patina_acpi: do not specify resolver#1379

Merged
Javagedes merged 1 commit intoOpenDevicePartnership:mainfrom
Javagedes:personal/joeyvagedes/acpi-workspace
Mar 9, 2026
Merged

patina_acpi: do not specify resolver#1379
Javagedes merged 1 commit intoOpenDevicePartnership:mainfrom
Javagedes:personal/joeyvagedes/acpi-workspace

Conversation

@Javagedes
Copy link
Contributor

@Javagedes Javagedes commented Mar 9, 2026

Description

Updates patina_acpi toml configuration to not specify the resolver version as the resolver version is specified by the workspace configuration toml and any specified resolver version in individual crates are ignored.

Clears this warning:

image
  • Impacts functionality?
  • Impacts security?
  • Breaking change?
  • Includes tests?
  • Includes documentation?

How This Was Tested

N/A

Integration Instructions

N/A

Updates patina_acpi toml configuration to not specify the resolver
version as the resolver version is specified by the workspace
configuration toml and any specified resolver version in individual
crates are ignored.
@patina-automation
Copy link
Contributor

patina-automation bot commented Mar 9, 2026

✅ QEMU Validation Passed

All QEMU validation jobs completed successfully.

Note: Windows Q35 is only built (QEMU boot is disabled due to QEMU vfat issue).

Workflow run: https://github.com/OpenDevicePartnership/patina/actions/runs/22865148598

Boot Time to EFI Shell

Platform Elapsed
Linux Q35 35.4s
Linux SBSA 44.0s

This comment was automatically generated by the Patina QEMU PR Validation Post workflow.

@Javagedes Javagedes requested review from makubacki and os-d March 9, 2026 16:52
@github-actions github-actions bot added the impact:non-functional Does not have a functional impact label Mar 9, 2026
@codecov
Copy link

codecov bot commented Mar 9, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@Javagedes Javagedes merged commit 888fc46 into OpenDevicePartnership:main Mar 9, 2026
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

impact:non-functional Does not have a functional impact

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants