Bug 389701

Fails to build for me.  In %install:

make[2]: Nothing to be done for `install-exec-am'.
/bin/sh: /usr/bin/msgfmt: No such file or directory
cs.mo updated.
/bin/sh: /usr/bin/msgfmt: No such file or directory
de.mo updated.
/bin/sh: /usr/bin/msgfmt: No such file or directory
fr.mo updated.
/bin/sh: /usr/bin/msgfmt: No such file or directory
pl.mo updated.
/bin/sh: /usr/bin/msgfmt: No such file or directory
pt.mo updated.
for i in cs.mo de.mo fr.mo pl.mo pt.mo; do \
                lang=`echo $i | sed 's/\.mo$//'`; \
                /bin/sh /builddir/build/BUILD/osmo-0.1.2/install-sh -d
/var/tmp/osmo-0.1.2-1.fc9-root-mockbuild/usr/share/locale/$lang/LC_MESSAGES; \
                /usr/bin/install -c -m 644 $i
/var/tmp/osmo-0.1.2-1.fc9-root-mockbuild/usr/share/locale/$lang/LC_MESSAGES/osmo.mo;
\
        done
/usr/bin/install: cannot stat `cs.mo': No such file or directory
/usr/bin/install: cannot stat `de.mo': No such file or directory
/usr/bin/install: cannot stat `fr.mo': No such file or directory
/usr/bin/install: cannot stat `pl.mo': No such file or directory
/usr/bin/install: cannot stat `pt.mo': No such file or directory
make[2]: *** [install-data-local] Error 1

Did you do a scratch build before submitting?

There seems to be a typo in the included osmo.desktop file (source1):
Icon=/usr/share/pixmaps//usr/share/pixmaps//usr/share/pixmaps//usr/share/pixmaps
/osmo.svg

