On 2017/12/03 00:52, Klemens Nanni wrote: > On Sat, Dec 02, 2017 at 09:26:43PM +0000, Stuart Henderson wrote: > > On 2017/12/02 01:59, Klemens Nanni wrote: > > > Will "# statically linked" suffice? > > > > Yes. > > > > How about this diff on top of my tarball? > > > > Please send updated tarballs for changes to a new port - a diff in the > > accompanying mail can help to explain changes, but it's better not to > > ask reviewers to dig through old mails to assemble the whole thing. > New tarball attached incorporating all your feedback. > > > This still makes the whole file revolve around the fact that it's a > > reimplementation of sloccount. That's really of minor interest to DESCR, > > the primary function is to say what the package does - if the details of > > a similar program are needed at all then one paragraph at the bottom > > should suffice? > Point taken. > > > (Something like "loccount is a re-implementation of David A. Wheeler's > > sloccount tool in Go. The algorithms are largely unchanged and, bugs > > excepted, can be expected to produce identical numbers for languages > > supported by both tools." perhaps ..) > I slightly rephrased your suggestion and described how it works/what it > does.
What I am saying is, the DESCR file is first and foremost there to describe what the thing does. That is what should be first in the file. Your rewritten DESCR still has "loccount is a re-implementation of David A. Wheeler's sloccount tool in Go" at the top of the file. Apart from DESCR this looks good to me.
