{"action":"created","issue":{"url":"https://api.github.com/repos/siconos/siconos/issues/94","repository_url":"https://api.github.com/repos/siconos/siconos","labels_url":"https://api.github.com/repos/siconos/siconos/issues/94/labels{/name}","comments_url":"https://api.github.com/repos/siconos/siconos/issues/94/comments","events_url":"https://api.github.com/repos/siconos/siconos/issues/94/events","html_url":"https://github.com/siconos/siconos/issues/94","id":197733599,"number":94,"title":"removing unneeded component dependencies","user":{"login":"radarsat1","id":66805,"avatar_url":"https://avatars.githubusercontent.com/u/66805?v=3","gravatar_id":"","url":"https://api.github.com/users/radarsat1","html_url":"https://github.com/radarsat1","followers_url":"https://api.github.com/users/radarsat1/followers","following_url":"https://api.github.com/users/radarsat1/following{/other_user}","gists_url":"https://api.github.com/users/radarsat1/gists{/gist_id}","starred_url":"https://api.github.com/users/radarsat1/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/radarsat1/subscriptions","organizations_url":"https://api.github.com/users/radarsat1/orgs","repos_url":"https://api.github.com/users/radarsat1/repos","events_url":"https://api.github.com/users/radarsat1/events{/privacy}","received_events_url":"https://api.github.com/users/radarsat1/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":4,"created_at":"2016-12-27T17:25:27Z","updated_at":"2016-12-27T18:14:25Z","closed_at":null,"body":"While testing deb packages I'm noticing a lot of unnecessary dependencies, and the packaging tools complain as well. e.g.,\r\n~~~\r\ndpkg-shlibdeps: warning: package could avoid a useless dependency if\r\ndebian/libsiconos-numerics/usr/lib/x86_64-linux-gnu/libsiconos_numerics.so.4.0.0\r\nwas not linked against liblapack.so.3 (it uses none of the library's symbols)\r\n~~~\r\n(hundreds of these..)\r\n\r\nPerhaps we could figure out how to get cmake to remove unneeded libraries from certain components.  For instance, if you compile with WITH_BULLET=ON, then `ldd libsiconos_numerics.so` tells you that it links to libBulletCollision, etc., even though this is only really needed for `libsiconos_mechanics.so`.  I suspect a bit of work with refining e.g. `SICONOS_numerics_LIBRARIES` instead of throwing everything into `SICONOS_LINK_LIBRARIES` might do the trick.\r\n\r\nAlthough in general I think this is non-trivial to fix.\r\n\r\nI'll note that I get similar warnings for the Python modules:\r\n~~~\r\ndpkg-shlibdeps: warning: package could avoid a useless dependency if [...]debian/python-siconos/usr/lib/python2.7/dist-packages/siconos/mechanics/_joints.x86_64-linux-gnu.so were not linked against libopenblas.so.0 (they use none of the library's symbols)\r\n~~~\r\netc.."},"comment":{"url":"https://api.github.com/repos/siconos/siconos/issues/comments/269362042","html_url":"https://github.com/siconos/siconos/issues/94#issuecomment-269362042","issue_url":"https://api.github.com/repos/siconos/siconos/issues/94","id":269362042,"user":{"login":"radarsat1","id":66805,"avatar_url":"https://avatars.githubusercontent.com/u/66805?v=3","gravatar_id":"","url":"https://api.github.com/users/radarsat1","html_url":"https://github.com/radarsat1","followers_url":"https://api.github.com/users/radarsat1/followers","following_url":"https://api.github.com/users/radarsat1/following{/other_user}","gists_url":"https://api.github.com/users/radarsat1/gists{/gist_id}","starred_url":"https://api.github.com/users/radarsat1/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/radarsat1/subscriptions","organizations_url":"https://api.github.com/users/radarsat1/orgs","repos_url":"https://api.github.com/users/radarsat1/repos","events_url":"https://api.github.com/users/radarsat1/events{/privacy}","received_events_url":"https://api.github.com/users/radarsat1/received_events","type":"User","site_admin":false},"created_at":"2016-12-27T18:14:25Z","updated_at":"2016-12-27T18:14:25Z","body":"True, I'll have to test on other platforms.  So far it looks like it works quite well for the Debian packages!\r\nmechanics is now the only one that `ldd` reports linking to bullet, and numerics is the only one linking to lpsolve for example. I'll leave the flag on for now."}}
{"actor":{"display_login":"radarsat1"}}