Summary: | emulators/qemu emulators/qemu-devel sysutils/dtc devel/valgrind: use system dtc in qemu builds + additional updates | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | Ports & Packages | Reporter: | Sean Champ <lab+bsd> | ||||||
Component: | Individual Port(s) | Assignee: | Muhammad Moinur Rahman <bofh> | ||||||
Status: | Closed FIXED | ||||||||
Severity: | Affects Some People | CC: | bofh, imp, swills, uqs, zeising | ||||||
Priority: | --- | Flags: | linimon:
maintainer-feedback?
(bofh) |
||||||
Version: | Latest | ||||||||
Hardware: | Any | ||||||||
OS: | Any | ||||||||
Bug Depends on: | 239669 | ||||||||
Bug Blocks: | |||||||||
Attachments: |
|
Description
Sean Champ
2020-04-28 09:55:13 UTC
In an update to the previous remarks: I may've introduced the option to build emulators/qemu with ncurses from ports, under an earlier local change. I've not tested this change with regards to the ncurses used, beyond the initial build, for emulators/qemu or emulators/qemu-devel The remark with regards to extra-ldflags at https://bugs.freebsd.org/bugzilla/attachment.cgi?id=213883&action=diff#b/emulators/qemu-devel/Makefile_sec4 should not have been published in the diff similarly, the remark at the section https://bugs.freebsd.org/bugzilla/attachment.cgi?id=213883&action=diff#b/emulators/qemu/Makefile_sec5 It's taken a number of days to put this patch together. Candidly, I'd thought I'd linted those remark out of the actual published diff. Can put together an updated diff, sometime later today Created attachment 213884 [details]
Linted patch for emulators/qemu emulators/qemu-devel sysutils/dtc devel/valgrind
I've updated the local patch branch such as to remove two ephemeral, local comments from the resulting diff. The updated patch is attached
^Triage: assign to maintainer of emulators/qemu* but also Cc: maintainers of sysutils/dtc and devel/valgrind. dtc was updated to 1.6.0 a few weeks ago. I'm not the official qemu maintainer, but do generally approve of this direction FWIW. 1. We are moving qemu-devel to SNAPSHOTS version where we cannot use that specific version as it requires HEAD of dtc. Hopefully this will be committed by today or tomorrow. So I am going to add a CONFLICT with sysutils/dtc for qemu-devel. 2. Please see the latest qemu/Makefile for ncurses implementation giving the option of both. Giving the same for qemu-devel. 3. I am not keen on OPTIONIZING DEBUG for qemu-devel although will do it for qemu. Unless there is a very good reason behind it. 4. Please run portlint -abct before submitting patches. My eyes say that qemu/Makefile and qemu-devel/Makefile should throw errors with Contiguous blank lines. 5. When submitting a bug report please try to submit multiple bugs with different ports as bugzilla automagically assigns the bugs based on portname. Multiple portname causes it to have a panick attack. Depends or Blocks is easier to handle and easier to track. A commit references this bug: Author: bofh Date: Sat Sep 12 22:25:07 UTC 2020 New revision: 548428 URL: https://svnweb.freebsd.org/changeset/ports/548428 Log: emulators/qemu-devel: Update version 4.2.0=>5.1.0.20200909 - Move to SNAPSHOT versions of qemu - Remove i386 support temporarily as build is failing on i386 - Change MASTER_SITES to GITHUB - Mark IGNORE for 11.X as it cannot build Position Independent Code on 11 - OPTIONIZE NCURSES for support of both base and ports [1] - Add OPTIONS CAPSTONE and use CAPSTONE from ports - Remove PCAP related patches as was unable to create PANIC while using TSO - Fixes for the RISC-V PLIC that are required to boot FreeBSD/riscv64 HEAD after r362977 PR: 245997 [1] Submitted by: lab+bsd@thinkum.space [1] Reported by: lwhsu jhb Changes: head/emulators/qemu-devel/Makefile head/emulators/qemu-devel/distinfo head/emulators/qemu-devel/files/patch-Makefile head/emulators/qemu-devel/files/patch-configure head/emulators/qemu-devel/files/patch-docs_system_qemu-cpu-models.rst head/emulators/qemu-devel/files/patch-include_net_net.h head/emulators/qemu-devel/files/patch-include_qemu_atomic.h head/emulators/qemu-devel/files/patch-qemu-doc.texi head/emulators/qemu-devel/files/pcap-patch-net_clients.h head/emulators/qemu-devel/files/pcap-patch-net_net.c head/emulators/qemu-devel/files/pcap-patch-qapi_net.json head/emulators/qemu-devel/pkg-plist This seems to happen with emulators/qemu50 as well. (In reply to Steve Wills from comment #9) It's better not to use qemu50 as it is a placeholder for qemu 5.0.X branches in the future when 5.1.X or 5.2.X or 6.X branch moves to stable qemu will be moved to qemu50. In fact that is a reason you sill see that qemu is on 5.0.1 but qemu is still on 5.0.0. Looking forward to work on qemu-devel in the upcoming weekend. (In reply to Muhammad Moinur Rahman from comment #10) Ports shouldn't just be broken and left that way. If it's there, it should build. If it shouldn't be used, it should be marked deprecated and removed. Also, I'm still seeing this issue for emulators/qemu as well. (In reply to Steve Wills from comment #12) Give me the weekend to check this. (In reply to Steve Wills from comment #12) Can you share me some build logs? I can't find it in my latest builds. (In reply to Muhammad Moinur Rahman from comment #14) I sent a link to the log via email, please let me know if you didn't get it. A commit in branch main references this bug: URL: https://cgit.FreeBSD.org/ports/commit/?id=9e06cd0454806020fc8e2f11df85baf500690902 commit 9e06cd0454806020fc8e2f11df85baf500690902 Author: Muhammad Moinur Rahman <bofh@FreeBSD.org> AuthorDate: 2021-04-20 19:41:51 +0000 Commit: Muhammad Moinur Rahman <bofh@FreeBSD.org> CommitDate: 2021-04-20 19:46:11 +0000 emulators/qemu: Fix build on 13 - Use ports dtc instead of packaged dtc [1] - bin/elf2dmp apparently depends on CURL [2] - Reduce build to only X86 architectures if X86_TARGETS is set [2] - rx is not an X86 architecture [2] PR: 245997 [1] 254385 [2] Reported by: swills [1] pstef [2] emulators/qemu/Makefile | 13 +++++++++++-- emulators/qemu/pkg-plist | 6 ++++-- 2 files changed, 15 insertions(+), 4 deletions(-) Thanks for making attention to the patch. I'll try to take a look at this portlint tool, before sending more patches to the bugzilla |