[Bug 275636] devel/qtcreator: Bump revision, use llvm17 as recommended.
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=275636 Jason E. Hale changed: What|Removed |Added Flags|maintainer-feedback?(kde@Fr |maintainer-feedback+ |eeBSD.org) | Resolution|--- |FIXED Status|Open|Closed --- Comment #8 from Jason E. Hale --- Committed, thanks! -- You are receiving this mail because: You are on the CC list for the bug. You are the assignee for the bug.
[Bug 275636] devel/qtcreator: Bump revision, use llvm17 as recommended.
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=275636 Jason E. Hale changed: What|Removed |Added Attachment #246985|maintainer-approval?(kde@Fr |maintainer-approval+ Flags|eeBSD.org) | -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug.
[Bug 275636] devel/qtcreator: Bump revision, use llvm17 as recommended.
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=275636 --- Comment #7 from commit-h...@freebsd.org --- A commit in branch main references this bug: URL: https://cgit.FreeBSD.org/ports/commit/?id=a75fb0906770e806bcb69c2796bff9dbe0a1181a commit a75fb0906770e806bcb69c2796bff9dbe0a1181a Author: Florian Walpen AuthorDate: 2023-11-25 22:13:54 + Commit: Jason E. Hale CommitDate: 2023-12-12 19:44:16 + devel/qtcreator: Use llvm17 as recommended QtCreator would crash in the settings dialog if not rebuilt after an update of Qt packages to 6.6.1 release. Make build and run depend on llvm17 as recommended upstream, for better clangd compatibility: https://www.qt.io/blog/qt-creator-12-c-code-model-update Since llvm17 seems to be more strict about non-standard alloca stack allocation, patch that out of the internal SQLite build config. This matches the config of databases/sqlite3 in ports. PR: 275636 devel/qtcreator/Makefile | 3 ++- .../files/patch-src_libs_3rdparty_sqlite_config.h (new)| 10 ++ 2 files changed, 12 insertions(+), 1 deletion(-) -- You are receiving this mail because: You are on the CC list for the bug. You are the assignee for the bug.
[Bug 275636] devel/qtcreator: Bump revision, use llvm17 as recommended.
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=275636 --- Comment #6 from Jason E. Hale --- (In reply to Florian Walpen from comment #5) I'm fine with that, although I don't think using the base compiler will be an issue as that was the behavior before the aforementioned change to Mk/Uses/llvm.mk. For expediency, I'll just commit your patch as-is, because I have a few other pots on the boil and don't have time to test further changes today. -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug.
[Bug 275636] devel/qtcreator: Bump revision, use llvm17 as recommended.
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=275636 --- Comment #5 from Florian Walpen --- (In reply to Jason E. Hale from comment #4) Thanks for the hint, llvm:noexport does find the ccache symlink, but uses compiler from base instead of llvm17. That may not matter anymore, since QtCreator now uses external clangd language servers. But I'd still prefer to test drive such a change for some time. I'd suggest to commit this patch as is, ASAP for the crash fix. QtCreator 12.0.1 just came out, and we could include the switch to base compiler / ccache with that update. -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug.
[Bug 275636] devel/qtcreator: Bump revision, use llvm17 as recommended.
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=275636 --- Comment #4 from Jason E. Hale --- (In reply to Florian Walpen from comment #3) We probably need to add USES=llvm:noexport to fix the ccache issue because after [1], CC/CXX/CPP are exported by default. [1] https://cgit.freebsd.org/ports/commit/Mk/Uses/llvm.mk?id=4df0e66a3662ac9d427c2dcd79eb8555236083dd -- You are receiving this mail because: You are on the CC list for the bug. You are the assignee for the bug.
[Bug 275636] devel/qtcreator: Bump revision, use llvm17 as recommended.
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=275636 --- Comment #3 from Florian Walpen --- Regarding build times BTW, devel/qtcreator build used to pick up ccache previously, but doesn't anymore. Maybe for some weeks now, can't pinpoint exactly. Any ideas? I added clang16 and clang17 to the ccache symlink list, to no avail. But the issue was already present with clang15. -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug.
maintainer-approval requested: [Bug 275636] devel/qtcreator: Bump revision, use llvm17 as recommended. : [Attachment 246985] Bump revision for Qt 6.6.1, use llvm17 as recommended.
Florian Walpen has asked freebsd-kde (group) for maintainer-approval: Bug 275636: devel/qtcreator: Bump revision, use llvm17 as recommended. https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=275636 Attachment 246985: Bump revision for Qt 6.6.1, use llvm17 as recommended. https://bugs.freebsd.org/bugzilla/attachment.cgi?id=246985=edit --- Comment #2 from Florian Walpen --- Created attachment 246985 --> https://bugs.freebsd.org/bugzilla/attachment.cgi?id=246985=edit Bump revision for Qt 6.6.1, use llvm17 as recommended. Thanks for the warning, I wasn't aware that there are other reasons to disable PCH apart from the original build failure. Omitted from the patch, let's keep PCH disabled.
[Bug 275636] devel/qtcreator: Bump revision, use llvm17 as recommended.
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=275636 Florian Walpen changed: What|Removed |Added Attachment #246908|maintainer-approval?(kde@Fr | Flags|eeBSD.org) | Attachment #246908|0 |1 is obsolete|| Attachment #246985||maintainer-approval?(kde@Fr Flags||eeBSD.org) --- Comment #2 from Florian Walpen --- Created attachment 246985 --> https://bugs.freebsd.org/bugzilla/attachment.cgi?id=246985=edit Bump revision for Qt 6.6.1, use llvm17 as recommended. Thanks for the warning, I wasn't aware that there are other reasons to disable PCH apart from the original build failure. Omitted from the patch, let's keep PCH disabled. -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug.
[Bug 275636] devel/qtcreator: Bump revision, use llvm17 as recommended.
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=275636 Jason E. Hale changed: What|Removed |Added Status|New |Open CC||jh...@freebsd.org --- Comment #1 from Jason E. Hale --- Looks good, although I'd be a bit hesitant to enable PCH without thorough testing, especially for non-amd64/i386 platforms as they have a tendency to cause build errors and can significantly slow down the build even on amd64 as they do with www/qt6-webengine. I haven't done any benchmarking with qtcreator, though. For the time being, I'd rather just leave it as is or make it a non-default option for advanced users. -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug.
maintainer-approval requested: [Bug 275636] devel/qtcreator: Bump revision, use llvm17 as recommended. : [Attachment 246908] Bump revision for Qt 6.6.1, use llvm17 as recommended.
Florian Walpen has asked freebsd-kde (group) for maintainer-approval: Bug 275636: devel/qtcreator: Bump revision, use llvm17 as recommended. https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=275636 Attachment 246908: Bump revision for Qt 6.6.1, use llvm17 as recommended. https://bugs.freebsd.org/bugzilla/attachment.cgi?id=246908=edit --- Description --- After an update of Qt packages to 6.6.1, QtCreator would reproducibly crash in some parts of the settings dialog. Rebuild fixes that, thus bump port revision. Builds fine now without disabling precompiled headers, so remove that flag. Build and run depend on llvm17 as recommended upstream, for better clangd compatibility: https://www.qt.io/blog/qt-creator-12-c-code-model-update Previously llvm15 was chosen because it's usually already installed as a dependency of mesa. But I suppose having less trouble with clangd is worth using llvm17 instead. Since llvm17 seems to be more strict about non-standard alloca stack allocation, patch that out of the internal SQLite build config. This matches the config of databases/sqlite3 in ports.
maintainer-feedback requested: [Bug 275636] devel/qtcreator: Bump revision, use llvm17 as recommended.
Bugzilla Automation has asked freebsd-kde (group) for maintainer-feedback: Bug 275636: devel/qtcreator: Bump revision, use llvm17 as recommended. https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=275636 --- Description --- After an update of Qt packages to 6.6.1, QtCreator would reproducibly crash in some parts of the settings dialog. Rebuild fixes that, thus bump port revision. Builds fine now without disabling precompiled headers, so remove that flag. Build and run depend on llvm17 as recommended upstream, for better clangd compatibility: https://www.qt.io/blog/qt-creator-12-c-code-model-update Previously llvm15 was chosen because it's usually already installed as a dependency of mesa. But I suppose having less trouble with clangd is worth using llvm17 instead. Since llvm17 seems to be more strict about non-standard alloca stack allocation, patch that out of the internal SQLite build config. This matches the config of databases/sqlite3 in ports.
[Bug 275636] devel/qtcreator: Bump revision, use llvm17 as recommended.
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=275636 Bug ID: 275636 Summary: devel/qtcreator: Bump revision, use llvm17 as recommended. Product: Ports & Packages Version: Latest Hardware: Any URL: https://www.qt.io/blog/qt-creator-12-c-code-model-upda te OS: Any Status: New Severity: Affects Only Me Priority: --- Component: Individual Port(s) Assignee: k...@freebsd.org Reporter: d...@submerge.ch CC: k...@freebsd.org Flags: maintainer-feedback?(k...@freebsd.org) Assignee: k...@freebsd.org Attachment #246908 maintainer-approval?(k...@freebsd.org) Flags: CC: k...@freebsd.org Created attachment 246908 --> https://bugs.freebsd.org/bugzilla/attachment.cgi?id=246908=edit Bump revision for Qt 6.6.1, use llvm17 as recommended. After an update of Qt packages to 6.6.1, QtCreator would reproducibly crash in some parts of the settings dialog. Rebuild fixes that, thus bump port revision. Builds fine now without disabling precompiled headers, so remove that flag. Build and run depend on llvm17 as recommended upstream, for better clangd compatibility: https://www.qt.io/blog/qt-creator-12-c-code-model-update Previously llvm15 was chosen because it's usually already installed as a dependency of mesa. But I suppose having less trouble with clangd is worth using llvm17 instead. Since llvm17 seems to be more strict about non-standard alloca stack allocation, patch that out of the internal SQLite build config. This matches the config of databases/sqlite3 in ports. -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug.