Bug 246524 (libquikgrid) - Review Request: libquikgrid - The QuikGrid generator and contouring routine library
Summary: Review Request: libquikgrid - The QuikGrid generator and contouring routine ...
Alias: libquikgrid
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
Target Milestone: ---
Assignee: Nicolas Chauvet (kwizart)
QA Contact: Fedora Extras Quality Assurance
Depends On:
TreeView+ depends on / blocked
Reported: 2007-07-02 20:19 UTC by Rick L Vinyard Jr
Modified: 2013-02-19 11:12 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Last Closed: 2013-02-19 11:12:58 UTC
Type: ---

Attachments (Terms of Use)

Description Rick L Vinyard Jr 2007-07-02 20:19:52 UTC
Spec URL: http://miskatonic.cs.nmsu.edu/pub/libquikgrid.spec
SRPM URL: http://miskatonic.cs.nmsu.edu/pub/libquikgrid-4.0-1.src.rpm

This library contains the grid generator (called xpand) and contouring routine
(called contour) in the QuikGrid program.

Comment 2 Nicolas Chauvet (kwizart) 2007-07-03 11:49:56 UTC
from spec file:
mv lesser.txt LICENSE
you should better use this. That will prevent timestramp  (install used as a
better cp )
install -pm 0644 lesser.txt LICENSE
Also i wonder if this:
"Permission to copy for non commercial purposes is granted provided the source
is acknowledged." is allowded by LGPL as this suggest that commercial
distribution (like RHEL), is not allowed to redistribute it freely...?!
Also this http://www.perspectiveedge.com/disclaimer.html disagree with the LGPL
license. (Does this page plan be updated ?)

Does this license change have been made recently ?
Upstream suggest that 5.3 is avaible but only 0.4 source is downloadable...

You should usually not uses %{version} for patch (unless changes are made for
the patch between each version) This will prevent to uses cvs history for the
file... For that, that would be better to merge your patch upstream also...

From build.log:
ckward_warning.h:32:2: warning: #warning This file includes at least one depreca
ted or antiquated header. Please consider using one of the 32 headers found in s
ection of the C++ standard. Examples include substituting the <X> heade
r for the <X.h> header for C++ includes, or <iostream> instead of the deprecated
 header <iostream.h>. To disable this warning use -Wno-deprecated.
 -> This should be reported upstream (only warnings anyway)

That would be better that archive name follow the name package, i don't knwo if
upstream will change it also. I think that you set the right package name...

License in unclear
Merge autotools patch upstream (with the license in the main package)

Comment 3 Rick L Vinyard Jr 2007-07-03 20:17:08 UTC
Here's the latest version:
Spec URL: http://miskatonic.cs.nmsu.edu/pub/libquikgrid.spec
SRPM URL: http://miskatonic.cs.nmsu.edu/pub/libquikgrid-4.0-3.fc7.src.rpm

- Removed version from autotools patch
- Removed renaming of license and readme files
- Fixed CFLAGS in quikgrid.pc.in
- Fixed deprecated c++ headers in patch

Rather than messing with the install option I'll just leave the files alone and
put them in the docs.

With respect to the license, I think there are two issues. The version 5.3
refers to the QuikGrid windows application which is at version 5.3, whereas this
package is the quikgrid grid generation and contouring library which is at
version 4.0.

I was also a little confused about the way the license was worded, since the
non-commercial seemed contradictory to the statement that it was under the LGPL.
What you sometimes see is statements like "licensed under the LGPL except for
commercial purposes", etc. I wasn't sure if it was dual-licensed, originally
licensed non-commercial then then LGPL added, etc.

To clarify I contacted the author and his reply was:
"Is the problem that first line "Permission to copy for non commercial purposes
is granted"? If so I will delete it - I had no intention of putting any
limitations on the LGPL license. My general feeling is that I would like to see
the software used where it is useful and I don't want to put a lot of roadblocks
in front of people who want to do that and are willing to share changes to just
those functions... If I simply repackage it with a reference to the new LGPL and
place it on my web site I assume that is all that is required? I hope I can put
this off until August when we get back from our cruising."

Comment 4 Nicolas Chauvet (kwizart) 2007-07-05 17:13:50 UTC
The problem with this license if more serious...
I cannot approve this package because the integrated license from the original
package is not compatible with Fedora licenses...

As soon as we have a new package from the original website 
(with the new license), then we can integrate it to the collection...

continuing with others reviews...

Comment 5 Rick L Vinyard Jr 2007-08-12 17:23:03 UTC
The author has updated the license to remove the confusion and issued the
clarifying statement on the website.

I'll get the source updated in the next week or so and publish a new revision.

Comment 6 Nicolas Chauvet (kwizart) 2007-08-12 18:02:19 UTC

So, do we have some news about minizip to be enabled from withit zlib ?!
There is also a mini update with the appropriate license bundled (but not
autotools patched merged in...)

