Bug 650995
| Summary: | [Compiz] Unable to rename printer | ||||||
|---|---|---|---|---|---|---|---|
| Product: | [Fedora] Fedora | Reporter: | Tim Waugh <twaugh> | ||||
| Component: | system-config-printer | Assignee: | Tim Waugh <twaugh> | ||||
| Status: | CLOSED ERRATA | QA Contact: | Fedora Extras Quality Assurance <extras-qa> | ||||
| Severity: | medium | Docs Contact: | |||||
| Priority: | low | ||||||
| Version: | 14 | CC: | adel.gadllah, hidoufr, james, jpopelka, plarsen, twaugh | ||||
| Target Milestone: | --- | Keywords: | Reopened | ||||
| Target Release: | --- | ||||||
| Hardware: | Unspecified | ||||||
| OS: | Unspecified | ||||||
| Whiteboard: | |||||||
| Fixed In Version: | system-config-printer-1.2.7-2.fc13 | Doc Type: | Bug Fix | ||||
| Doc Text: | Story Points: | --- | |||||
| Clone Of: | Environment: | ||||||
| Last Closed: | 2011-02-07 20:00:43 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
Tim Waugh
2010-11-08 15:49:39 UTC
Hi Tim, It indeed doesn't look like a compiz issue as I have been able to reproduce this issue without activating compiz (try and insist sometimes you will be able to trigger it).m I see you are the maintainer of s-c-p, can I send it back to you please ? (Could it be an issue with GTK indeed ? ..) Regards Hamidou Dia -- Fedora Bugzappers volunteer triage team https://fedoraproject.org/wiki/BugZappers After some more playing I've found the problem. http://git.fedorahosted.org/git/?p=system-config-printer.git;a=commitdiff;h=3648725ac532ecb938118bb27834f3abfbc0341e (In reply to comment #2) > http://git.fedorahosted.org/git/?p=system-config-printer.git;a=commitdiff;h=3648725ac532ecb938118bb27834f3abfbc0341e Patch works for me here (system-config-printer-1.2.5-6.fc14.x86_64). system-config-printer-1.2.5-8.fc14 has been submitted as an update for Fedora 14. https://admin.fedoraproject.org/updates/system-config-printer-1.2.5-8.fc14 system-config-printer-1.2.5-8.fc13 has been submitted as an update for Fedora 13. https://admin.fedoraproject.org/updates/system-config-printer-1.2.5-8.fc13 system-config-printer-1.2.5-8.fc13 has been pushed to the Fedora 13 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing update system-config-printer'. You can provide feedback for this update here: https://admin.fedoraproject.org/updates/system-config-printer-1.2.5-8.fc13 system-config-printer-1.2.5-8.fc14 has been pushed to the Fedora 14 stable repository. If problems still persist, please make note of it in this bug report. This still fails for me. Even 1.2.6.1 does not update correct and segfaults. Steps: 1. Open system-config-priner 2. Select (left click) on a printer 3. Right click and choose rename 4. Editing does NOT show key-in 5. Hit enter 6. The field now takes input 7. Type something 8. Hit enter 9. Name is now what was entered "blindly" 10. Repeat rename 11. On second "enter" -> segfault System - F13 system-config-printer-1.2.6.1 $ system-config-printer /usr/share/system-config-printer/system-config-printer.py:3492: GtkWarning: gdk_window_new(): parent is destroyed self.view_area_vbox.show_all () /usr/share/system-config-printer/system-config-printer.py:3492: GtkWarning: gdk_window_set_user_data: assertion `GDK_IS_WINDOW (window)' failed self.view_area_vbox.show_all () /usr/share/system-config-printer/system-config-printer.py:3492: GtkWarning: gdk_drawable_get_size: assertion `GDK_IS_DRAWABLE (drawable)' failed self.view_area_vbox.show_all () /usr/share/system-config-printer/system-config-printer.py:3492: GtkWarning: Native Windows wider or taller than 65535 pixels are not supported self.view_area_vbox.show_all () /usr/share/system-config-printer/system-config-printer.py:3492: GtkWarning: gtk_style_attach: assertion `window != NULL' failed self.view_area_vbox.show_all () /usr/share/system-config-printer/system-config-printer.py:3492: GtkWarning: gdk_drawable_get_colormap: assertion `GDK_IS_DRAWABLE (drawable)' failed self.view_area_vbox.show_all () /usr/share/system-config-printer/system-config-printer.py:3492: GtkWarning: gdk_window_set_background: assertion `GDK_IS_WINDOW (window)' failed self.view_area_vbox.show_all () Segmentation fault (core dumped) Created attachment 473952 [details]
Core dump from using system-config-printer to rename printers
Renaming prompts twice. Only the first input is "used" but not reflected by system-config-printer until refreshed. If the same printer is attempted to be renamed twice the segdump happens.
OK, reverted. The grab_focus() call is what was preventing editing from actually starting. :-( Seems to also work in compiz now (tested 611d90982f3703e292918f6cbffadc4ed5651b15). system-config-printer-1.2.6-3.fc14 has been submitted as an update for Fedora 14. https://admin.fedoraproject.org/updates/system-config-printer-1.2.6-3.fc14 system-config-printer-1.2.6-3.fc13 has been pushed to the Fedora 13 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing update system-config-printer'. You can provide feedback for this update here: https://admin.fedoraproject.org/updates/system-config-printer-1.2.6-3.fc13 system-config-printer-1.2.6-3.fc14 has been pushed to the Fedora 14 stable repository. If problems still persist, please make note of it in this bug report. system-config-printer-1.2.7-2.fc13 has been pushed to the Fedora 13 stable repository. If problems still persist, please make note of it in this bug report. This has shown up again with system-config-printer-1.2.8-1.fc14.i686. Might the more stable solution be to just show a dialogue prompting for the new name, or add it as a new field in the printer's properties window? It would be nice to fix whatever the real bug is though. Failing that maybe it would be better to remove this feature altogether... |