Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
Summary: Review Request: <main package name here> - <short summary here>
https://bugzilla.redhat.com/show_bug.cgi?id=624705
Summary: Review Request: <main package name here> - <short summary here> Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nobody@fedoraproject.org ReportedBy: mail@marcus-moeller.ch QAContact: extras-qa@fedoraproject.org CC: notting@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora
Spec URL: http://www.marcus-moeller.de/share/build/pam_script/1.1.4-3/pam_script.spec SRPM URL: http://www.marcus-moeller.de/share/build/pam_script/1.1.4-3/pam_script-1.1.4... Description: pam_script is a module which allows to execute scripts e.g. after opening and/or closing a session using PAM
rpmlint SRPMS/pam_script-1.1.4-3.fc12.src.rpm RPMS/i686/pam_script-1.1.4-3.fc12.i686.rpm RPMS/i686/pam_script-debuginfo-1.1.4-3.fc12.i686.rpm SPECS/pam_script.spec 3 packages and 1 specfiles checked; 0 errors, 0 warnings.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=624705
Marcus Moeller mail@marcus-moeller.ch changed:
What |Removed |Added ---------------------------------------------------------------------------- Summary|Review Request: <main |Review Request: pam_script |package name here> - <short |- execute scripts from |summary here> |within pam
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=624705
Steve Traylen steve.traylen@cern.ch changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED CC| |steve.traylen@cern.ch AssignedTo|nobody@fedoraproject.org |steve.traylen@cern.ch Flag| |fedora-review?
--- Comment #1 from Steve Traylen steve.traylen@cern.ch 2010-08-25 03:27:26 EDT --- Taking.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=624705
--- Comment #2 from Marcus Moeller mail@marcus-moeller.ch 2010-08-25 03:38:21 EDT --- Just a note. If we want to integrate it in Fedora I guess the SELinux rules have to be included within the default targeted policy. Otherwhise I have to add a:
Requires: selinux-policy-targeted
Greets Marcus
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=624705
--- Comment #3 from Steve Traylen steve.traylen@cern.ch 2010-08-30 14:59:19 EDT --- Hi Marcus, sorry for the slowness from me.
I'm afraid a mock fails to build. f15-x86_64.
+ cp -pr AUTHORS COPYING ChangeLog README NEWS etc/README.module_types etc/README.pam_script /builddir/build/BUILDROOT/pam_script-1.1.4-3.fc15.x86_64/usr/share/doc/pam_script-1.1.4 + exit 0 RPM build errors: File not found by glob: /builddir/build/BUILDROOT/pam_script-1.1.4-3.fc15.x86_64/lib64/security/* Child returncode was: 1 EXCEPTION: Command failed. See logs for output. # ['bash', '--login', '-c', 'rpmbuild -bb --target x86_64 --nodeps builddir/build/SPECS/pam_script.spec'] Traceback (most recent call last):
At a guess lib vs lib64 problem.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=624705
--- Comment #4 from Steve Traylen steve.traylen@cern.ch 2010-11-23 12:18:14 EST --- Ping.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=624705
Steve Traylen steve.traylen@cern.ch changed:
What |Removed |Added ---------------------------------------------------------------------------- Status Whiteboard| |BuildFails Flag|fedora-review? |
--- Comment #5 from Steve Traylen steve.traylen@cern.ch 2011-03-13 04:34:55 EDT --- Build still fails with same error, removing myself from review and updating white board to reflect. Please update whiteboard on fix. Steve.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=624705
Steve Traylen steve.traylen@cern.ch changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |NEW CC|steve.traylen@cern.ch |
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=624705
--- Comment #6 from Marcus Moeller marcus.moeller@gmx.ch 2011-03-13 04:42:03 EDT --- Sorry for not responding earlier. Going to check that next week and report back.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=624705
Marcus Moeller marcus.moeller@gmx.ch changed:
What |Removed |Added ---------------------------------------------------------------------------- Status Whiteboard|BuildFails |
--- Comment #7 from Marcus Moeller marcus.moeller@gmx.ch 2011-05-04 07:45:24 EDT --- Updated Spec:
http://mmoeller.fedorapeople.org/build/pam_script/1.1.4-6/pam_script.spec
and SRPM:
http://mmoeller.fedorapeople.org/build/pam_script/1.1.4-6/pam_script-1.1.4-6...
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=624705
--- Comment #8 from Marcus Moeller marcus.moeller@gmx.ch 2011-05-11 10:34:45 EDT --- fixed post Scripts to install the correct policy package.
http://mmoeller.fedorapeople.org/build/pam_script/1.1.4-7/pam_script.spec
http://mmoeller.fedorapeople.org/build/pam_script/1.1.4-6/pam_script-1.1.4-7...
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=624705
--- Comment #9 from Marcus Moeller marcus.moeller@gmx.ch 2011-05-12 07:03:11 EDT --- corrected module name:
http://mmoeller.fedorapeople.org/build/pam_script/1.1.4-8/pam_script.spec
http://mmoeller.fedorapeople.org/build/pam_script/1.1.4-8/pam_script-1.1.4-8...
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=624705
Anders Blomdell anders.blomdell@control.lth.se changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |anders.blomdell@control.lth | |.se
--- Comment #10 from Anders Blomdell anders.blomdell@control.lth.se 2011-06-01 14:18:09 EDT --- Shouldn't:
BuildRequires: /usr/share/selinux/devel/policyhelp
be replaced (or augmented with, if it is likely to change package) with:
BuildRequires: selinux-policy-doc
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=624705
--- Comment #11 from Marcus Moeller marcus.moeller@gmx.ch 2011-08-04 03:24:08 EDT --- http://www.marcus-moeller.de/share/build/pam_script/1.1.4-9/pam_script.spec SRPM URL: http://www.marcus-moeller.de/share/build/pam_script/1.1.4-9/pam_script-1.1.4...
Updated to require selinux-policy-doc.
Still in need for review ;)
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=624705
Marcus Moeller marcus.moeller@gmx.ch changed:
What |Removed |Added ---------------------------------------------------------------------------- AssignedTo|steve.traylen@cern.ch |nobody@fedoraproject.org
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=624705
--- Comment #12 from Marcus Moeller marcus.moeller@gmx.ch 2011-09-09 09:22:23 EDT --- http://www.marcus-moeller.de/share/build/pam_script/1.1.4-10/pam_script.spec SRPM URL: http://www.marcus-moeller.de/share/build/pam_script/1.1.4-10/pam_script-1.1....
Added selinux_policyver global variable
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=624705
--- Comment #13 from Marcus Moeller marcus.moeller@gmx.ch 2012-04-11 08:47:19 EDT --- Prepared pam_script 1.1.5
http://www.marcusmoeller.ch/share/build/pam_script/1.1.5-1/pam_script.spec
SRPM URL: http://www.marcusmoeller.ch/share/build/pam_script/1.1.5-1/pam_script-1.1.5-...
Meant to be built on Fedora 17+
package-review@lists.fedoraproject.org