https://bugzilla.redhat.com/show_bug.cgi?id=1199296 --- Comment #2 from Devrim GÜNDÜZ <devrim@xxxxxxxxxx> --- 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 package-review@xxxxxxxxxxxxxxxxxxxxxxx https://admin.fedoraproject.org/mailman/listinfo/package-review