Bug 588319 - f13 new icons for setroubleshoot
Summary: f13 new icons for setroubleshoot
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: setroubleshoot
Version: 13
Hardware: All
OS: Linux
low
medium
Target Milestone: ---
Assignee: Daniel Walsh
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: f13artwork
TreeView+ depends on / blocked
 
Reported: 2010-05-03 13:30 UTC by Máirín Duffy
Modified: 2010-05-10 14:49 UTC (History)
6 users (show)

Fixed In Version: setroubleshoot-2.2.81-1.fc13
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2010-05-07 05:47:20 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)
new icons for setroubleshoot (3.90 KB, application/x-bzip2)
2010-05-03 13:30 UTC, Máirín Duffy
no flags Details
icons with a red bolt for critical setroubleshoot alerts (3.85 KB, application/x-bzip)
2010-05-04 19:33 UTC, Máirín Duffy
no flags Details
new icons, including the plain and criticals (22.08 KB, application/x-bzip)
2010-05-06 19:58 UTC, Máirín Duffy
no flags Details
comp showing new icons in PNG format for quick viewing (16.75 KB, image/png)
2010-05-06 20:01 UTC, Máirín Duffy
no flags Details
sorry i forgot the scalable versions in the last upload (29.37 KB, application/x-bzip)
2010-05-06 20:13 UTC, Máirín Duffy
no flags Details

Description Máirín Duffy 2010-05-03 13:30:59 UTC
Created attachment 410986 [details]
new icons for setroubleshoot

Description of problem:

The current setroubleshoot icons are bluecurve and outdated. Here's some icons created to match the rest of the Fedora icon theme.

Comment 1 Máirín Duffy 2010-05-04 19:33:02 UTC
Created attachment 411388 [details]
icons with a red bolt for critical setroubleshoot alerts

Comment 2 Máirín Duffy 2010-05-04 19:33:31 UTC
just a note, these icons aren't from upstream, I created them specifically for setroubleshoot from scratch. Whatever license / etc works is fine.

Comment 3 Daniel Walsh 2010-05-04 20:07:38 UTC
We are upstream

Comment 4 Máirín Duffy 2010-05-05 00:46:21 UTC
Hi Dan! Sorry for the vague comment. I meant the upstream gnome-icon-theme project, which these icons are attempting to fit in with!

Comment 5 Paul W. Frields 2010-05-05 18:44:06 UTC
Hi Dan, is there a new build coming with these icons fixed?  We're at the wire to get blocking bugs fixed, and this is one of them according to our release criteria:

https://fedoraproject.org/wiki/Fedora_13_Final_Release_Criteria

Thanks for your time as always.

Comment 6 Daniel Walsh 2010-05-05 20:52:24 UTC
Paul, I am afraid this one is waiting on Máirín, as soon as she gets me icons, I will build a package.

Comment 7 Daniel Walsh 2010-05-05 22:20:43 UTC
Updated in setroubleshoot-2.2.79-1.fc13

Please test....

Comment 8 Kamil Páral 2010-05-06 12:11:45 UTC
(In reply to comment #7)
> Updated in setroubleshoot-2.2.79-1.fc13
> 
> Please test....    

The menu item is updated. The browser frame does has only stock (=no) icon, is that correct?

How can I trigger some alert so I see the notification area applet?

Comment 9 James Laska 2010-05-06 12:16:17 UTC
(In reply to comment #7)
> Updated in setroubleshoot-2.2.79-1.fc13
> 
> Please test....    

Confirmed new icon for "Applications -> System Tools -> SELinux troubleshooter "

Confirmed kparal's finding that the application icon does not match.

Also, please note that a bodhi update will need to be submitted for this fix to arrive in Fedora 13.  Thanks!

Comment 10 Daniel Walsh 2010-05-06 12:59:03 UTC
No that is a bug.


sandbox rpm -qa 

Should trigger avc messages

Comment 11 Daniel Walsh 2010-05-06 13:47:29 UTC
Updated in setroubleshoot-2.2.80-1.fc13

Comment 12 Fedora Update System 2010-05-06 13:49:48 UTC
setroubleshoot-2.2.80-1.fc13 has been submitted as an update for Fedora 13.
http://admin.fedoraproject.org/updates/setroubleshoot-2.2.80-1.fc13

Comment 13 Kamil Páral 2010-05-06 14:14:28 UTC
Ok, now even application has the icon. I don't like it being so blurry. 24x24px is good for menu item, but not for application icon (when switching between apps). Also scalable icon should be present, I suppose it was done as vector icon anyway.

'sandbox rpm -qa' did not trigger any AVCs, so I couldn't test the applet. Any other way?

Comment 14 Daniel Walsh 2010-05-06 14:46:56 UTC
Kamil, The icon developers are working to get me a larger icon.  As of now the best I have is 24x24.

Will fix when new icons are ready.  You might need to logout to get new version to work.  Or kill and restart seapplet.

Comment 15 Máirín Duffy 2010-05-06 19:58:31 UTC
Created attachment 412147 [details]
new icons, including the plain and criticals

Comment 16 Máirín Duffy 2010-05-06 20:01:39 UTC
Created attachment 412149 [details]
comp showing new icons in PNG format for quick viewing

Comment 17 Máirín Duffy 2010-05-06 20:13:31 UTC
Created attachment 412152 [details]
sorry i forgot the scalable versions in the last upload

Comment 18 Daniel Walsh 2010-05-06 21:12:13 UTC
Updated in setroubleshoot-2.2.81-1.fc13

Comment 19 Fedora Update System 2010-05-07 05:47:16 UTC
setroubleshoot-2.2.81-1.fc13 has been pushed to the Fedora 13 stable repository.  If problems still persist, please make note of it in this bug report.

Comment 20 Kamil Páral 2010-05-07 06:31:19 UTC
The new icon set is present. Unfortunately I wasn't able to test notification area applet, because I don't know how to trigger avc messages (sandbox rpm -qa doesn't work).

Comment 21 Daniel Walsh 2010-05-07 20:47:48 UTC
kamil log out and log back in should start the process again.

Comment 22 Kamil Páral 2010-05-10 09:16:21 UTC
(In reply to comment #21)
> kamil log out and log back in should start the process again.    

I have done complete machine reboot after setroubleshoot upgrade. That command does not trigger any avc denials for me. It just prints "error: cannot open Packages database in /var/lib/rpm" twice. Nothing in audit.log either.

Comment 23 Daniel Walsh 2010-05-10 14:28:28 UTC
Make sure auditing is running.

service auditd restart

There is some bugs in readahead that can turn off the audit subsystem.

Are you seeing avc messages in /var/log/messages?

Comment 24 Kamil Páral 2010-05-10 14:49:01 UTC
Oh, that was it, auditd was not configured to run. Ok, now it throws avc messages and applet icon looks good, thanks. Verified.


Note You need to log in before you can comment on or make changes to this bug.