Bug 983961 (pyqt-mail-checker)

It seems there is nothing to deal in %build section.

You can add a note to explain.

Issue:

You forgot to update the icon cache, see:

http://fedoraproject.org/wiki/Packaging:ScriptletSnippets#Icon_Cache

Please provide a *downloadable* link:
https://raw.github.com/F1ash/pyqt-mail-checker/2.0.1/pyqt-mail-checker.spec
Your spec file link points to the website, not to the raw file. Well, I cannot speak for other reviewers, but I like to download both spec and srpm to a local folder and have a look at them. And if someone wants to view the file online, a browser should also be able to display plain text correctly. Github in particular (and probably some other VCS web viewers) doesn't have syntax highlighting, so there's no advantage of viewing it in the browser.

Additionally to the missing icon cache update mentioned by Christopher, I'm missing the desktop file scriptlet:
http://fedoraproject.org/wiki/Packaging:Guidelines#desktop-file-install_usage

make install DESTDIR=$RPM_BUILD_ROOT/usr
Don't use /usr. We have the %{_prefix} macro therefore.

Thanks :)

SPEC: https://raw.github.com/F1ash/pyqt-mail-checker/2.0.1/pyqt-mail-checker.spec
SRPM: http://kojipkgs.fedoraproject.org//work/tasks/9420/5619420/pyqt-mail-checker-2.0.1-2.fc19.src.rpm

(In reply to Mario Blättermann from comment #3)
> Additionally to the missing icon cache update mentioned by Christopher, I'm
> missing the desktop file scriptlet:
> http://fedoraproject.org/wiki/Packaging:Guidelines#desktop-file-install_usage

Please add the appropriate scriptlet. Your comment about desktop-file-utils is wrong. This build requirement is not needed for updating the icon cache, but for the installation or validation of a desktop file.

Ups, sorry...

SPEC: https://raw.github.com/F1ash/pyqt-mail-checker/2.0.1/pyqt-mail-checker.spec
SRPM: http://kojipkgs.fedoraproject.org//work/tasks/2714/5622714/pyqt-mail-checker-2.0.1-3.fc19.src.rpm

You've mixed 2 styles for build root tag. 

Should only use macro style or variable style. 

Which one do you prefer, %{buildroot} or $RPM_BUILD_ROOT?

> You've mixed 2 styles for build root tag. 
> 
> Should only use macro style or variable style. 
> 
SPEC: https://raw.github.com/F1ash/pyqt-mail-checker/2.0.1/pyqt-mail-checker.spec
SRPM: http://kojipkgs.fedoraproject.org//work/tasks/7155/5627155/pyqt-mail-checker-2.0.1-4.fc19.src.rpm

The srpm is not available. Please fix. I don't know how Koji behaves regarding such packages, maybe the package has been crowded unintendendly, as it happens with scratch builds after two weeks? Wouldn't it be better to use your fedorapeople webspace instead?

The %{name} tag inside %description is not needed as long as you don't have subpackages which need a different description.

> %description
> %{name}

That's very unusual, since the description is supposed to be constructed of full sentences.

Same for sub-packages. %name would not be used at all in a description's first line. Unless you specify the full sub-package name directly behind the %description macro:

  %description -n %{name}-subname

Normally one simply uses the short form for sub-package names derived from the base %name:

  %description subname

Fresh build: http://koji.fedoraproject.org/koji/taskinfo?taskID=5673839

SPEC: https://raw.github.com/F1ash/pyqt-mail-checker/2.0.2/pyqt-mail-checker.spec
SRPM: http://f1ash.fedorapeople.org/pyqt-mail-checker/pyqt-mail-checker-2.0.2-1.fc19.src.rpm

1. pyqt-mail-checker.noarch: E: non-executable-script /usr/share/pyqt-mail-checker/SettingsDialog/__init__.py 0644L /usr/bin/python
pyqt-mail-checker.noarch: E: non-executable-script /usr/share/pyqt-mail-checker/misc/__init__.py 0644L /usr/bin/python
pyqt-mail-checker.noarch: E: non-executable-script /usr/share/pyqt-mail-checker/Utils/__init__.py 0644L /usr/bin/python
pyqt-mail-checker.noarch: E: non-executable-script /usr/share/pyqt-mail-checker/MessageStack/__init__.py 0644L /usr/bin/python
pyqt-mail-checker.noarch: E: non-executable-script /usr/share/pyqt-mail-checker/Threads/__init__.py 0644L /usr/bin/python
pyqt-mail-checker.noarch: E: non-executable-script /usr/share/pyqt-mail-checker/__init__.py 0644L /usr/bin/python

2. Can you rename your binary to %{name} but not %{name}.py?

Fresh build: http://koji.fedoraproject.org/koji/taskinfo?taskID=5686528

SPEC: https://raw.github.com/F1ash/pyqt-mail-checker/2.0.2/pyqt-mail-checker.spec
SRPM: http://f1ash.fedorapeople.org/pyqt-mail-checker/pyqt-mail-checker-2.0.2-2.fc19.src.rpm

APPROVED.

Please fix the grammar in your Summary, one checks FOR new messages, and "a messages" doesn't make sense, there's no indefinite article in the plural in English.

"mailboxes list" also sounds strange.

(In reply to Kevin Kofler from comment #15)
> Please fix the grammar in your Summary, one checks FOR new messages, and "a
> messages" doesn't make sense, there's no indefinite article in the plural in
> English.
> 
> "mailboxes list" also sounds strange.

"The applet checks for the new messages in the mail boxes periodically"

^^ This variant is correct?

Sorry, but still no. I understand that the proper use of articles is hard to grasp if your mother tongue doesn't have them.

I'd say "for new messages" rather than "for the new messages". As I said, the plural of "a" (indefinite article) is no article at all, and here we want the indefinite form: "for the new messages" implies there are some specific messages we need to check for (i.e. we already know which messages we are talking about), but that doesn't make sense because if we have to check for new messages, we don't even know whether there are any messages.

And "mailbox" is usually written in one word, that was also correct initially.

Then, "The applet checks for new messages in the mailboxes periodically" is a correct sentence, but still doesn't sound right. I'd write it as:

Summary: Applet periodically checking for new messages in the mailboxes

Notice how I put the noun describing what the package is first, without an article (as the original summary also had it, and most RPM Summary tags are written in that style) and how the Summary is a noun phrase ("applet checking …") rather than a sentence ("the applet checks…"). I also put "periodically" before the verb, which is more common usage in English.

...
> Summary: Applet periodically checking for new messages in the mailboxes
> 
https://raw.github.com/F1ash/pyqt-mail-checker/2.0.2/pyqt-mail-checker.spec

Thanks :)
This is a good lesson.

New Package SCM Request
=======================
Package Name: pyqt-mail-checker
Short Description: Applet periodically checking for new messages in the mailboxes
Owners: f1ash
Branches: f18 f19
InitialCC:

Git done (by process-git-requests).

pyqt-mail-checker-2.0.2-3.fc18 has been submitted as an update for Fedora 18.
https://admin.fedoraproject.org/updates/pyqt-mail-checker-2.0.2-3.fc18

pyqt-mail-checker-2.0.2-3.fc19 has been submitted as an update for Fedora 19.
https://admin.fedoraproject.org/updates/pyqt-mail-checker-2.0.2-3.fc19

pyqt-mail-checker-2.0.2-3.fc18 has been pushed to the Fedora 18 testing repository.

pyqt-mail-checker-2.0.2-3.fc19 has been pushed to the Fedora 19 stable repository.

pyqt-mail-checker-2.0.2-3.fc18 has been pushed to the Fedora 18 stable repository.