{"action":"created","issue":{"url":"https://api.github.com/repos/hashdist/hashstack/issues/639","labels_url":"https://api.github.com/repos/hashdist/hashstack/issues/639/labels{/name}","comments_url":"https://api.github.com/repos/hashdist/hashstack/issues/639/comments","events_url":"https://api.github.com/repos/hashdist/hashstack/issues/639/events","html_url":"https://github.com/hashdist/hashstack/pull/639","id":57965080,"number":639,"title":"Build PETSc with ssl only when a dependency","user":{"login":"johannesring","id":3841655,"avatar_url":"https://avatars.githubusercontent.com/u/3841655?v=3","gravatar_id":"","url":"https://api.github.com/users/johannesring","html_url":"https://github.com/johannesring","followers_url":"https://api.github.com/users/johannesring/followers","following_url":"https://api.github.com/users/johannesring/following{/other_user}","gists_url":"https://api.github.com/users/johannesring/gists{/gist_id}","starred_url":"https://api.github.com/users/johannesring/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/johannesring/subscriptions","organizations_url":"https://api.github.com/users/johannesring/orgs","repos_url":"https://api.github.com/users/johannesring/repos","events_url":"https://api.github.com/users/johannesring/events{/privacy}","received_events_url":"https://api.github.com/users/johannesring/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":8,"created_at":"2015-02-17T18:53:19Z","updated_at":"2015-02-19T00:18:29Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/hashdist/hashstack/pulls/639","html_url":"https://github.com/hashdist/hashstack/pull/639","diff_url":"https://github.com/hashdist/hashstack/pull/639.diff","patch_url":"https://github.com/hashdist/hashstack/pull/639.patch"},"body":"When building `python` with HashDist it will be built with `openssl` and when building `petsc` it will by default be built with `ssl` from the system. This leads to problems like the following when building `dolfin`:\r\n\r\n    CMake Warning at bench/fem/speedup/cpp/CMakeLists.txt:44 (add_executable):\r\n      Cannot generate a safe runtime search path for target bench_solve-poisson\r\n      because files in some directories may conflict with libraries in implicit\r\n      directories:\r\n    \r\n        runtime library [libssl.so.1.0.0] in /usr/lib/x86_64-linux-gnu may be hidden by files in:\r\n          /home/martinal/.hashdist/bld/profile/5c5xotjofrcc/lib\r\n        runtime library [libcrypto.so.1.0.0] in /usr/lib/x86_64-linux-gnu may be hidden by files in:\r\n          /home/martinal/.hashdist/bld/profile/5c5xotjofrcc/lib\r\n    \r\n      Some of these libraries may not be found correctly.\r\n\r\nThe problem is that `python` and `petsc` is linked with different `ssl` libraries.\r\n\r\nThis PR makes sure that `ssl` is only turned on for `petsc` when `openssl` is added as a dependency. Otherwise it is turned off."},"comment":{"url":"https://api.github.com/repos/hashdist/hashstack/issues/comments/74977382","html_url":"https://github.com/hashdist/hashstack/pull/639#issuecomment-74977382","issue_url":"https://api.github.com/repos/hashdist/hashstack/issues/639","id":74977382,"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":"2015-02-19T00:18:29Z","updated_at":"2015-02-19T00:18:29Z","body":"Hmm, is there no way to comment on another comment? \r\n\r\nThe default SSL is not actually used by the crappy web interface voodoo; it is used by something even more obscure, the ability to access Google Docs and Globus files from the application. \r\n\r\nOne reason for making it auto is that it helps very quickly detect problems with the code since the ssl.py tests and the source code in PETSc related to it are always run. By making it off by default it gets pretty much no testing nor chance for finding the problems.\r\n\r\nAnd actually it has worked out pretty well. We've only had two issues reported related to the SSL stuff (one by Mark Adams and one here) which is much more stable than almost any of the \"optional\" packages we support. \r\n\r\nWith regard to cross compilation, as with all our tests the tests are suppose to handle the cross compilation business properly, that is they do not just test the \"thing\" on the host and assume that that it works on the cross system, the test is suppose to be relevent for the true system it will be run on. Of course this doesn't help when some vendors put dummy stubs into their libraries so ./configure runs fine but then the \"dummy stubs\" fail at run time.\r\n\r\nGiven that you guys have a solution to the problem and we've no other legitiment complaints I don't see any reason to not make it default.\r\n\r\n"}}