I will take a quick look on vtp asap...

Comment 7 Nicolas Chauvet (kwizart) 2007-10-03 15:58:36 UTC
Indeed, the license is there (update the license field to  LGPLv2 )
Can you update the spec for version 5 ?

Comment 8 Rick L Vinyard Jr 2007-11-09 18:17:14 UTC
I'll have an updated spec next week.

Comment 9 Rick L Vinyard Jr 2007-12-01 22:02:22 UTC
Well, not quite next week, but close.

Here are the new files:

Comment 10 Nicolas Chauvet (kwizart) 2007-12-18 14:43:41 UTC
- License is LGPLv2+ (mean you can use any later version) - sorry for the mistake
Please use %{SOURCE1} etc to define the source path in %prep
- BuildRequiremenet and use for dos2unix could be avoided by using sed:
sed -i 's/\r//' readme.txt lesser.txt
then please keep timestamps with any reference file like:
touch -r reference_file.txt readme.txt lesser.txt

- Please keep directory ownership for %{_includedir}/quikgrid, like:
%dir %{_includedir}/quikgrid

rpmlint on installed package:
undefined-non-weak-symbol /usr/lib64/libquikgrid.so.0.0.0 _Z8DoLineToffi
You are probably missing a LDFLAG compiling libquikgrid... This can cause others
package to fails while linking to it...(or maybe this is related to mini as i'm
using F-7 x86_64 - anyway zlib-devel wasn't BuildRequired, so I'm still
searching where it come from)

I don't know if there is something wrong with the headers as it do not include
others files (so I wonder if there isn't some missing).

For the quikgrid.pc , there are something missing, it should point to the
appropriate include directory:
with includedir=/usr/include/quikgrid
also version seems wrong (it is 5.0)

I may try to build something against libquikgrid for proper testing...

Comment 11 Rick L Vinyard Jr 2008-05-22 20:46:44 UTC
Here are the new spec and srpm. These should take everything into account:


Comment 12 Nicolas Chauvet (kwizart) 2008-05-23 12:22:03 UTC
This still seems unsafe to me to approve a package when the build tools aren't
merged in the upstream tarball. Did you have further explanation from the
upstream author about why autotools support hasn't been merged in the archive ?
(same for libMini).

Comment 13 Nicolas Chauvet (kwizart) 2008-06-23 19:49:30 UTC
ping ? (this is also true for libMini)
Can you cc'd me when you will ask the mains developpers of the related packages ?

Comment 14 Rakesh Pandit 2008-09-03 14:46:08 UTC
update is required here?
Will be closed within one week in case no update is provided.

Comment 15 Rick L Vinyard Jr 2008-09-05 15:30:39 UTC
Still planning to work on it soon. I need to contact upstream and see if they'll incorporate the build tools.

Comment 16 Rick L Vinyard Jr 2008-09-22 15:21:27 UTC
I contacted upstream and the short version is that for now to build files won't be incorporated.

The long version is that the creator of quikgrid (John Coulthard) is semi-retired and really doesn't want to maintain quikgrid any more so 4.0 is going to be _his_ last release.

But, the project isn't dead. John is transitioning quikgrid to Ben Discoe (the VTP maintainer) who is setting up quikgrid on either google code or sourceforge.

I've already contacted Ben and I think we'll be able to transition the build files to upstream once the quikgrid project is transitioned.

Since 4.0 works, I think that the inclusion of the build files for now is mitigated by the fact that there is a pathway to transition the build files into quikgrid in the future.

Comment 17 Nicolas Chauvet (kwizart) 2008-10-14 08:21:53 UTC
Is there a newer snapshot for this ?
I will test the whole dependencies against current rawhide using my own experimental repository.

Comment 18 Rick L Vinyard Jr 2008-10-14 20:46:35 UTC
Unfortunately, no snapshots. Just the 4.0 release.

Comment 19 Nicolas Chauvet (kwizart) 2008-10-28 09:54:32 UTC
rpmlint on installed package:
libquikgrid.x86_64: W: undefined-non-weak-symbol /usr/lib64/libquikgrid.so.0.0.0 _ZN8QuikGrid8DoLineToEffi
Can this be fixed easily ? 

Any news from buildtools integration ?

Comment 20 Rick L Vinyard Jr 2008-11-07 21:00:25 UTC
No word yet. I'm hoping to hear something soon since it would simplify some of the VTP patches as well.

Comment 21 Nicolas Chauvet (kwizart) 2009-01-22 14:47:38 UTC
Is there any news ?

Comment 22 Rick L Vinyard Jr 2009-02-02 16:17:18 UTC
I should have some time in the next week or two to move this forward a bit.

Comment 23 Miroslav Suchý 2013-02-19 11:12:58 UTC
Stalled Review. Closing per:
If you ever want to continue with this review, please reopen or
submit new review.

Note You need to log in before you can comment on or make changes to this bug.