Bug#849387: RFS: farmhash/0~20161014-g92e897b-1 [ITP]

2016-12-29 Thread lumin
On Tue, 2016-12-27 at 22:47 +0100, Adam Borowski wrote:
> I don't see why you would do that...?

It's my bad -- I misunderstood your comments.

> What I meant is that you install a lot of cruft that doesn't appear to be
> user docs at all

I agree with that. I reverted the package split and moved those
stuff (incl. test suite) into the example directory. So the doc
directory is cleaner:

  $ ls /usr/share/doc/libfarmhash-dev/
  changelog.Debian.gz  copyright  examples/  NEWS.gz  README.gz  
Understanding_Hash_Functions.gz

There is no file seems like a library guide there, so maybe the user
will go take a look at the header directly.

package contents:
http://debomatic-amd64.debian.net/distribution#experimental/farmhash/0~20161014-g92e897b-1/contents

mentors:
https://mentors.debian.net/debian/pool/main/f/farmhash/farmhash_0~20161014-g92e897b-1.dsc

dom-amd64 build:
http://debomatic-amd64.debian.net/distribution#experimental/farmhash/0~20161014-g92e897b-1/buildlog



Bug#849387: RFS: farmhash/0~20161014-g92e897b-1 [ITP]

2016-12-27 Thread Adam Borowski
On Tue, Dec 27, 2016 at 09:14:25AM +, lumin wrote:
> On Tue, 2016-12-27 at 05:10 +0100, Adam Borowski wrote:
> > 
> > I'm afraid that the -dev package installs a lot of cruft to /usr/share/doc/,
> > this includes all the sources and test cases for the testsuite, instructions
> > how to run the testsuite, and so on.
> > 
> > That place is meant for user documentation, or possibly examples.
> 
> Thank you for reviewing the package. Initially I was thinking of the
> convenience of shipping the docs in the -dev package but the fact shows
> the docs are somewhat mess...

There's nothing wrong with shipping docs in -dev if they don't take too much
space, or break other considerations (such as multiarch).  And here you
hardly have any actual docs: besides "NEWS" (a changelog) there's a "README"
that describes some performance notes and (irrelevant for a compiled
package) build instructions, and "Understanding_Hash_Functions" (some generic
design talk).

> I've split up those docs and examples into a separate package
> farmhash-doc, and uploaded the latest version to mentors:

I don't see why you would do that...?

What I meant is that you install a lot of cruft that doesn't appear to be
user docs at all: testsuite sources, testsuite test cases, a warning about
required syntax for the testsuite, that's it.  Nothing that's of any use to
someone who wants to use the library in his own programs rather than to make
changes in the library itself -- ie, stuff that's pointless outside a source
package.

Thus sorry if you misunderstood me, but I did not request a split, but
questioned inclusion of stuff in the docs directory (and now package).


On the other hand, I see no end-user docs, although the installed .h is
moderately easy to read.


Meow!
-- 
Autotools hint: to do a zx-spectrum build on a pdp11 host, type:
  ./configure --host=zx-spectrum --build=pdp11



Bug#849387: RFS: farmhash/0~20161014-g92e897b-1 [ITP]

2016-12-27 Thread lumin
On Tue, 2016-12-27 at 05:10 +0100, Adam Borowski wrote:
> 
> I'm afraid that the -dev package installs a lot of cruft to /usr/share/doc/,
> this includes all the sources and test cases for the testsuite, instructions
> how to run the testsuite, and so on.
> 
> That place is meant for user documentation, or possibly examples.

Thank you for reviewing the package. Initially I was thinking of
the convenience of shipping the docs in the -dev package but
the fact shows the docs are somewhat mess...

I've split up those docs and examples into a separate package
farmhash-doc, and uploaded the latest version to mentors:

https://mentors.debian.net/debian/pool/main/f/farmhash/farmhash_0~20161014-g92e897b-1.dsc

DoM-amd64 build is passing:

http://debomatic-amd64.debian.net/distribution#experimental/farmhash/0~20161014-g92e897b-1/buildlog



Bug#849387: RFS: farmhash/0~20161014-g92e897b-1 [ITP]

2016-12-26 Thread Adam Borowski
Control: tags -1 +moreinfo

On Mon, Dec 26, 2016 at 03:23:22PM +, lumin wrote:
>   I am looking for a sponsor for my package "farmhash"
> 
>  * Package name: farmhash
>    Version : 0~20161014-g92e897b-1
>  * URL : https://github.com/google/farmhash

I'm afraid that the -dev package installs a lot of cruft to /usr/share/doc/,
this includes all the sources and test cases for the testsuite, instructions
how to run the testsuite, and so on.

That place is meant for user documentation, or possibly examples.


Meow!
-- 
Autotools hint: to do a zx-spectrum build on a pdp11 host, type:
  ./configure --host=zx-spectrum --build=pdp11



Bug#849387: RFS: farmhash/0~20161014-g92e897b-1 [ITP]

2016-12-26 Thread lumin
Package: sponsorship-requests
Severity: wishlist

Note,
* This is a Tensorflow dependency.
* DoM-amd64 build is passing.
  
http://debomatic-amd64.debian.net/distribution#experimental/farmhash/0~20161014-g92e897b-1/buildlog

Dear mentors,

  I am looking for a sponsor for my package "farmhash"

 * Package name: farmhash
   Version : 0~20161014-g92e897b-1
   Upstream Author : Google Inc.
 * URL : https://github.com/google/farmhash
 * License : Expat
   Section : science

  It builds those binary packages:

libfarmhash-dev - FarmHash, a family of hash functions (development files, 
docs)
 libfarmhash0 - FarmHash, a family of hash functions (shared library)

  To access further information about this package, please visit the following 
URL:

  https://mentors.debian.net/package/farmhash


  Alternatively, one can download the package with dget using this command:

dget -x 
https://mentors.debian.net/debian/pool/main/f/farmhash/farmhash_0~20161014-g92e897b-1.dsc


  Changes since the last upload:

farmhash (0~20161014-g92e897b-1) experimental; urgency=medium

  * Initial release. (Closes: #848883)