| Summary: | runqemu: use qemuboot.json to replace qemuboot.conf | ||
|---|---|---|---|
| Product: | [Build System, Metadata & Runtime] OE-Core | Reporter: | Robert Yang <liezhi.yang> |
| Component: | core | Assignee: | New Comer Bugs <newcomer> |
| Status: | RESOLVED INVALID | QA Contact: | |
| Severity: | enhancement | ||
| Priority: | Medium | CC: | champagne.guillaume.c, meta.mr.watcher, meta.watcher, randy.macleod |
| Version: | 2.5 | ||
| Target Milestone: | 3.4 | ||
| Hardware: | x86 | ||
| OS: | Multiple | ||
| Whiteboard: | NEWCOMER | ||
| OS type for building Yocto: | --- | Type of Regression: | --- |
| Verified: | Documentation change: | Yes (doc changes required) | |
|
Description
Robert Yang
2018-01-19 03:01:36 UTC
Hi Robert, I'm willing to make this change if no one is currently implementing it. Should backward compatibility be kept with qemuboot.conf? `runqemu` seems to accept qemuboot.conf configuration files as an argument, so is changing this to support only json configuration files considered a breaking change (or is that not a major concern?). Thanks! Guillaume, thanks. If no one responds here, try sending an RFC email to the list. Hi Randy, thank you for your answer. I ended up sending an RFC on the mailing list a month ago, but it seems that this bugzilla report was not answering to any real world need, so the discussion ended there: https://lists.openembedded.org/g/openembedded-core/topic/80577032#147978 I think this report should eventually be closed or at least removed from the newcomer bug list. On the list: https://lists.openembedded.org/g/openembedded-core/topic/80577032#147978 Ross Burton said: > I understand that you're simply fixing a newcomer bug but I just don't > see the rationale behind the bug in the first place. Copying in > Robert for his opinion. > > I'm unaware of any real world limitations of INI files that are a > problem for runqemu. We either break compatibility with existing > builds, or have two separate parsing codepaths in runqemu which > complicates things more (as the ini path won't get tested). > > Ross so I'm closing this enhancement. If there's a clear need to support qemuboot.json, please feel free to explain and re-open. Thanks to Guillaume for looking into this issue and poking me to close it. |