[Bug 222594] Review Request: seedit: SELinux Policy Editor
bugzilla at redhat.com
bugzilla at redhat.com
Thu Jan 25 18:17:32 UTC 2007
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.
Summary: Review Request: seedit: SELinux Policy Editor
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=222594
------- Additional Comments From mtasaka at ioa.s.u-tokyo.ac.jp 2007-01-25 13:17 EST -------
Well, for packaing issue:
* Requires:
- gnome-python2 (required by -gui)
Please check if this is really required. By checking with
"grep import" no module seems to be installed from
gnome-python2.
* File ownership issue/scriptlet
- - policy package:
-------------------------------------------------------
if [ $1 = 2 ]; then
#Mark to initialize RBAC config when upgrade
touch /usr/share/seedit/sepolicy/need-rbac-init
fi
-------------------------------------------------------
However:
-------------------------------------------------------
[root at localhost ~]# touch /usr/share/seedit/sepolicy/need-rbac-init
[root at localhost ~]# LANG=C rpm -qf /usr/share/seedit/sepolicy/need-rbac-init
file /usr/share/seedit/sepolicy/need-rbac-init is not owned by any package
-------------------------------------------------------
This file (and perhaps also /usr/share/seedit/sepolicy/need-init)
should be marked as %ghost file
And please check if any other file which should be marked
as such exists so that all _unnessary_ files are correctly
removed on the complete removal of seedit.
* Version dependency requirement
-------------------------------------------------------
Requires: seedit >= 2.1.0
-------------------------------------------------------
- Usually these types of requirement should be version-release
dependent, i.e.
-------------------------------------------------------
Requires: %{name} = %{version}-%{release}
-------------------------------------------------------
* Desktop file:
-------------------------------------------------------
Categories=Application;SystemSetup;X-Red-Hat-Base;
-------------------------------------------------------
Both categories: "Application" "X-Red-Hat-Base" "SystemSetup"
are now deprecated and these should be removed.
From desktop-file-validate:
-------------------------------------------------------
Categories values must be one of
"AudioVideo", "Audio", "Video", "Development", "Education",
"Game", "Graphics", "Network", "Office", "Settings",
"System", "Utility", "Building", "Debugger", "IDE",
"GUIDesigner", "Profiling", "RevisionControl", "Translation",
"Calendar", "ContactManagement", "Database", "Dictionary",
"Chart", "Email", "Finance", "FlowChart", "PDA",
"ProjectManagement", "Presentation", "Spreadsheet", "WordProcessor",
"2DGraphics", "VectorGraphics", "RasterGraphics", "3DGraphics",
"Scanning", "OCR", "Photography", "Viewer", "DesktopSettings",
"HardwareSettings", "PackageManager", "Dialup", "InstantMessaging",
"IRCClient", "FileTransfer", "HamRadio", "News", "P2P",
"RemoteAccess", "Telephony", "WebBrowser", "WebDevelopment",
"Midi", "Mixer", "Sequencer", "Tuner", "TV", "AudioVideoEditing",
"Player", "Recorder", "DiscBurning", "ActionGame", "AdventureGame",
"ArcadeGame", "BoardGame", "BlocksGame", "CardGame", "KidsGame",
"LogicGame", "RolePlaying", "Simulation", "SportsGame",
"StrategyGame", "Art", "Construction", "Music", "Languages",
"Science", "Astronomy", "Biology", "Chemistry", "Geology", "Math",
"MedicalSoftware", "Physics", "Amusement", "Archiving",
"Electronics", "Emulator", "Engineering", "FileManager",
"TerminalEmulator", "Filesystem", "Monitor", "Security",
"Accessibility", "Calculator", "Clock", "TextEditor", "Core",
"KDE", "GNOME", "GTK", "Qt", "Motif", "Java", "ConsoleOnly",
"Screensaver", "TrayIcon", "Applet", "Shell"
-------------------------------------------------------
* Timestamps
- These packages include many text files, image files
and keeping timestamps on these files are generally
preferred. Please fix so that the timestamps on these
files are kept.
-------------------------------------------------------
install -m 0644 %{SOURCE2} ${RPM_BUILD_ROOT}%{_datadir}/pixmaps/seedit-gui.png
-------------------------------------------------------
Also, please use "install -p".
* Macros
-------------------------------------------------------
%define selinuxconf /etc/selinux/config
%define auditrules /etc/audit/audit.rules
-------------------------------------------------------
Please check if the directory /etc should be written
as hardcoded or as %{_sysconfdir}.
* Pam requirement
%{_sysconfdir}/pam.d/seedit-gui includes:
-------------------------------------------------------
auth include config-util
-------------------------------------------------------
This sentence requires pam >= 0.80 so I think
adding "Requires: pam >= 0.80" is preferable.
* $RPM_BUILD_ROOT vs %{buildroot}
Please use one, not both.
--
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.
More information about the Fedora-package-review
mailing list