search

OpenModelica / OpenModelicaBuildScripts

Scripts to build OpenModelica packages on miscellaneous platforms
3 stars 9 forks source link

Adding libsuitesparseconfig to debian install #7

Closed AnHeuermann closed 3 years ago

AnHeuermann commented 3 years ago

I also ordered the libs a tiny bit.

AnHeuermann commented 3 years ago

@sjoelund This should fix https://trac.openmodelica.org/OpenModelica/ticket/6375. Can you review and merge if its looking correct?

sjoelund commented 3 years ago

Ah, I thought suitesparse was used by OMC, too. So I had it as part of libomc. If you add the config to here, you need to remove it from the other file

AnHeuermann commented 3 years ago

Ah I didn't even see you had fixed it already until just now.

SuiteSparse should only be used from the C and C++ runtime, but both link to the libs in lib/**/omc/. So I think adding them to https://github.com/OpenModelica/OpenModelicaBuildScripts/blob/master/debian/libomcsimulation.install should be better, at least the other SuiteSparse libs are there as well.

AnHeuermann commented 3 years ago

Is the order important? I just sorted them alphabetically. And how are the *.install files are used?

sjoelund commented 3 years ago

Order doesn't matter. The files listed become part of the corresponding .deb package. You can't have the same file in two installed packages.

AnHeuermann commented 3 years ago

I think we should merge this PR so the master and maintenance/v1.17 branches are in sync.