Bug 910835

I've updated the package with some forgotten runtime deps:

Spec URL: http://psabata.fedorapeople.org/pkgs/snotes/snotes.spec
SRPM URL: http://psabata.fedorapeople.org/pkgs/snotes/snotes-1.0-2.fc18.src.rpm

URL and Source0 are usable. Ok.
Source tar ball is original (SHA-256: 782547cc4db6591d9db015eac49062e62837b2bd4bc4bdaa358a60e268081242). Ok.

FIX: Run-require `git' (snotes:49).

perl is optional. Ok.

TODO: Run-require `xterm'. This the similar to already run-required `vim-minimal' (snotes:30).

$ rpmlint snotes.spec ../SRPMS/snotes-1.0-1.fc19.src.rpm ../RPMS/noarch/snotes-1.0-1.fc19.noarch.rpm 
snotes.noarch: W: no-manual-page-for-binary snotes
snotes.noarch: W: no-manual-page-for-binary snotes-open
2 packages and 1 specfiles checked; 0 errors, 2 warnings.
rpmlint is Ok.

$ rpm -q -lv -p ../RPMS/noarch/snotes-1.0-1.fc19.noarch.rpm
-rwxr-xr-x    1 root    root                     5028 Feb 14 09:20 /usr/bin/snotes
-rwxr-xr-x    1 root    root                     2793 Feb 14 09:20 /usr/bin/snotes-open
drwxr-xr-x    2 root    root                        0 Feb 14 09:20 /usr/share/doc/snotes-1.0
-rw-r--r--    1 root    root                     1089 Feb 11 20:35 /usr/share/doc/snotes-1.0/LICENSE
-rw-r--r--    1 root    root                      773 Feb 11 20:35 /usr/share/doc/snotes-1.0/README
File layout and permissions are Ok.

$ rpm -q --requires -p ../RPMS/noarch/snotes-1.0-1.fc19.noarch.rpm | sort | uniq -c
      1 /bin/sh
      1 dmenu
      1 rpmlib(CompressedFileNames) <= 3.0.4-1
      1 rpmlib(FileDigests) <= 4.6.0-1
      1 rpmlib(PayloadFilesHavePrefix) <= 4.0-1
      1 rpmlib(PayloadIsXz) <= 5.2-1
      1 vim-minimal
FIX: Run-require git
TODO: Run-require xterm.

$ rpm -q --provides -p ../RPMS/noarch/snotes-1.0-1.fc19.noarch.rpm | sort | uniq -c
      1 snotes = 1.0-1.fc19
Binary provides are Ok.

$ resolvedeps rawhide ../RPMS/noarch/snotes-1.0-1.fc19.noarch.rpm 
Binary dependencies resolvable. Ok.

Package builds in F19 (http://koji.fedoraproject.org/koji/taskinfo?taskID=4969472). Ok.

Otherwise package is in line with Fedora packaging guidelines.

Please correct all `FIX' issues, consider fixing `TODO' items, and provide new spec file.
Resolution: NOT approved.

All fixed in r2, see Comment#1.

Spec file changes:
--- snotes.spec.old     2013-02-13 17:30:40.000000000 +0100
+++ snotes.spec 2013-02-14 09:20:41.000000000 +0100
@@ -1,13 +1,16 @@
 Name:           snotes
 Version:        1.0
-Release:        1%{?dist}
+Release:        2%{?dist}
 Summary:        A flexible and easy to use notes system
 License:        MIT
 URL:            https://github.com/v4hn/%{name}
 Source0:        https://github.com/v4hn/%{name}/archive/v%{version}.tar.gz
 BuildArch:      noarch
 Requires:       dmenu
+Requires:       git
+# The default editor
 Requires:       vim-minimal
+Requires:       xterm

 %description
 %{summary}.
@@ -27,5 +30,8 @@
 %{_bindir}/snotes*

 %changelog
+* Thu Feb 14 2013 Petr Šabata <contyk> - 1.0-2
+- Add some forgotted runtime dependecies
+
 * Wed Feb 13 2013 Petr Šabata <contyk> - 1.0-1
 - Initial package

> FIX: Run-require `git' (snotes:49).
+Requires:       git
Ok.

> TODO: Run-require `xterm'. This the similar to already run-required `vim-minimal' (snotes:30).
+Requires:       xterm
Ok.

$ rpmlint snotes.spec ../SRPMS/snotes-1.0-2.fc19.src.rpm ../RPMS/noarch/snotes-1.0-2.fc19.noarch.rpm 
snotes.noarch: W: no-manual-page-for-binary snotes
snotes.noarch: W: no-manual-page-for-binary snotes-open
2 packages and 1 specfiles checked; 0 errors, 2 warnings.
rpmlint is Ok.

$ rpm -q --requires -p ../RPMS/noarch/snotes-1.0-2.fc19.noarch.rpm | sort | uniq -c
      1 /bin/sh
      1 dmenu
      1 git
      1 rpmlib(CompressedFileNames) <= 3.0.4-1
      1 rpmlib(FileDigests) <= 4.6.0-1
      1 rpmlib(PayloadFilesHavePrefix) <= 4.0-1
      1 rpmlib(PayloadIsXz) <= 5.2-1
      1 vim-minimal
      1 xterm
Binary requires are Ok.

Package builds in F19 (http://koji.fedoraproject.org/koji/taskinfo?taskID=4969543). Ok.

Package is good.
Resolution: Package approved.

New Package SCM Request
=======================
Package Name: snotes
Short Description: A flexible and easy to use notes system
Owners: psabata
Branches: f17 f18
InitialCC:

Git done (by process-git-requests).

Thank you both.

snotes-1.0-2.fc18 has been submitted as an update for Fedora 18.
https://admin.fedoraproject.org/updates/snotes-1.0-2.fc18

snotes-1.0-2.fc17 has been submitted as an update for Fedora 17.
https://admin.fedoraproject.org/updates/snotes-1.0-2.fc17

snotes-1.0-2.fc17 has been pushed to the Fedora 17 stable repository.

snotes-1.0-2.fc18 has been pushed to the Fedora 18 stable repository.