{"action":"created","issue":{"url":"https://api.github.com/repos/UV-CDAT/uvcdat/issues/1370","labels_url":"https://api.github.com/repos/UV-CDAT/uvcdat/issues/1370/labels{/name}","comments_url":"https://api.github.com/repos/UV-CDAT/uvcdat/issues/1370/comments","events_url":"https://api.github.com/repos/UV-CDAT/uvcdat/issues/1370/events","html_url":"https://github.com/UV-CDAT/uvcdat/pull/1370","id":86325876,"number":1370,"title":"fix to get pyopensll build again","user":{"login":"doutriaux1","id":2781425,"avatar_url":"https://avatars.githubusercontent.com/u/2781425?v=3","gravatar_id":"","url":"https://api.github.com/users/doutriaux1","html_url":"https://github.com/doutriaux1","followers_url":"https://api.github.com/users/doutriaux1/followers","following_url":"https://api.github.com/users/doutriaux1/following{/other_user}","gists_url":"https://api.github.com/users/doutriaux1/gists{/gist_id}","starred_url":"https://api.github.com/users/doutriaux1/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/doutriaux1/subscriptions","organizations_url":"https://api.github.com/users/doutriaux1/orgs","repos_url":"https://api.github.com/users/doutriaux1/repos","events_url":"https://api.github.com/users/doutriaux1/events{/privacy}","received_events_url":"https://api.github.com/users/doutriaux1/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2015-06-08T20:48:43Z","updated_at":"2015-06-09T10:47:25Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/UV-CDAT/uvcdat/pulls/1370","html_url":"https://github.com/UV-CDAT/uvcdat/pull/1370","diff_url":"https://github.com/UV-CDAT/uvcdat/pull/1370.diff","patch_url":"https://github.com/UV-CDAT/uvcdat/pull/1370.patch"},"body":"@isubsmith please test on your system.\r\n\r\n@aashish24 @dlonie @sankhesh feel free to start reviewing\r\n\r\n@aashish24 I know you don't like building ffi, but until you guys implemented the USE_SYSTEM we have to build it (plus we will still need to build it if users don't have root and can't add this packages)\r\n\r\n@cdampi bit is only for PARALLEL build, I had to do this on my system to get mpi and esmf parallel to install correctly. If anybody as issues with it we can revert for this branch to what's in master\r\npnetcdf is also only built in parallel mode, same thing if anybody has issues with it we can take it off from this branch."},"comment":{"url":"https://api.github.com/repos/UV-CDAT/uvcdat/issues/comments/110316602","html_url":"https://github.com/UV-CDAT/uvcdat/pull/1370#issuecomment-110316602","issue_url":"https://api.github.com/repos/UV-CDAT/uvcdat/issues/1370","id":110316602,"user":{"login":"isubsmith","id":5223761,"avatar_url":"https://avatars.githubusercontent.com/u/5223761?v=3","gravatar_id":"","url":"https://api.github.com/users/isubsmith","html_url":"https://github.com/isubsmith","followers_url":"https://api.github.com/users/isubsmith/followers","following_url":"https://api.github.com/users/isubsmith/following{/other_user}","gists_url":"https://api.github.com/users/isubsmith/gists{/gist_id}","starred_url":"https://api.github.com/users/isubsmith/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/isubsmith/subscriptions","organizations_url":"https://api.github.com/users/isubsmith/orgs","repos_url":"https://api.github.com/users/isubsmith/repos","events_url":"https://api.github.com/users/isubsmith/events{/privacy}","received_events_url":"https://api.github.com/users/isubsmith/received_events","type":"User","site_admin":false},"created_at":"2015-06-09T10:47:25Z","updated_at":"2015-06-09T10:47:25Z","body":"This must be against master/HEAD?\n\nI was hoping to build 2.2 release but I suppose I can switch.\n\nBrian Smith\nsmithbe@ornl.gov<mailto:smithbe@ornl.gov>\n\n\n\nOn Jun 8, 2015, at 4:48 PM, Charles Doutriaux <notifications@github.com<mailto:notifications@github.com>> wrote:\n\n\n@isubsmith<https://github.com/isubsmith> please test on your system.\n\n@aashish24<https://github.com/aashish24> @dlonie<https://github.com/dlonie> @sankhesh<https://github.com/sankhesh> feel free to start reviewing\n\n@aashish24<https://github.com/aashish24> I know you don't like building ffi, but until you guys implemented the USE_SYSTEM we have to build it (plus we will still need to build it if users don't have root and can't add this packages)\n\n@cdampi bit is only for PARALLEL build, I had to do this on my system to get mpi and esmf parallel to install correctly. If anybody as issues with it we can revert for this branch to what's in master\npnetcdf is also only built in parallel mode, same thing if anybody has issues with it we can take it off from this branch.\n\n________________________________\nYou can view, comment on, or merge this pull request online at:\n\n  https://github.com/UV-CDAT/uvcdat/pull/1370\n\nCommit Summary\n\n  *   pipinstaller was using @ instead of $ for variable\n\nFile Changes\n\n  *   M CMake/cdat_modules/cffi_deps.cmake<https://github.com/UV-CDAT/uvcdat/pull/1370/files#diff-0> (2)\n  *   M CMake/cdat_modules/cython_pkg.cmake<https://github.com/UV-CDAT/uvcdat/pull/1370/files#diff-1> (8)\n  *   M CMake/cdat_modules/hdf5_deps.cmake<https://github.com/UV-CDAT/uvcdat/pull/1370/files#diff-2> (3)\n  *   M CMake/cdat_modules/hdf5_external.cmake<https://github.com/UV-CDAT/uvcdat/pull/1370/files#diff-3> (14)\n  *   M CMake/cdat_modules/hdf5_pkg.cmake<https://github.com/UV-CDAT/uvcdat/pull/1370/files#diff-4> (4)\n  *   M CMake/cdat_modules/netcdf_deps.cmake<https://github.com/UV-CDAT/uvcdat/pull/1370/files#diff-5> (3)\n  *   M CMake/cdat_modules/netcdf_external.cmake<https://github.com/UV-CDAT/uvcdat/pull/1370/files#diff-6> (13)\n  *   M CMake/cdat_modules/netcdf_pkg.cmake<https://github.com/UV-CDAT/uvcdat/pull/1370/files#diff-7> (4)\n  *   A CMake/cdat_modules/pnetcdf_deps.cmake<https://github.com/UV-CDAT/uvcdat/pull/1370/files#diff-8> (1)\n  *   A CMake/cdat_modules/pnetcdf_external.cmake<https://github.com/UV-CDAT/uvcdat/pull/1370/files#diff-9> (17)\n  *   A CMake/cdat_modules/pnetcdf_pkg.cmake<https://github.com/UV-CDAT/uvcdat/pull/1370/files#diff-10> (13)\n  *   A CMake/cdat_modules/pycparser_deps.cmake<https://github.com/UV-CDAT/uvcdat/pull/1370/files#diff-11> (1)\n  *   A CMake/cdat_modules/pycparser_external.cmake<https://github.com/UV-CDAT/uvcdat/pull/1370/files#diff-12> (6)\n  *   A CMake/cdat_modules/pycparser_pkg.cmake<https://github.com/UV-CDAT/uvcdat/pull/1370/files#diff-13> (10)\n  *   M CMake/cdat_modules/pyopenssl_deps.cmake<https://github.com/UV-CDAT/uvcdat/pull/1370/files#diff-14> (2)\n  *   A CMake/cdat_modules_extra/cdatmpi_configure_step.cmake.in<https://github.com/UV-CDAT/uvcdat/pull/1370/files#diff-15> (21)\n  *   M CMake/cmake_modules/pipinstaller.cmake<https://github.com/UV-CDAT/uvcdat/pull/1370/files#diff-16> (4)\n  *   M CMakeLists.txt<https://github.com/UV-CDAT/uvcdat/pull/1370/files#diff-17> (15)\n\nPatch Links:\n\n  *   https://github.com/UV-CDAT/uvcdat/pull/1370.patch\n  *   https://github.com/UV-CDAT/uvcdat/pull/1370.diff\n\n—\nReply to this email directly or view it on GitHub<https://github.com/UV-CDAT/uvcdat/pull/1370>.\n\n"}}