Bug 616120 (spacecmd)

== MUST ==
[x] Run rpmlint on all parts of the package and post output

    spacecmd.noarch: W: non-conffile-in-etc /etc/bash_completion.d/spacecmd
    1 packages and 0 specfiles checked; 0 errors, 1 warnings.

    I agree files in /etc/bash_completion.d should not have %config

[x] Package is named according to Naming Guidelines
[x] Specfile matches %{name}.spec
[x] Package matches Packaging Guidelines
[x] Package license matches Licensing Guidelines
[x] Actual license is in License tag in spec file
[x] License text in %doc if applicable
[x] Spec file is in English
[x] Spec file is legible
[x] MD5 sum of sources match upstream MD5 sum (through Source0)
[x] Must compile on at least *one* major architecture
[-] If package does not compile, it has ExcludeArch for that architecture,
with a bug filed for the architecture the package does not build on
[-] Build-time dependencies are in BuildRequires
[-] Uses %find_lang if appropriate; *not* %{_datadir}/locale/*
[-] Calls ldconfig in %post and %postun if carries libraries
[x] Does not carry copies of system libraries
[x] Doesn't do explicit relocatability
[!] Owns all directories it creates; if it uses a directory it does not create itself, it should depend on the package that *does* create it

    spacecmd should own /etc/bash_completion.d:
    http://fedoraproject.org/wiki/PackagingGuidelines#Multiple_packages_own_files_in_a_common_directory_but_none_of_them_needs_to_require_the_others.

    %{rhnroot} (/usr/share/rhn) is not owned by any packages: https://fedoraproject.org/wiki/Packaging/UnownedDirectories#Common_Mistakes

    spacecmd should either depend on the package that creates %{rhnroot} or own it.        

[x] Has sane permissions, defattr() in every %files section
[!] Consistently uses macros
    
    According to
    http://fedoraproject.org/wiki/Packaging/Guidelines#.25global_preferred_over_.25define it is preferred to use %global instead of %define 

[x] Contains only code or permissable content
[-] Large documentation goes in -doc package
[-] Header files go in -devel package
[-] static libraries go in -static package
[-] Libaries without a numeric suffix (.so.1.1) mostly go in -devel package
[-] Devel package requires base package like this
[-] All .la libtool archives are removed from the package
[-] GUI apps include a %{name}.desktop file
[x] Does not conflict with other packages in file ownership
[-] Filenames are in UTF-8

=== SHOULD ==
[-] If license text is not in separate file, query upstream to do so
[-] If available, the summary and description in the spec can be in other languages than English too
[x] Test build in mock
[x] Builds on all architectures
[x] Runs
[x] Scriptlets in spec file must be sane
[-] Subpackages require the base package
[-] .pc files go in -devel, mostly, unless the package is a devel tool, like gcc
[-] requires packages, not files in packages
[-] Try to have upstream create and add manpages for all binaries

There's a couple of issues with the package at the moment. They're marked with a [!] above.

There is one bigger issue with the RPM as it is now though and that is the fact that it does not contain *.pyc and *.pyo files when built in mock or Koji. The RPM *should* contain those files in order to be able to properly cleaning up when upgrading or uninstalling. See http://fedoraproject.org/wiki/Packaging:Python#Files_to_include

The directory ownerships and %global fixes have been committed.

It seems that rpmbuild is not doing what it should in regards to byte compiling the Python files.  I tried the following to see if the spec was wrong:
- moving the *.py into %{python_sitelib}
- listing %{rhnroot}/spacecmd/*.py in the %files section

The only way I could get rpmbuild to byte-compile the Python files was to explicitly call 'brp-python-bytecompile' in %install, which probably isn't something that should be done.  Any ideas on what's going on?

You must add

BuildRequires: python-devel

to your specfile.

Ah, I missed that part on the wiki. :-(

Fixes have all been committed and are tagged as spacecmd-0.5.2-1.

Aron, I like the fact that you have moved the whole lot into site_packages.

I see you have changed the specfile to require just 'python'. This is not necessary: the Python requirement is inserted by rpmbuild during creation of the RPM in the form of python(abi). This works for both EL5 and F13, so the 'python' requirement can be taken out.

One more request is to add the -p flag to your install commands. I forgot about this initially, sorry. This is to keep the original timestamps on the files you install; see http://fedoraproject.org/wiki/Packaging/Guidelines#Timestamps for futher info. Take care to also keep the orignal timestamp on the source tarball when building srpms.

Please upload the corrected spec and srpm files somewhere and provide a link to them.

Am I right in assuming Miroslav is going to actually maintain this package when it is approved?

I addressed issues from #5.

New files:
http://miroslav.suchy.cz/spacewalk/spacecmd/spacecmd.spec
http://miroslav.suchy.cz/spacewalk/spacecmd/spacecmd-0.5.3-1.el6.src.rpm

> Am I right in assuming Miroslav is going to actually maintain this package when
it is approved?

Yes. We are both from upstream team. But he is more upstream and I'm more maintainer for Fedora.

Looks good. Looking forward to using this.

--------------------------------------------------------
spacecmd was approved by Maxim  Burgerhout d.d. 20100726
--------------------------------------------------------

New Package CVS Request
=======================
Package Name: spacecmd
Short Description: Command-line interface to Spacewalk and Satellite servers
Owners: msuchy
Branches: F-13, F-12, EL-5, EL-6, EL-4
InitialCC:

GIT done (by process-cvs-requests.py).

With f14 branch added.

spacecmd-0.5.4-1.fc13 has been submitted as an update for Fedora 13.
http://admin.fedoraproject.org/updates/spacecmd-0.5.4-1.fc13

spacecmd-0.5.4-1.fc13 has been pushed to the Fedora 13 stable repository.  If problems still persist, please make note of it in this bug report.

Package Change Request
======================
Package Name: spacecmd
New Branches: epel7
Owners: msuchy
InitialCC:

Git done (by process-git-requests).