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

Artur Frenszek-Iwicki <fed...@svgames.pl> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |fed...@svgames.pl
           Doc Type|---                         |If docs needed, set a value



--- Comment #1 from Artur Frenszek-Iwicki <fed...@svgames.pl> ---
>Group:         Sciences/Geosciences
Not used in Fedora.
https://docs.fedoraproject.org/en-US/packaging-guidelines/#_tags_and_sections

>%description
>KEALib is a project to provide an implementation of the GDAL specification 
>within the the HDF5 file format. Specifically, the format will support...
Whoa. The description should be wrapped so that all lines are 80 characters
max.
https://docs.fedoraproject.org/en-US/packaging-guidelines/#_summary_and_description

>mkdir -p %{_target_platform}
>pushd %{_target_platform}
Not needed. CMake out-of-source builds are now the default.

>%cmake \
>    -DLIBDIR:PATH=/usr/lib64 \
>    -DGDAL_LIB_PATH:PATH=/usr/lib64 \
Fedora still builds packages for some 32-bit architectures (i686 and armv7hl,
to be precise) - use "%{_libdir}" here.

>make %{?_smp_mflags} -C %{_target_platform}
Use "%cmake_build" instead.
https://docs.fedoraproject.org/en-US/packaging-guidelines/CMake/

>make DESTDIR=%{buildroot} install
Try "%cmake_install" instead.

>mkdir -p %{buildroot}%{_libdir}/ %{buildroot}%{_libdir}/gdalplugins/
>mv %{buildroot}%{_prefix}/lib/libkea* %{buildroot}%{_libdir}/
>mv %{buildroot}%{_prefix}/lib/gdalplugins/* %{buildroot}%{_libdir}/gdalplugins/
Add some check whether "%{_prefix}/lib" is different than "%{_libdir}" so that
this doesn't fail on 32-bit architectures. Alternatively, try patching the
CMakeLists.

>%files
>%{_libdir}/libkea.so*
>%{_includedir}/libkea
Un-versioned .so files and C/C++ headers should go in a -devel package.
https://docs.fedoraproject.org/en-US/packaging-guidelines/#_devel_packages


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

Reply via email to