Bug 225945

Jeff has disclaimed ownership of jfsutils.  See bug 226558 for the specific comment.

I plan on taking this package over via the orphan package process.

I'll take on the review.

rpmlint output:
jfsutils.src:28: W: configure-without-libdir-spec
jfsutils.src: W: summary-ended-with-dot Utilities for managing the JFS filesystem.
jfsutils.x86_64: W: summary-ended-with-dot Utilities for managing the JFS filesystem.
jfsutils-debuginfo.x86_64: W: spurious-executable-perm /usr/src/debug/jfsutils-1.1.13/include/jfs_dtree.h
jfsutils-debuginfo.x86_64: W: spurious-executable-perm /usr/src/debug/jfsutils-1.1.13/include/jfs_dinode.h
jfsutils-debuginfo.x86_64: W: spurious-executable-perm /usr/src/debug/jfsutils-1.1.13/libfs/log_work.c
jfsutils-debuginfo.x86_64: W: spurious-executable-perm /usr/src/debug/jfsutils-1.1.13/include/jfs_superblock.h
jfsutils-debuginfo.x86_64: W: spurious-executable-perm /usr/src/debug/jfsutils-1.1.13/mkfs/inodes.h
jfsutils-debuginfo.x86_64: W: spurious-executable-perm /usr/src/debug/jfsutils-1.1.13/include/jfs_dmap.h
jfsutils-debuginfo.x86_64: W: spurious-executable-perm /usr/src/debug/jfsutils-1.1.13/libfs/message.c
jfsutils-debuginfo.x86_64: W: spurious-executable-perm /usr/src/debug/jfsutils-1.1.13/include/jfs_unicode.h
jfsutils-debuginfo.x86_64: W: spurious-executable-perm /usr/src/debug/jfsutils-1.1.13/include/jfs_xtree.h
jfsutils-debuginfo.x86_64: W: spurious-executable-perm /usr/src/debug/jfsutils-1.1.13/include/jfs_types.h
jfsutils-debuginfo.x86_64: W: spurious-executable-perm /usr/src/debug/jfsutils-1.1.13/include/jfs_btree.h
jfsutils-debuginfo.x86_64: W: spurious-executable-perm /usr/src/debug/jfsutils-1.1.13/libfs/devices.c
jfsutils-debuginfo.x86_64: W: spurious-executable-perm /usr/src/debug/jfsutils-1.1.13/mkfs/mkfs.c
jfsutils-debuginfo.x86_64: W: spurious-executable-perm /usr/src/debug/jfsutils-1.1.13/include/jfs_imap.h
3 packages and 0 specfiles checked; 0 errors, 17 warnings.


- Change buildroot to 
%(mktemp -ud %{_tmppath}/%{name}-%{version}-%{release}-XXXXXX)

- Use %configure --sbindir=/sbin, and enable smp make.

- Remove
# let brp-compress handle this policy
rm -f $RPM_BUILD_ROOT/%{_mandir}/*/*.gz

- Remove buildroot check from clean phase.

I've done most of this locally now.  Still need to look at the weird debuginfo stuff.

While you're at it, add %{?dist} to the release tag.

(In reply to comment #5)
> I've done most of this locally now.  Still need to look at the weird debuginfo
> stuff.  

The debuginfo stuff should be trivial to fix, just remove the executable bits from the source code files and headers in the setup phase.

All fixed in devel

.. Not so fast! I didn't do the review yet, or approve the package!

- Please use %configure instead of ./configure. If you
%global _sbindir /sbin
you can use plain %configure and %{_sbindir}/* in the files section.

The review as follows:

- source files match upstream:
 * Tarball matches upstream, but
MUST: Source URL is missing!
X package meets naming and versioning guidelines.
X specfile is properly named, is cleanly written and uses macros consistently.
X dist tag is present.
X build root is correct.
X license field matches the actual license.
X license is open source-compatible.
- license text included in package.
MUST: Missing from documentation: AUTHORS ChangeLog COPYING NEWS
X latest version is being packaged.
X BuildRequires are proper.
X compiler flags are appropriate.
X %clean is present.
X package builds in mock.
X package installs properly.
X debuginfo package looks complete.
X rpmlint is silent.
X final provides and requires are sane.
X no shared libraries are added to the regular linker search paths.
X owns the directories it creates.
X doesn't own any directories it shouldn't.
X no duplicates in %files.
X file permissions are appropriate.
X no scriptlets present.
X code, not content.
X documentation is small, so no -docs subpackage is necessary.
X %docs are not necessary for the proper functioning of the package.
X no headers.
X no pkgconfig files.
X no libtool .la droppings.
X no desktop files

Fix the above and I'll approve the package.

(In reply to comment #9)
> .. Not so fast! I didn't do the review yet, or approve the package!
> 
> - Please use %configure instead of ./configure. If you
> %global _sbindir /sbin

Why is that needed?  It should be done in the buildsys already...

(In reply to comment #10)
> (In reply to comment #9)
> > .. Not so fast! I didn't do the review yet, or approve the package!
> > 
> > - Please use %configure instead of ./configure. If you
> > %global _sbindir /sbin
> 
> Why is that needed?  It should be done in the buildsys already...  

Because by default _sbindir is /usr/sbin. This redefinition makes the spec file cleaner, and is used in other system tool specs too.

(In reply to comment #11)
> (In reply to comment #10)
> > (In reply to comment #9)
> > > .. Not so fast! I didn't do the review yet, or approve the package!
> > > 
> > > - Please use %configure instead of ./configure. If you
> > > %global _sbindir /sbin
> > 
> > Why is that needed?  It should be done in the buildsys already...  
> 
> Because by default _sbindir is /usr/sbin. This redefinition makes the spec file
> cleaner, and is used in other system tool specs too.  

Yeah, noticed that.  I'm not thrilled with overriding the macro though, so I'll just specify /sbin.

(In reply to comment #9)
> - license text included in package.
> MUST: Missing from documentation: AUTHORS ChangeLog COPYING NEWS

Sorry, disregard this. However, you can drop INSTALL and README from the
package as they are related to the compilation.

(In reply to comment #12)
> Yeah, noticed that.  I'm not thrilled with overriding the macro though, so I'll
> just specify /sbin.  

OK, you can do that if you use %configure --sbindir=/sbin as mentioned above.

Also, please change %{_mandir}/*/* to %{_mandir}/man8/*, since if translations appear then the package will end up owning system directories.

Changes are in cvs now (except the INSTALL/README thing because I forgot it and it's not really that important).

I'll wait before doing another build until I get the magic approval

(In reply to comment #14)
> Changes are in cvs now (except the INSTALL/README thing because I forgot it and
> it's not really that important).
> 
> I'll wait before doing another build until I get the magic approval  

Drop the unnecessary stuff from the configure phase, it should read just %configure --sbindir=/sbin
since all of the other stuff is already in the %configure script.

Or, if you redefine sbindir as I mentioned before the configure phase is just
%configure
as the redefined sbindir is automatically expanded.

IMHO it's nicer to just redefine sbindir and use %{_sbindir} everywhere instead of /sbin.

Changes in CVS

Well, you still don't need to
CFLAGS="${RPM_OPT_FLAGS}"

since it is already in %configure. Remove the CFLAGS stuff (and INSTALL and README from %doc since they only contain information about building the package, which is not necessary in the binary rpm).

Anyway, all of the blockers have been fixed, so the package is

APPROVED.


Josh: please remove the unnecessary stuff before you rebuild in rawhide.