Bug 246524 (libquikgrid)

Spec URL: http://miskatonic.cs.nmsu.edu/pub/libquikgrid.spec
SRPM URL: http://miskatonic.cs.nmsu.edu/pub/libquikgrid-4.0-2.fc7.src.rpm

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 17.4.1.2 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...

sum-up:
License in unclear
Merge autotools patch upstream (with the license in the main package)

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

Changelog:
- 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."

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...

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.

good!

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...

Indeed, the license is there (update the license field to  LGPLv2 )
Can you update the spec for version 5 ?

I'll have an updated spec next week.

Well, not quite next week, but close.

Here are the new files:
http://miskatonic.cs.nmsu.edu/pub/libquikgrid-5.0-1.fc8.src.rpm
http://miskatonic.cs.nmsu.edu/pub/libquikgrid.spec

- 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:
%{_includedir}/quikgrid/
  or
%dir %{_includedir}/quikgrid
%{_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:
-I${includedir}
with includedir=/usr/include/quikgrid
also version seems wrong (it is 5.0)

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

Here are the new spec and srpm. These should take everything into account:

http://miskatonic.cs.nmsu.edu/pub/libquikgrid.spec
http://miskatonic.cs.nmsu.edu/pub/libquikgrid-5.0-2.fc9.src.rpm

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).

ping ? (this is also true for libMini)
Can you cc'd me when you will ask the mains developpers of the related packages ?

@Rick
update is required here?
Will be closed within one week in case no update is provided.

Still planning to work on it soon. I need to contact upstream and see if they'll incorporate the build tools.

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.

Is there a newer snapshot for this ?
I will test the whole dependencies against current rawhide using my own experimental repository.

Unfortunately, no snapshots. Just the 4.0 release.

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 ?

No word yet. I'm hoping to hear something soon since it would simplify some of the VTP patches as well.

Is there any news ?

I should have some time in the next week or two to move this forward a bit.

Stalled Review. Closing per:
https://fedoraproject.org/wiki/Policy_for_stalled_package_reviews
If you ever want to continue with this review, please reopen or
submit new review.