Bug 428642 - Review Request: remctl - GSSAPI secured remote command execution
Summary: Review Request: remctl - GSSAPI secured remote command execution
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review   
(Show other bugs)
Version: rawhide
Hardware: All
OS: Linux
Target Milestone: ---
Assignee: Mamoru TASAKA
QA Contact: Fedora Extras Quality Assurance
Depends On:
TreeView+ depends on / blocked
Reported: 2008-01-14 10:58 UTC by Simon Wilkinson
Modified: 2008-01-27 20:45 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Last Closed: 2008-01-27 20:45:27 UTC
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---
mtasaka: fedora-review+
kevin: fedora-cvs+

Attachments (Terms of Use)

Description Simon Wilkinson 2008-01-14 10:58:32 UTC
Spec URL: http://homepages.inf.ed.ac.uk/sxw/remctl.spec
SRPM URL: http://homepages.inf.ed.ac.uk/sxw/remctl-2.11-1.src.rpm

remctl (the client) and remctld (the server) implement a client/server
protocol for running single commands on a remote host using Kerberos
v5 authentication and returning the output. They use a very simple
GSS-API-authenticated network protocol, combined with server-side ACL
support and a server configuration file that maps remctl commands to
programs that should be run when that command is called by an
authorised user.

This is my first package, and I'm in need of a sponsor. Thanks!

Comment 1 Mamoru TASAKA 2008-01-18 18:05:03 UTC
Well, I have only did a quick glance at your spec file
as rebuild failed on dist-f9.
For general packaging guidelines, please refer to

? Disttag
  - Please consider to use %{?dist} tag.

* Missing BR
  - Rebuild failed on dist-f9.
    From build.log, at least 'perl(ExtUtils::MakeMaker)' is
    missing from BuildRequires.

  ! Note
    'perl(ExtUtils::MakeMaker)' is provides by perl-ExtUtils-MakeMaker
    rpm, however for perl modules dependency please don't write the
    rpm names but write the module names needed.

* Setup
  - Please make %setup quiet.

* Man pages section
  - Usually man pages with section 3 are for the explanation of
    libraries/functions and they should be in -devel subpackage.

Comment 2 Simon Wilkinson 2008-01-22 19:38:55 UTC
>For general packaging guidelines, please refer to

I'd read both of these before submitting these spec files.


The spec file I supplied already uses the distag as detailed in the document you reference - am I 
missing something?

> Missing BR

Fixed. I've verified that the new specfile builds on f9 using mock.

> Setup

I've made setup quiet

> Man pages section

I've moved the section 3 manpages to the -devel subpackage.

Updated SRPM, and spec are at

Spec URL: http://homepages.inf.ed.ac.uk/sxw/remctl.spec
SRPM URL: http://homepages.inf.ed.ac.uk/sxw/remctl-2.11-2.src.rpm

Thanks for taking a look!

Comment 3 Mamoru TASAKA 2008-01-23 14:20:21 UTC
For 2.11-2:

* rpmlint issues:
[tasaka1@localhost remctl]$ rpmlint remctl remctl-devel remctl-perl
remctl.i386: E: library-without-ldconfig-postin /usr/lib/libremctl.so.1.0.1
remctl.i386: E: library-without-ldconfig-postun /usr/lib/libremctl.so.1.0.1
remctl.i386: W: unused-direct-shlib-dependency /usr/lib/libremctl.so.1.0.1
remctl.i386: W: unused-direct-shlib-dependency /usr/lib/libremctl.so.1.0.1
remctl.i386: W: unused-direct-shlib-dependency /usr/lib/libremctl.so.1.0.1
remctl-perl.i386: E: non-standard-executable-perm
  - /sbin/ldconfig mustt be run on %post, %postun in remtcl package.
  - The permission of Remctl.so must be 0755, not 0555
    ! Please fix the permission by using "chmod" in %install, not
      by using %attr(...).

* manfiles
  - Files under %_mandir are automatically marked as %doc.

* Timestamps
  - To keep timestamps, I recommend to use
make install DESTDIR=$RPM_BUILD_ROOT INSTALL="install -p"
    This usually works for recent autotool-based Makefiles.

* Undesirably owned directories
%files perl
  - This is not allowed because the directory %perl_vendorarch/auto is 
    already owned by perl.

