search

ARPA-SIMC / libsim

Command-line utilities and fortran modules for meteorological data processing
GNU General Public License v2.0
7 stars 1 forks source link

Remove some dependencies from BuildRequires #90

Closed edigiacomo closed 2 years ago

edigiacomo commented 2 years ago

If I understand correctly, the following BuildRequires are not direct dependencies and they can be safely removed:

dcesari commented 2 years ago

In principle I agree, but moncic said no: https://simc.arpae.it/moncic-ci/libsim/202202080820/master/centos7/build.log, we probably need to erase -lcairo somewhere else, maybe in che configure.ac

edigiacomo commented 2 years ago

I'm sorry, It Worked On My Computer™ :grimacing: ... Yesterday I tested only Rocky Linux 8, but in CentOS 7 ncl-devel doesn't require cairo-devel nor png-devel

[edigiacomo@rocky8 ~]$ rpm -qR ncl-devel | grep cairo
cairo-devel
[edigiacomo@sette ~]$ rpm -qR ncl-devel | grep cairo

Let's see if I fixed the issue in commit 823feb0 :crossed_fingers:

dcesari commented 2 years ago

Stavo provando la stessa cosa :-)

edigiacomo commented 2 years ago

Fixed https://simc.arpae.it/moncic-ci/libsim/last/master/centos7/build.log