"Dr. David Alan Gilbert" <[email protected]> writes: > * Markus Armbruster ([email protected]) wrote: >> We compile pci-hmp-cmds.c always, but pci-qmp-cmds.c only when >> CONFIG_PCI. hw/pci/pci-stub.c keeps the linker happy when >> !CONFIG_PCI. Build pci-hmp-cmds.c that way, too. >> >> Signed-off-by: Markus Armbruster <[email protected]> > > Had you considered wrapping the hmp-commands-info.hx entry > with a #if defined instead?
No. Would you prefer that? Code containing #ifdef CONFIG_PCI is target-dependent. Looks like the affected monitor code already is, so no new headaches. Aside: splitting off its target-independent parts could be nice. Not today. [...]
