Bug 494471
Summary: | Close button doesn't work in qt4.5 + needs update | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Orcan Ogetbil <oget.fedora> | ||||||
Component: | qjackctl | Assignee: | Orcan Ogetbil <oget.fedora> | ||||||
Status: | CLOSED NEXTRELEASE | QA Contact: | Fedora Extras Quality Assurance <extras-qa> | ||||||
Severity: | medium | Docs Contact: | |||||||
Priority: | low | ||||||||
Version: | rawhide | CC: | kevin, nando, rdieter | ||||||
Target Milestone: | --- | Keywords: | Patch | ||||||
Target Release: | --- | ||||||||
Hardware: | All | ||||||||
OS: | Linux | ||||||||
Whiteboard: | |||||||||
Fixed In Version: | 0.3.4-1.fc11 | Doc Type: | Bug Fix | ||||||
Doc Text: | Story Points: | --- | |||||||
Clone Of: | Environment: | ||||||||
Last Closed: | 2009-04-28 01:18:54 UTC | Type: | --- | ||||||
Regression: | --- | Mount Type: | --- | ||||||
Documentation: | --- | CRM: | |||||||
Verified Versions: | Category: | --- | |||||||
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |||||||
Cloudforms Team: | --- | Target Upstream Version: | |||||||
Embargoed: | |||||||||
Attachments: |
|
Description
Orcan Ogetbil
2009-04-07 02:58:21 UTC
Created attachment 338452 [details]
close button fix (qt4.5)
Created attachment 338453 [details]
fix locale path
When/if new builds are done against qt-4.5, I'd also suggest adding (something like): Requires: qt4 >= %{_qt4_version} and it can be added to pending qt45 updates F-9: https://admin.fedoraproject.org/updates/F9/FEDORA-2009-3356 F-10: https://admin.fedoraproject.org/updates/F10/FEDORA-2009-3389 (or pushed separately, provided it doesn't land in stable before qt45 does). I'm not convinced the locale path patch really makes sense. The spirit of the "%{_datadir}/locale/* is forbidden" guideline is that all language-specific files need to be qualified with %lang for that language (e.g. %lang(de)), which the %find_lang macro does for you (for standard .mo files). So throwing those files into a subdirectory does not actually fix anything. What is really needed is a %find_lang-type macro for Qt translations. (That or get the upstream projects to use gettext instead like the rest of the world, including KDE, does. But I'm sure there will always be projects using Qt's translation mechanism, starting from Qt itself.) IMHO we should get the Qt 4.5 fixes out ASAP and leave the locale issues for a later cleanup when we're not in a hurry. You should also file a separate bug for the locale stuff, 2 issues in one bug report = bad. The patch as it is there is IMHO wrong. In addition: do we actually want to push out that version upgrade? What does it fix? I'd rather just apply the Qt 4.5 patch to what we currently have unless the maintainer agrees to an upgrade. So, installing .qm files directly in /usr/share/locale/ = right ? If you look closely, the patch does not put them in a subdirectory of /usr/share/locale/; it puts them into /usr/share/%{name}/locale/ As for the upgrade, this is mostly a bugfix release. I find it useful. But you're right, it's up to the packager. Kevin, did you see my question above? As far as I can tell, there are many other packages, which put their locale files inside %{_datadir}/%{name}/locale or %{_datadir}/%{name}/lang or something similar. Examples include: amsn, muse, lmms, ... Why is it right to put the non-gettext language files inside %{_datadir}/%{name} for these packages, but not for qjackctl? Right, we can do that. (Ideally, the maintainer should do it, but he appears to be AWOL. :-( His fedoraproject.org e-mail address bounces, indicating he hasn't renewed his password as of now.) They should still get %lang-marked as well though. Right now, my priority was to get a build which works with Qt 4.5 into the Qt 4.5 update set ASAP. qjackctl-0.3.4-1.fc11 has been submitted as an update for Fedora 11. http://admin.fedoraproject.org/updates/qjackctl-0.3.4-1.fc11 I think we are done with this. qjackctl-0.3.4-1.fc11 has been pushed to the Fedora 11 stable repository. If problems still persist, please make note of it in this bug report. |