[Bug 522988] Review Request: dyndns - A dynamic DNS client for the free service afraid.org

[bugzilla at redhat.com]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=522988





--- Comment #9 from Erick Calder <e at arix.com>  2009-09-14 23:47:44 EDT ---
(In reply to comment #5)
> rpmlint is also unhappy with the binary package: :)

gone as per BuildArch

> Adding BuildArch: noarch will remove the first and third error

done

> Concerning the zero-length file I'm not sure. Is it really necessary
> to package the empty file /var/cache/dyndns?

I've ghosted it as per Christoph's suggestion, with a half-hearted objection.

> Erick, you should also increase the release tag every time you make changes to
> the spec file (so the current release is 2). 

done

> Finally two minor notes: your email address given in the changelog seems to
> lack at least the TLD. Also, the list items in each changeset should start 
> with a hyphen (even if there's only one entry)

done

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.