Bug 811601

Looks like you are upstream for this.

Don't you have a url/upstream repository for this?

These are fedora specific for the moment,
so it was thought more appropriate just to
store the few script in the fedora repo

General comment: I think you should add a comment about how Fedora
itself is the upstream for this package.  Also I think man pages would
improve this package.

- rpmlint output

Some notable issues raised by rpmlint:

openstack-utils.src: W: strange-permission openstack-config 0775L
openstack-utils.src: W: strange-permission openstack-status 0775L
openstack-utils.src: W: strange-permission openstack-demo-install 0775L
openstack-utils.src: W: strange-permission openstack-db 0775L
openstack-utils.noarch: W: wrong-file-end-of-line-encoding /usr/share/doc/openstack-utils-2012.1/LICENSE
openstack-utils.noarch: W: no-manual-page-for-binary openstack-demo-install
openstack-utils.noarch: W: no-manual-page-for-binary openstack-config
openstack-utils.noarch: W: no-manual-page-for-binary openstack-status
openstack-utils.noarch: W: no-manual-page-for-binary openstack-db

I'm not sure what this one means:

openstack-utils.noarch: W: devel-file-in-non-devel-package /usr/bin/openstack-config

+ package name satisfies the packaging naming guidelines
+ specfile name matches the package base name
+ package should satisfy packaging guidelines
+ license meets guidelines and is acceptable to Fedora
+ license matches the actual package license
+ %doc includes license file
+ spec file written in American English
+ spec file is legible
n/a upstream sources match sources in the srpm
+ package successfully builds on at least one architecture
n/a ExcludeArch bugs filed
n/a BuildRequires list all build dependencies
n/a %find_lang instead of %{_datadir}/locale/*
n/a binary RPM with shared library files must call ldconfig in %post and %postun
+ does not use Prefix: /usr
n/a package owns all directories it creates
+ no duplicate files in %files
+ consistent use of macros
+ package must contain code or permissible content
n/a large documentation files should go in -doc subpackage
+ files marked %doc should not affect package
n/a header files should be in -devel
n/a static libraries should be in -static
n/a packages containing pkgconfig (.pc) files need 'Requires: pkgconfig'
n/a libfoo.so must go in -devel
n/a -devel must require the fully versioned base
n/a packages should not contain libtool .la files
n/a packages containing GUI apps must include %{name}.desktop file
n/a packages must not own files or directories owned by other packages
+ filenames must be valid UTF-8
n/a use %global instead of %define

Optional:

n/a if there is no license file, packager should query upstream
n/a translations of description and summary for non-English languages, if available
n/a reviewer should build the package in mock
n/a the package should build into binary RPMs on all supported architectures
n/a review should test the package functions as described
n/a scriptlets should be sane
n/a pkgconfig files should go in -devel
n/a shouldn't have file dependencies outside /etc /bin /sbin /usr/bin or /usr/sbin

Thanks a lot for the review.

I've updated the package at the above links.

I've made it a bit more standard by storing the upstream
as a subproject of the fedora-openstack support repos at
https://github.com/fedora-openstack/openstack-utils

The devel-file-in-non-devel-package warning looks bogus.

(In reply to comment #3)
> General comment: I think you should add a comment about how Fedora
> itself is the upstream for this package.  Also I think man pages would
> improve this package.

Both of these issues are fixed.

> - rpmlint output
> 
> Some notable issues raised by rpmlint:
> 
> openstack-utils.src: W: strange-permission openstack-config 0775L
> openstack-utils.src: W: strange-permission openstack-status 0775L
> openstack-utils.src: W: strange-permission openstack-demo-install 0775L
> openstack-utils.src: W: strange-permission openstack-db 0775L

Fixed by using `install -m' in spec file.

> openstack-utils.noarch: W: wrong-file-end-of-line-encoding
> /usr/share/doc/openstack-utils-2012.1/LICENSE

Fixed upstream.

> openstack-utils.noarch: W: no-manual-page-for-binary openstack-demo-install
> openstack-utils.noarch: W: no-manual-page-for-binary openstack-config
> openstack-utils.noarch: W: no-manual-page-for-binary openstack-status
> openstack-utils.noarch: W: no-manual-page-for-binary openstack-db

Fixed by adding man pages for all utilities.

> I'm not sure what this one means:
> 
> openstack-utils.noarch: W: devel-file-in-non-devel-package
> /usr/bin/openstack-config

We agree this is not important.

=========================
This package is APPROVED by rjones
=========================

New Package SCM Request
=======================
Package Name: openstack-utils
Short Description: Helper utilities for OpenStack services
Owners: pbrady
Branches: f16 f17 el6
InitialCC:

Git done (by process-git-requests).

cheers!