[Bug 1750501] Review Request: fastbit - An Efficient Compressed Bitmap Index Technology (UNRETIRE)

2019-09-17 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1750501

Zbigniew Jędrzejewski-Szmek  changed:

   What|Removed |Added

 Status|POST|CLOSED
   Fixed In Version||fastbit-2.0.3-14.fc29
 Resolution|--- |RAWHIDE
Last Closed||2019-09-17 15:40:02



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1750501] Review Request: fastbit - An Efficient Compressed Bitmap Index Technology (UNRETIRE)

2019-09-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1750501



--- Comment #14 from Philip Kovacs  ---
They restored the fastbit dist-git and unblocked the needed branches.  All
done.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1750501] Review Request: fastbit - An Efficient Compressed Bitmap Index Technology (UNRETIRE)

2019-09-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1750501



--- Comment #13 from Philip Kovacs  ---
Zbyszek, 

I will make the changes you discussed and thank you for your time and effort. 

Phil

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1750501] Review Request: fastbit - An Efficient Compressed Bitmap Index Technology (UNRETIRE)

2019-09-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1750501

Zbigniew Jędrzejewski-Szmek  changed:

   What|Removed |Added

  Flags||fedora-review+



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1750501] Review Request: fastbit - An Efficient Compressed Bitmap Index Technology (UNRETIRE)

2019-09-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1750501

Zbigniew Jędrzejewski-Szmek  changed:

   What|Removed |Added

 Status|NEW |POST
 CC||zbys...@in.waw.pl
   Assignee|nob...@fedoraproject.org|zbys...@in.waw.pl



--- Comment #12 from Zbigniew Jędrzejewski-Szmek  ---
Replying to some earlier comments first:

> Unversioned so-files
> 
> fastbit-java: /usr/lib64/fastbit/libfastbitjni.so
That's OK. The file is not in the normal library lookup path.

> - Package does not use a name that already exists.
This is a re-review, so the package obviously exists. fedora-review could be
smarter, but it's not.
When using fedora-review, please remove such comments in the future from it's
output when it is
clear that it is wrong.

> # Main package is BSD; the Bison-generated parsers are GPLv3+ with exceptions;
> # contribs/fbmerge is GPLv2+
> License:BSD and GPLv3+ with exceptions and GPLv2+

This one actually should be changed. A license describes the *binary* package
[1]. In this particular case,
the parser parts can only be included in the package because of the license
exception. As the GPLv3-exception
header says, "you may distribute that work under the terms of your choice".
This is exactly what is happening
here: the authors are distributing fastbit under the BSD license. Please remove
any mention of GPLv3+ from the
License lines.

Also, if fbmerge is GPLv2+, then this doesn't matter for the -devel and -java
subpackages. Their license
is just "BSD".

[1]
https://fedoraproject.org/wiki/Licensing:FAQ?rd=Licensing/FAQ#Does_the_License:_tag_cover_the_SRPM_or_the_binary_RPM.3F

Otherwise, looks OK.
+ package name is OK
+ latest version
+ builds and installs OK
+ fedora-review is happy
+ scriptlets look OK (there are none ;))
+ Provides/Requires/BR look OK

Package is RE-APPROVED. Please fix the license bits when re-importing.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1750501] Review Request: fastbit - An Efficient Compressed Bitmap Index Technology (UNRETIRE)

2019-09-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1750501



--- Comment #11 from Philip Kovacs  ---
SPEC: https://src.fedoraproject.org/rpms/fastbit/raw/f30/f/fastbit.spec
SRPM:
https://kojipkgs.fedoraproject.org//packages/fastbit/2.0.3/13.fc30/src/fastbit-2.0.3-13.fc30.src.rpm

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org