diff mbox

policycoreutils: Use "new" sepolicy icon in .desktop file for sepolicy gui

Message ID 20161102130437.15464-1-bigon@debian.org (mailing list archive)
State Not Applicable
Headers show

Commit Message

Laurent Bigonville Nov. 2, 2016, 1:04 p.m. UTC
From: Laurent Bigonville <bigon@bigon.be>

New hires icons have been added to the repository end 2013 but the
sepolicy .desktop file is not using it, let's fix that.
---
 policycoreutils/sepolicy/sepolicy.desktop | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Stephen Smalley Nov. 2, 2016, 1:27 p.m. UTC | #1
On 11/02/2016 09:04 AM, Laurent Bigonville wrote:
> From: Laurent Bigonville <bigon@bigon.be>
> 
> New hires icons have been added to the repository end 2013 but the
> sepolicy .desktop file is not using it, let's fix that.

Thanks, applied.  I plan to move these icon files over to the gui/
directory in the splitpolicycoreutils branch.

> ---
>  policycoreutils/sepolicy/sepolicy.desktop | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/policycoreutils/sepolicy/sepolicy.desktop b/policycoreutils/sepolicy/sepolicy.desktop
> index f9d0cec..181a1c5 100644
> --- a/policycoreutils/sepolicy/sepolicy.desktop
> +++ b/policycoreutils/sepolicy/sepolicy.desktop
> @@ -1,7 +1,7 @@
>  [Desktop Entry]
>  Name=SELinux Policy Management Tool
>  Comment=Generate SELinux policy modules
> -Icon=system-config-selinux
> +Icon=sepolicy
>  Exec=/usr/bin/sepolicy gui
>  Type=Application
>  Terminal=false
>
Laurent Bigonville Nov. 3, 2016, 4:16 p.m. UTC | #2
Le 02/11/16 à 14:27, Stephen Smalley a écrit :
> On 11/02/2016 09:04 AM, Laurent Bigonville wrote:
>> From: Laurent Bigonville<bigon@bigon.be>
>>
>> New hires icons have been added to the repository end 2013 but the
>> sepolicy .desktop file is not using it, let's fix that.
> Thanks, applied.  I plan to move these icon files over to the gui/
> directory in the splitpolicycoreutils branch.
Thanks,

I should probably reply to your initial mail about the split, but 
shouldn't the gui directory completely dropped? I mean 
system-config-selinux is python2 only and need to be ported to GObject 
introspection and GTKBuilder if we want it to work in python3 (or it 
should be ported, but it looked non trivial to my meager knowledge).
diff mbox

Patch

diff --git a/policycoreutils/sepolicy/sepolicy.desktop b/policycoreutils/sepolicy/sepolicy.desktop
index f9d0cec..181a1c5 100644
--- a/policycoreutils/sepolicy/sepolicy.desktop
+++ b/policycoreutils/sepolicy/sepolicy.desktop
@@ -1,7 +1,7 @@ 
 [Desktop Entry]
 Name=SELinux Policy Management Tool
 Comment=Generate SELinux policy modules
-Icon=system-config-selinux
+Icon=sepolicy
 Exec=/usr/bin/sepolicy gui
 Type=Application
 Terminal=false