aqemu icon indicating copy to clipboard operation
aqemu copied to clipboard

Removed writing 'vlan' from parameter string

Open arthemis95 opened this issue 5 years ago • 3 comments

Basically just removing the vlan parameter from the qemu arguments since it's depreciated. I did only change actually putting it in the string, the gui doesn't reflect the changes.

arthemis95 avatar Mar 05 '19 10:03 arthemis95

@pcwizzy37 completely removes vlan with this commit https://github.com/pcwizzy37/aqemu/commit/37d5447126343cc7a70b95c6e73d670be444a05d

TBK avatar Mar 25 '19 12:03 TBK

I would accept a patch that fixes the GUI and all issues associated with the removal of vlan. There has to be an alternative way of doing those things, or QEMU wouldn't remove that feature, I'd think.

tobimensch avatar Apr 30 '20 09:04 tobimensch

How many people would use the vlan replacement? I don't. In issue https://github.com/tobimensch/aqemu/issues/58#issue-358112099 might contain the answer for a replacement.

I found out this commit, which removes the vlan parameter: qemu/qemu@af1a5c3

the commit message says...

...should have been enough time for everybody to either just drop unnecessary "vlan=0" parameters, to switch to the modern -device + -netdev syntax for connecting guest NICs with host network backends, or to switch to the "hubport" netdev in case hubs are really wanted instead.

So how hard would it be to implement a replacement to fix this issue? To me it doesn't seem that hard but, things always seem easier than they are to me.

jake01756 avatar May 14 '20 20:05 jake01756