Skip to content

Conversation

@main--
Copy link
Owner

@main-- main-- commented Aug 17, 2017

🔉 Fresh meat!

cc @Ediacarium

@main-- main-- added this to the Beta release milestone Aug 17, 2017
@oberien
Copy link
Collaborator

oberien commented Aug 18, 2017

I miss the amazing usb picker .

Copy link
Collaborator

@Ediacarium Ediacarium left a comment

Choose a reason for hiding this comment

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

I love a good shoulder

}
*/

fn write_vfio_modconf<W: Write>(machine: &MachineConfig, mut w: W) -> Result<()> {
Copy link
Collaborator

Choose a reason for hiding this comment

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

Shouldn't we make it consistent with the USB write_udev_rules?

Like, either use the config or get a list of devices

Copy link
Owner Author

Choose a reason for hiding this comment

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

perhaps

@Ediacarium This sucks, how about we don't do it?

OptString(Property<Option<String>>),

Container {
Copy link
Collaborator

@oberien oberien Sep 16, 2017

Choose a reason for hiding this comment

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

This should be named Struct.

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