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

Robert-André Mauchin 🐧 <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[email protected]
              Flags|                            |needinfo?([email protected]
                   |                            |)



--- Comment #5 from Robert-André Mauchin 🐧 <[email protected]> ---
Finishing this review or do you need help?

 - Use libdir here too:


    -DHDF5_LIB_PATH:PATH=/usr/lib \

    -DHDF5_LIB_PATH:PATH=%{_libdir} \

 - %{_builddir}/%{name}-%{version} is not needed here, at each stage, prep
build and install, you are cd to this directory, so you can use CMakeLists.txt
and . directly:

%{_builddir}/%{name}-%{version}/CMakeLists.txt


%cmake
[…]
    %{_builddir}/%{name}-%{version}

 - In order to avoid untintentional SONAME bump, we recommend not globbing the
major soname version in library. Be more specific instead:

%{_libdir}/libkea.so.1*

 - You must install the LICENSE.txt with %license in %files

 - You should provide README.md with %doc in %files


-- 
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]
To unsubscribe send an email to [email protected]
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/[email protected]

Reply via email to