Skip to content

Minor changes - #274

Merged
ddiss merged 3 commits into
rapido-linux:masterfrom
ddiss:minor_changes
Jun 26, 2026
Merged

Minor changes#274
ddiss merged 3 commits into
rapido-linux:masterfrom
ddiss:minor_changes

Conversation

@ddiss

@ddiss ddiss commented Jun 26, 2026

Copy link
Copy Markdown
Collaborator
The following changes since commit 4ae962faeb141c9bb3011d73df32c70318b48131:

  ci: drop explicit rust v1.85 installation (2026-06-09 17:53:07 +1000)

are available in the Git repository at:

  https://github.com/ddiss/rapido.git minor_changes

for you to fetch changes up to 860d8739f2491e40f3f9f9fcefe37e4f67abf3d4:

  rapido-vm: add explicit lifetime syntax (2026-06-26 19:02:45 +1000)

----------------------------------------------------------------
David Disseldorp (3):
      cut/lio-local: add dmesg
      rapido-vm: don't panic if qemu is missing
      rapido-vm: add explicit lifetime syntax

 cut/lio_local.sh     |  1 +
 src/bin/rapido-vm.rs | 14 +++++++++-----
 2 files changed, 10 insertions(+), 5 deletions(-)

ddiss added 3 commits June 26, 2026 19:14
Signed-off-by: David Disseldorp <ddiss@suse.de>
Print and propagate an error instead.

Signed-off-by: David Disseldorp <ddiss@suse.de>
This addresses the following compiler warning:

156 | fn vm_qemu_args_get(conf: &HashMap<String, String>) -> io::Result<QemuArgs> {
    |                           ^^^^^^^^^^^^^^^^^^^^^^^^                ^^^^^^^^ the same lifetime is hidden here
    |                           |
    |                           the lifetime is elided here
    |
    = help: the same lifetime is referred to in inconsistent ways, making the signature confusing

The fix was suggested by rustc.

Signed-off-by: David Disseldorp <ddiss@suse.de>
@ddiss
ddiss merged commit a444e1f into rapido-linux:master Jun 26, 2026
2 checks passed
@ddiss
ddiss deleted the minor_changes branch June 26, 2026 09:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant