{"action":"created","issue":{"url":"https://api.github.com/repos/xiaoyeli/superlu/issues/6","repository_url":"https://api.github.com/repos/xiaoyeli/superlu","labels_url":"https://api.github.com/repos/xiaoyeli/superlu/issues/6/labels{/name}","comments_url":"https://api.github.com/repos/xiaoyeli/superlu/issues/6/comments","events_url":"https://api.github.com/repos/xiaoyeli/superlu/issues/6/events","html_url":"https://github.com/xiaoyeli/superlu/pull/6","id":135133294,"number":6,"title":"Do not set TPL_BLAS_LIBRARIES this prevents usage such as -lxxx from …","user":{"login":"BarrySmith","id":512185,"avatar_url":"https://avatars.githubusercontent.com/u/512185?v=3","gravatar_id":"","url":"https://api.github.com/users/BarrySmith","html_url":"https://github.com/BarrySmith","followers_url":"https://api.github.com/users/BarrySmith/followers","following_url":"https://api.github.com/users/BarrySmith/following{/other_user}","gists_url":"https://api.github.com/users/BarrySmith/gists{/gist_id}","starred_url":"https://api.github.com/users/BarrySmith/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/BarrySmith/subscriptions","organizations_url":"https://api.github.com/users/BarrySmith/orgs","repos_url":"https://api.github.com/users/BarrySmith/repos","events_url":"https://api.github.com/users/BarrySmith/events{/privacy}","received_events_url":"https://api.github.com/users/BarrySmith/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":3,"created_at":"2016-02-20T23:15:06Z","updated_at":"2016-03-15T19:08:20Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/xiaoyeli/superlu/pulls/6","html_url":"https://github.com/xiaoyeli/superlu/pull/6","diff_url":"https://github.com/xiaoyeli/superlu/pull/6.diff","patch_url":"https://github.com/xiaoyeli/superlu/pull/6.patch"},"body":"…working which is neccesary for complicated blas installs\r\n\r\nThe current code is also unnecessary\r\n"},"comment":{"url":"https://api.github.com/repos/xiaoyeli/superlu/issues/comments/196977339","html_url":"https://github.com/xiaoyeli/superlu/pull/6#issuecomment-196977339","issue_url":"https://api.github.com/repos/xiaoyeli/superlu/issues/6","id":196977339,"user":{"login":"BarrySmith","id":512185,"avatar_url":"https://avatars.githubusercontent.com/u/512185?v=3","gravatar_id":"","url":"https://api.github.com/users/BarrySmith","html_url":"https://github.com/BarrySmith","followers_url":"https://api.github.com/users/BarrySmith/followers","following_url":"https://api.github.com/users/BarrySmith/following{/other_user}","gists_url":"https://api.github.com/users/BarrySmith/gists{/gist_id}","starred_url":"https://api.github.com/users/BarrySmith/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/BarrySmith/subscriptions","organizations_url":"https://api.github.com/users/BarrySmith/orgs","repos_url":"https://api.github.com/users/BarrySmith/repos","events_url":"https://api.github.com/users/BarrySmith/events{/privacy}","received_events_url":"https://api.github.com/users/BarrySmith/received_events","type":"User","site_admin":false},"created_at":"2016-03-15T19:08:20Z","updated_at":"2016-03-15T19:08:20Z","body":"  The blas/lapack libraries are generated automatically by scripts (they are often not given by users directly) and they can be very nasty with many -R etc types of options, for example what MKL requires or strange stuff introduced by the Fortran compiler (that is needed to link against blas that is generated with the Fortran compiler dependencies). There is no way to resolve them down to a list of fully resolved libraries. In the real world blas not just /usr/lib/libblas.a\n\n   Aside from the fact that accepting these messy things violates CMAKE \"conventions\" there does not seem to be any harm in allowing them.\n\n   I'll note that Trilinos accepts these messy lists without full resolution; otherwise PETSc couldn't use the Trilinos cmake :-)\n\n  Barry\n\n\n\n> On Mar 15, 2016, at 1:56 PM, Roscoe A. Bartlett <notifications@github.com> wrote:\n> \n> Barry, why is this change needed? How is PETSc trying to install SuperLU? I general, I think you should not be specifying -l<lib> with CMake. That is not really the CMake way as I understand it. See https://cmake.org/cmake/help/v3.5/command/target_link_libraries.html . CMake will often translate <abs-path>/lib<lib-name>.[a|so] into -l<lib-name> but that is CMake's business. Why can't PETSc just supply the absolute paths to the BLAS libraries? What error are getting without this change? That was the agreement for the TPL_<TPL_NAME>_LIBRARIES variable.\n\n \n> \n> —\n> You are receiving this because you authored the thread.\n> Reply to this email directly or view it on GitHub\n> \n\n"}}