Mock build fails with:
2007-11-18 22:24:36,281 - util.py:218:DEBUG:   /usr/bin/install -c 'osmo'
'/var/tmp/osmo-0.1.2-1.fc9-root-mockbuild/usr/b
in/osmo'
2007-11-18 22:24:36,281 - util.py:218:DEBUG: make[2]: Nothing to be done for
`install-data-am'.
2007-11-18 22:24:36,282 - util.py:218:DEBUG: make[2]: Leaving directory
`/builddir/build/BUILD/osmo-0.1.2/src'
2007-11-18 22:24:36,282 - util.py:218:DEBUG: make[1]: Leaving directory
`/builddir/build/BUILD/osmo-0.1.2/src'
2007-11-18 22:24:36,282 - util.py:218:DEBUG: Making install in po
2007-11-18 22:24:36,282 - util.py:218:DEBUG: make[1]: Entering directory
`/builddir/build/BUILD/osmo-0.1.2/po'
2007-11-18 22:24:36,282 - util.py:218:DEBUG: make[2]: Entering directory
`/builddir/build/BUILD/osmo-0.1.2/po'
2007-11-18 22:24:36,282 - util.py:218:DEBUG: make[2]: Nothing to be done for
`install-exec-am'.
2007-11-18 22:24:36,282 - util.py:218:DEBUG: /bin/sh: /usr/bin/msgfmt: No such
file or directory
2007-11-18 22:24:36,282 - util.py:218:DEBUG: cs.mo updated.
2007-11-18 22:24:36,283 - util.py:218:DEBUG: /bin/sh: /usr/bin/msgfmt: No such
file or directory
2007-11-18 22:24:36,283 - util.py:218:DEBUG: de.mo updated.
2007-11-18 22:24:36,283 - util.py:218:DEBUG: /bin/sh: /usr/bin/msgfmt: No such
file or directory
2007-11-18 22:24:36,283 - util.py:218:DEBUG: fr.mo updated.
2007-11-18 22:24:36,283 - util.py:218:DEBUG: /bin/sh: /usr/bin/msgfmt: No such
file or directory
2007-11-18 22:24:36,283 - util.py:218:DEBUG: pl.mo updated.
2007-11-18 22:24:36,283 - util.py:218:DEBUG: /bin/sh: /usr/bin/msgfmt: No such
file or directory
2007-11-18 22:24:36,283 - util.py:218:DEBUG: pt.mo updated.
2007-11-18 22:24:36,284 - util.py:218:DEBUG: for i in cs.mo de.mo fr.mo pl.mo
pt.mo; do \
2007-11-18 22:24:36,284 - util.py:218:DEBUG:            lang=`echo $i | sed
's/\.mo$//'`; \
2007-11-18 22:24:36,284 - util.py:218:DEBUG:            /bin/sh
/builddir/build/BUILD/osmo-0.1.2/install-sh -d /var/tmp/o
smo-0.1.2-1.fc9-root-mockbuild/usr/share/locale/$lang/LC_MESSAGES; \
2007-11-18 22:24:36,284 - util.py:218:DEBUG:            /usr/bin/install -c -m
644 $i /var/tmp/osmo-0.1.2-1.fc9-root-mock
build/usr/share/locale/$lang/LC_MESSAGES/osmo.mo; \
2007-11-18 22:24:36,284 - util.py:218:DEBUG:    done
2007-11-18 22:24:36,284 - util.py:218:DEBUG: /usr/bin/install: cannot stat
`cs.mo': No such file or directory
2007-11-18 22:24:36,284 - util.py:218:DEBUG: /usr/bin/install: cannot stat
`de.mo': No such file or directory
2007-11-18 22:24:36,285 - util.py:218:DEBUG: /usr/bin/install: cannot stat
`fr.mo': No such file or directory
2007-11-18 22:24:36,285 - util.py:218:DEBUG: /usr/bin/install: cannot stat
`pl.mo': No such file or directory
2007-11-18 22:24:36,285 - util.py:218:DEBUG: /usr/bin/install: cannot stat
`pt.mo': No such file or directory
2007-11-18 22:24:36,285 - util.py:218:DEBUG: make[2]: *** [install-data-local]
Error 1
2007-11-18 22:24:36,285 - util.py:218:DEBUG: make[2]: Leaving directory
`/builddir/build/BUILD/osmo-0.1.2/po'
2007-11-18 22:24:36,285 - util.py:218:DEBUG: make[1]: *** [install-am] Error 2
2007-11-18 22:24:36,285 - util.py:218:DEBUG: make[1]: Leaving directory
`/builddir/build/BUILD/osmo-0.1.2/po'
2007-11-18 22:24:36,285 - util.py:218:DEBUG: make: *** [install-recursive] Error 1
2007-11-18 22:24:36,286 - util.py:218:DEBUG: error: Bad exit status from
/var/tmp/rpm-tmp.79694 (%install)
2007-11-18 22:24:36,286 - util.py:218:DEBUG:
2007-11-18 22:24:36,286 - util.py:218:DEBUG:
2007-11-18 22:24:36,286 - util.py:218:DEBUG: RPM build errors:
2007-11-18 22:24:36,286 - util.py:218:DEBUG:     Bad exit status from
/var/tmp/rpm-tmp.79694 (%install)
2007-11-18 22:24:36,287 - trace_decorator.py:27:DEBUG: EXCEPTION:
Command(rpmbuild -bb --target x86_64 --nodeps //builddi
r/build/SPECS/osmo.spec) failed. See logs for output.

i've forgot about translations.

Spec URL: http://liviopl.fedorapeople.org/osmo.spec
SRPM URL: http://liviopl.fedorapeople.org/osmo-0.1.2-2.fc8.src.rpm

i've forgot about translations.

Spec URL: http://liviopl.fedorapeople.org/osmo.spec
SRPM URL: http://liviopl.fedorapeople.org/osmo-0.1.2-2.fc8.src.rpm

God... Did I sent my comment twice? I don't remember...
Sorry, nevermind.

This time the package builds OK, but %{_datadir}/locale/ should not be included
under %files, because 
- locales are already included via -f %{name}.lang, so this triggers "files
listed twice" warnings  (for instance:  File listed twice:
usr/share/locale/cs/LC_MESSAGES/osmo.mo)
- it takes ownership of %{_datadir}/locale/ and its descendant directories

It would also be nice if you would preserve the timestamp of the files which are
not modified (mainly those listed under %doc). I did not test with osmo, but
usually passing INSTALL="install -p" to %make install is enough for that.

i had bad day, fixing.

Spec URL: http://liviopl.fedorapeople.org/osmo.spec
SRPM URL: http://liviopl.fedorapeople.org/osmo-0.1.2-3.fc8.src.rpm

According to
http://fedoraproject.org/wiki/Packaging/Guidelines#head-254ddf07aae20a23ced8cecc219d8f73926e9755

you have to add desktop-file-utils as a BR and use it to install osmo.desktop.
Something along
           desktop-file-install --vendor="fedora" \
--dir=${RPM_BUILD_ROOT}%{_datadir}/applications %{SOURCE1}
should do. Mind the usage of "fedora" as vendor.

There are also a couple more things to tweak:
- The package owns /usr/share/pixmaps, which is a standard directory. Please
replace 
%{_datadir}/pixmaps/
with
%{_datadir}/pixmaps/*

Another thing whih is a MUSTFIX is obeying to compiler flags (see
http://fedoraproject.org/wiki/Packaging/Guidelines#head-8b14098227aebff1cf6188939e9d0877295ac448).

I hope you have no more issues to tell me :] .

Spec URL: http://liviopl.fedorapeople.org/osmo.spec
SRPM URL: http://liviopl.fedorapeople.org/osmo-0.1.2-4.fc8.src.rpm

Created attachment 266191 [details]
failed build log with first attempt to use $RPM_OPT_FLAGS

Unfortunately 
- you have not used desktop-file-utils to actually install the desktop file
- your attempt to pass CFLAGS to make leads to failing builds

Please see the log in comment #10.

Jesus, forgot to add few lines and few chars...
I hope now everything is ok.

Spec URL: http://liviopl.fedorapeople.org/osmo.spec
SRPM URL: http://liviopl.fedorapeople.org/osmo-0.1.2-5.fc8.src.rpm

Build in mock fails because you have to replace the last line in %files with 
%{_datadir}/applications/fedora-%{name}.desktop. Otherwise everything looks OK.

I'll post a full review later today.

I have little sclerosis on today...

Spec URL: http://liviopl.fedorapeople.org/osmo.spec
SRPM URL: http://liviopl.fedorapeople.org/osmo-0.1.2-6.fc8.src.rpm

Package Review for osmo-0.1.2-6.fc8.src.rpm
==============

Key:
 - = N/A
 x = Check
 ! = Problem
 ? = Not evaluated

=== REQUIRED ITEMS ===
 [x] Package is named according to the Package Naming Guidelines.
 [x] Spec file name must match the base package %{name}, in the format %{name}.spec.
 [x] Package meets the Packaging Guidelines.
 [x] Package successfully compiles and builds into binary rpms on at least one
supported architecture.
     Tested on:rawhide/x86_64
 [x] Rpmlint output: none
 [x] Package is not relocatable.
 [x] Buildroot is correct
(%{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n))
 [x] Package is licensed with an open-source compatible license and meets other
legal requirements as defined in the legal section of Packaging Guidelines.
 [x] License field in the package spec file matches the actual license.
     License type:GPLv2+
 [x] If (and only if) the source package includes the text of the license(s) in
its own file, then that file, containing the text of the license(s) for the
package is included in %doc.
 [x] Spec file is legible and written in American English.
 [x] Sources used to build the package matches the upstream source, as provided
in the spec URL.
     SHA1SUM of package: 766c5020b08cee4f43c45fb23d5623fbea9b2b11
 [x] Package is not known to require ExcludeArch
 [x] All build dependencies are listed in BuildRequires, except for any that are
listed in the exceptions section of Packaging Guidelines.
 [x] The spec file handles locales properly.
 [-] ldconfig called in %post and %postun if required.
 [x] Package must own all directories that it creates.
 [x] Package requires other packages for directories it uses.
 [x] Package does not contain duplicates in %files.
 [x] Permissions on files are set properly.
 [x] Package has a %clean section, which contains rm -rf %{buildroot} (or
$RPM_BUILD_ROOT).
 [x] Package consistently uses macros.
 [x] Package contains code, or permissable content.
 [-] Large documentation files are in a -doc subpackage, if required.
 [x] Package uses nothing in %doc for runtime.
 [-] Header files in -devel subpackage, if present.
 [-] Static libraries in -devel subpackage, if present.
 [-] Package requires pkgconfig, if .pc files are present.
 [-] Development .so files in -devel subpackage, if present.
 [-] Fully versioned dependency in subpackages, if present.
 [x] Package does not contain any libtool archives (.la).
 [x] Package contains a properly installed %{name}.desktop file if it is a GUI
application.
 [x] Package does not own files or directories owned by other packages.

=== SUGGESTED ITEMS ===
 [x] Latest version is packaged.
 [x] Package does not include license text files separate from upstream.
 [x] Description and summary sections in the package spec file contains
translations for supported Non-English languages, if available.
 [x] Reviewer should test that the package builds in mock.
     Tested on:rawhide and F7
 [x] Package should compile and build into binary rpms on all supported
architectures.
     Tested on:rawhide and F7
 [x] Package functions as described.
 [x] Scriptlets must be sane, if used.
 [-] The placement of pkgconfig(.pc) files are correct.
 [x] File based requires are sane.


=== Issues ===
1. None

=== Final Notes ===
1. It would be awesome if you would preserve the timestamps of the pixmaps files
(png and svg).
2. For esthetics's sake and compliance with
http://fedoraproject.org/wiki/Packaging/Guidelines#head-f3d77b27a5d29dfc1f5600ef3fc836f2e317badf,
please use (in %make) $RPM_OPT_FLAGS instead of %{optflags} (everywhere else you
have used the uppercase version). You could do this change in the current
version, before commit, no need to bump release for that.

================
*** APPROVED ***
================

@Notes:
1. I tried to preserve them, didn't helped out. BTW, is this REALLY needed?
2. Changed.

Spec URL: http://liviopl.fedorapeople.org/osmo.spec
SRPM URL: http://liviopl.fedorapeople.org/osmo-0.1.2-6.fc8.src.rpm

New Package CVS Request
=======================
Package Name: osmo
Short Description: Personal organizer
Owners: liviopl
Branches: F-7 F-8
InitialCC: liviopl
Cvsextras Commits: yes

Well, for a start your call to sed will modify the timestamp of the file. You
could preserve it with a bit of magic, like in
   touch -r osmo.svg some_temp_file
   <your call to sed goes here>
   touch -r some_temp_file osmo.svg

Sorry, I usually don't use sed, awk, whatever.
Could you help me out with this?

cvs done.

It looks I was wrong, the call to sed from %prep affects the desktop file, not
the pixmaps.

The only needed change is to use
   make install DESTDIR=$RPM_BUILD_ROOT INSTALL='install -p'
instead of
   INSTALL='install -p' make install DESTDIR=$RPM_BUILD_ROOT

Please do that modification before uploading to cvs.
And do not forget to close the bug once you are done building in koji and the
packages are pushed via bodhi.

Too bad for me, that I have built packages and submitted updates to testing.
I'm not uploading updated spec and srpm, insted I'm commiting these files to CVS.
Now make build in every branch and then sumbit to bodhi.