https://bugzilla.redhat.com/show_bug.cgi?id=1199296



--- Comment #2 from Devrim GÜNDÜZ <[email protected]> ---

Hi,

Thanks for the review. I fixed most of the stuff, except:

(In reply to Sinny Kumari from comment #1)
> 
> Issues found
> -------------
> * Any significance of having ._configure.ac, ._Makefile.am and ._README file
> in source tar?

That comes from upstream. I will ask them to remove them in next releases.

> Why does --includedir=%{_includedir}/laszip is used to include include
> headers of same package here?

The packages that look for laszip-devel dependency are looking into
/usr/include/laszip, and the Makefile installs these under /usr/include, so I
had to force it to that directory.

So:

Spec URL: http://www.gunduz.org/epel/laszip.spec
SRPM URL: http://www.gunduz.org/epel/laszip-2.2.0-3.rhel7.src.rpm


Regards, Devrim

-- 
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
[email protected]
https://admin.fedoraproject.org/mailman/listinfo/package-review

Reply via email to