Re: RFS: softbeep (updated package)
On Fri, Jan 26, 2007 at 02:18:56PM +0100, Thijs Kinkhorst wrote:
> * There are open bugs in the bts for softbeep, which have no response
> yet, and might have some merit. Have you looked at the bugs? Can they
> be fixed in this upload? If not, you could respond to the reporters
> why it will not (yet) be resolved.
Ok, will look at it.
> * I personally think that "188.8.131.52" is too precise for a
> standards-version, since the last part should not make any difference
> for a package. I suggest to keep only the significant "3.7.2" part.
Ok, kept it as it is.
> * You install the upstream file "RELEASES" as documentation, but it is
> some sort of a changelog. I propose to have it installed by
> dh_installchangelogs instead.
Well, that's how the package was. Anyways, I just read the manpage of
dh_installchangelogs, but I don't understand how to use it to specify
RELEASES as a changelog.
> * Your patches have very generic names: 01-shlibs.dpatch 02-sb.dpatch
> 03-sb-beep.dpatch, and there's multiple unrelated changes per file.
> I'd prefer if the filenames describe what the patch does, and that you
> separate the patches per subject. This makes it easy to add, update or
> remove a specific patch.
Again that's how the package was, I only added 03-sb-beep.dpatch.