Comment 4 Simon Wilkinson 2008-01-23 18:15:30 UTC
> rpmlint
Now resolved. Sorry that I'd missed these. I've added ldconfig to %post and %postun for both remctl and 
remctl-devel. rpmlint is now clean

> manfiles
I've removed the %doc directives

> Timestamps
I've fixed the timestamps

> Undesireably owned directories

remctl-perl now owns:

Spec URL: http://homepages.inf.ed.ac.uk/sxw/remctl.spec
SRPM URL: http://homepages.inf.ed.ac.uk/sxw/remctl-2.11-3.src.rpm

Thanks again!


Comment 5 Mamoru TASAKA 2008-01-24 14:39:54 UTC
For 2.11-3:

* ldconfig for -devel
  - For remctl-devel subpackge, calling /sbin/ldconfig is not

* Again directory ownership issue
  - Well, actually I am not sure what package should own the followng
    On my system these directories are owned by other packages,
    however they are not needed for remctl-perl.
    IMO remctl-perl should own these directories, too (for now).

* Permission
  - Well, I didn't notice this, however modify the permissions of the
    following files to 0644.

NOTE: Before being sponsored:

This package will be accepted with another few work. 
But before I accept this package, someone (I am a candidate) 
must sponsor you.

Once you are sponsored, you have the right to review other 
submitters' review requests and approve the packages formally. 
For this reason, the person who want to be sponsored (like you) 
are required to "show that you have an understanding 
of the process and of the packaging guidelines" as is described
on :

Usually there are two ways to show this.
A. submit other review requests with enough quality.
B. Do a "pre-review" of other person's review request
   (at the time you are not sponsored, you cannot do
   a formal review)

When you have submitted a new review request or have pre-reviewed other 
person's review request, please write the bug number on this bug report 
so that I can check your comments or review request.

Fedora package collection review requests which are waiting for someone to
review can be checked on:
(NOTE: please don't choose "Merge Review")

Review guidelines are described mainly on:

Comment 6 Simon Wilkinson 2008-01-27 14:57:43 UTC
A new version of the remctl spec, and SRPM addressing these issues is available at:
Spec URL: http://homepages.inf.ed.ac.uk/sxw/remctl.spec
SRPM URL: http://homepages.inf.ed.ac.uk/sxw/remctl-2.11-4.src.rpm

I've placed a review request for another of our local packages for consideration in bug #430399 - 
hopefully that's got fewer issues than this one had!

I've also done a package pre-review - for bug #430366

Thanks again for your time...

Comment 7 Mamoru TASAKA 2008-01-27 15:06:07 UTC
(In reply to comment #6)
> SRPM URL: http://homepages.inf.ed.ac.uk/sxw/remctl-2.11-4.src.rpm

This seems 404 (not found).

Comment 8 Simon Wilkinson 2008-01-27 15:13:51 UTC
Sorry - now fixed.

Comment 9 Mamoru TASAKA 2008-01-27 15:51:20 UTC

- This package itself is now okay.
- Your another review request and your pre-review seems good from
  my very quick glance.

    This package (remctl) is APPROVED by me

Please follow the procedure written on:
from "Get a Fedora Account".
At a point a mail should be sent to sponsor members which notifies
that you need a sponsor. At the stage, please also write on
this bug for confirmation that you requested for sponsorship and
your FAS (Fedora Account System) name. Then I will sponsor you.

If you want to import this package into Fedora 7/8, you also have
to look at
(after once you rebuilt this package on koji Fedora rebuilding system).

If you have questions, please ask me.

Comment 10 Simon Wilkinson 2008-01-27 17:40:47 UTC
I've requested sponsorship.

My FAS name is 'sxw'

Thanks again!

Comment 11 Mamoru TASAKA 2008-01-27 17:43:24 UTC
Now I should be sponsoring you. Please follow "Join" wiki

Comment 12 Simon Wilkinson 2008-01-27 18:34:07 UTC
New Package CVS Request
Package Name: remctl
Short Description: GSSAPI secured remote command execution
Owners: sxw
Branches: F-7 F-8 EL-4 EL-5
Cvsextras Commits: yes

Comment 13 Kevin Fenzi 2008-01-27 19:26:45 UTC
cvs done.

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