[Bug 1005785] Review Request: snowball - Libstemmer Java bindings

2014-01-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1005785

Michael Simacek msima...@redhat.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
   Assignee|nob...@fedoraproject.org|msima...@redhat.com
  Flags||fedora-review?



--- Comment #5 from Michael Simacek msima...@redhat.com ---
 i dont need C libraries ... and for now i dont have intentions to package
 this libs

Ok. It is distributed separately by upstream, so it shouldn't be a problem.

The description is now ok, but I still insist on changing the name. If someone
decided to package the snowball compiler or the C version of the library, there
would be problem. snowball-java or libstemmer-java sounds more reasonable.

Regarding that POM - why do you add dependency on junit? I don't see any unit
test in there.

-- 
You are receiving this mail because:
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1005785] Review Request: snowball - Libstemmer Java bindings

2014-01-17 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1005785

Michael Simacek msima...@redhat.com changed:

   What|Removed |Added

 CC||msima...@redhat.com



--- Comment #1 from Michael Simacek msima...@redhat.com ---
I think the name is not correct. There is also a C version of this available on
the site. Therefore we need to diambiguate the name in case someone wanted to
package the C version of algorithms.

The summary also doesn't seem right. If those are bindings - then bindings to
what? On the site it is summarized as The Java version of the libstemmer
library. That also raises a question whether it should be really called
snowball. TBH I don't really understand what the package is supposed to do, but
it seems to me that sentence You can use the various ANSI C and Java stemmers
in programs of your own, without bothering yourself with the Snowball system
that generated them. implies that this is not snowball itself, but rather
something generated by it. Therefore I think it shouldn't be called snowball,
but libstemmer-java Java as indicated by the name of the tarball.

The descritption is insufficient, it doesn't say anything about the library
itself.

Also, where did you get that POM?

-- 
You are receiving this mail because:
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1005785] Review Request: snowball - Libstemmer Java bindings

2014-01-17 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1005785



--- Comment #2 from Michael Simacek msima...@redhat.com ---
Looking at their documentation and sources I made a different conclusion:
You should package http://snowball.tartarus.org/dist/snowball_code.tgz as
snowball instead, because that is the compiler that generated it and also the
source for the generated code. Thre is a GNUmakefile that can build the
compiler and generate the C and Java libraries (it seems like it needs to be
patched a little to get the Java version, though), which should become
snowball's subpackages.

-- 
You are receiving this mail because:
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1005785] Review Request: snowball - Libstemmer Java bindings

2014-01-17 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1005785



--- Comment #3 from gil cattaneo punto...@libero.it ---
(In reply to Michael Simacek from comment #1)
 I think the name is not correct. There is also a C version of this available
 on the site. Therefore we need to diambiguate the name in case someone
 wanted to package the C version of algorithms.
 
 The summary also doesn't seem right. If those are bindings - then bindings
 to what? On the site it is summarized as The Java version of the libstemmer
 library. That also raises a question whether it should be really called
 snowball. TBH I don't really understand what the package is supposed to do,
 but it seems to me that sentence You can use the various ANSI C and Java
 stemmers in programs of your own, without bothering yourself with the
 Snowball system that generated them. implies that this is not snowball
 itself, but rather something generated by it. Therefore I think it shouldn't
 be called snowball, but libstemmer-java Java as indicated by the name of the
 tarball.
 

 The descritption is insufficient, it doesn't say anything about the library
 itself.
 

better ?

Snowball is a small string processing language
designed for creating stemming algorithms
for use in Information Retrieval.

This package contains all you need to include the
snowball stemming algorithms into a Java
project of your own. If you use this,
you don't need to use the snowball compiler,
or worry about the internals of the
stemmers in any way.

 Also, where did you get that POM?

created by me

-- 
You are receiving this mail because:
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1005785] Review Request: snowball - Libstemmer Java bindings

2014-01-17 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1005785



--- Comment #4 from gil cattaneo punto...@libero.it ---
(In reply to Michael Simacek from comment #2)
 Looking at their documentation and sources I made a different conclusion:
 You should package http://snowball.tartarus.org/dist/snowball_code.tgz as
 snowball instead, because that is the compiler that generated it and also
 the source for the generated code. Thre is a GNUmakefile that can build the
 compiler and generate the C and Java libraries (it seems like it needs to be
 patched a little to get the Java version, though), which should become
 snowball's subpackages.

i dont need C libraries ... and for now i dont have intentions to package this
libs

-- 
You are receiving this mail because:
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1005785] Review Request: snowball - Libstemmer Java bindings

2013-09-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1005785

gil cattaneo punto...@libero.it changed:

   What|Removed |Added

 CC|package-review@lists.fedora |
   |project.org |
  Alias||snowball



-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=WOmE0yr6gRa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1005785] Review Request: snowball - Libstemmer Java bindings

2013-09-09 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1005785

gil cattaneo punto...@libero.it changed:

   What|Removed |Added

 Blocks||1005792



-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=YyEps4oQFSa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review