https://bugzilla.redhat.com/show_bug.cgi?id=2267876 loise(a)kde.org <loise(a)kde.org&gt; changed: What |Removed |Added ---------------------------------------------------------------------------- Doc Type|--- |If docs needed, set a value --- Comment #2 from loise(a)kde.org <loise(a)kde.org&gt; --- SPEC: https://download.copr.fedorainfracloud.org/results/loise/update-testing/f... SRPM: https://download.copr.fedorainfracloud.org/results/loise/update-testing/f... -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component https://bugzilla.redhat.com/show_bug.cgi?id=2267876 Report this comment as SPAM: https://bugzilla.redhat.com/enter_bug.cgi?product=Bugzilla&format=rep...

https://bugzilla.redhat.com/show_bug.cgi?id=2267876 Jan Grulich <jgrulich(a)redhat.com&gt; changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |jgrulich(a)redhat.com Assignee|nobody(a)fedoraproject.org |jgrulich(a)redhat.com Status|NEW |ASSIGNED -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component https://bugzilla.redhat.com/show_bug.cgi?id=2267876

https://bugzilla.redhat.com/show_bug.cgi?id=2267876 Jan Grulich <jgrulich(a)redhat.com&gt; changed: What |Removed |Added ---------------------------------------------------------------------------- Flags| |needinfo?(loise(a)kde.org) --- Comment #4 from Jan Grulich <jgrulich(a)redhat.com&gt; --- 1) Looks like BR: qt6-qtbase-static and qt6-qtdeclarative-static are not needed? At least I can build it without it and don't see any complain 2) The commented workaround can be dropped if the issue was fixed in GCC? 3) Was there an alternative of the cooker binary in Qt5? If not, we might not need to add the -qt6 suffix? 4) I think the thing checking "QMAKE_PRL_BUILD_DIR" is not needed? I guess it's not needed in any of Qt6 package we have and should be perhaps removed? I'm guessing because it's qmake related and we don't use qmake anymore. I also don't see any *.la library being installed. 5) Shouldn't this package own %{_qt6_libdir}/cmake/Qt6Quick3DPhysics and similar directories? -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component https://bugzilla.redhat.com/show_bug.cgi?id=2267876 Report this comment as SPAM: https://bugzilla.redhat.com/enter_bug.cgi?product=Bugzilla&format=rep...

https://bugzilla.redhat.com/show_bug.cgi?id=2267876 --- Comment #6 from Jan Grulich <jgrulich(a)redhat.com&gt; --- Please, fix it and post here updated SPEC file :). -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component https://bugzilla.redhat.com/show_bug.cgi?id=2267876 Report this comment as SPAM: https://bugzilla.redhat.com/enter_bug.cgi?product=Bugzilla&format=rep...

https://bugzilla.redhat.com/show_bug.cgi?id=2267876 --- Comment #8 from Jan Grulich <jgrulich(a)redhat.com&gt; --- I think we need to use "%dir path" for every directory owned by the package. E.g. %dir %{_qt6_libdir}/cmake/Qt6Quick3DPhysics/ %{_qt6_libdir}/cmake/Qt6Quick3DPhysics/*.cmake At least that's my understanding of this macro and that without it, when uninstalling, it would remove all the files in that directory, except the directory itself. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component https://bugzilla.redhat.com/show_bug.cgi?id=2267876 Report this comment as SPAM: https://bugzilla.redhat.com/enter_bug.cgi?product=Bugzilla&format=rep...

https://bugzilla.redhat.com/show_bug.cgi?id=2267876 --- Comment #10 from Jan Grulich <jgrulich(a)redhat.com&gt; --- Doesn't it report now that files are listed twice? :) -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component https://bugzilla.redhat.com/show_bug.cgi?id=2267876 Report this comment as SPAM: https://bugzilla.redhat.com/enter_bug.cgi?product=Bugzilla&format=rep...

https://bugzilla.redhat.com/show_bug.cgi?id=2267876 Jan Grulich <jgrulich(a)redhat.com&gt; changed: What |Removed |Added ---------------------------------------------------------------------------- Flags|fedora-review? |fedora-review+ --- Comment #12 from Jan Grulich <jgrulich(a)redhat.com&gt; --- Looks good to me now. Package approved. -- You are receiving this mail because: You are always notified about changes to this product and component You are on the CC list for the bug. https://bugzilla.redhat.com/show_bug.cgi?id=2267876 Report this comment as SPAM: https://bugzilla.redhat.com/enter_bug.cgi?product=Bugzilla&format=rep...

https://bugzilla.redhat.com/show_bug.cgi?id=2267876 Fedora Admin user for bugzilla script actions <fedora-admin-xmlrpc(a)fedoraproject.org&gt; changed: What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |POST --- Comment #13 from Fedora Admin user for bugzilla script actions <fedora-admin-xmlrpc(a)fedoraproject.org&gt; --- The Pagure repository was created at https://src.fedoraproject.org/rpms/qt6-qtquick3dphysics -- You are receiving this mail because: You are always notified about changes to this product and component You are on the CC list for the bug. https://bugzilla.redhat.com/show_bug.cgi?id=2267876 Report this comment as SPAM: https://bugzilla.redhat.com/enter_bug.cgi?product=Bugzilla&format=rep...