You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@climate.apache.org by lewismc <gi...@git.apache.org> on 2014/11/14 00:57:17 UTC
[GitHub] climate pull request: CLIMATE-548 Vagrant references should not ex...
GitHub user lewismc opened a pull request:
https://github.com/apache/climate/pull/137
CLIMATE-548 Vagrant references should not exist in easy-ocw ubuntu script
Pull request for https://issues.apache.org/jira/browse/CLIMATE-548
Additionally, this patch renames the Ubuntu install script and also updates this within the Vagrant file.
The Mac OSX script is not named by version, the Ubuntu script should shadow this behavior.
I am NOT sure about the new Python parth... is it OK to assume that Climate will always ben installed in ${HOME}/climate ?
i personally don't think it is but I want to hear from others please.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/lewismc/climate CLIMATE-548
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/climate/pull/137.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #137
----
commit ea719279d9e257001d82bf9407fb9bfbe0985571
Author: Lewis John McGibbney <le...@jpl.nasa.gov>
Date: 2014-11-13T23:42:29Z
CLIMATE-548 Vagrant references should not exist in easy-ocw ubuntu script
commit bf966683d2e828864ac88fd5b1d88e8f74a25534
Author: Lewis John McGibbney <le...@jpl.nasa.gov>
Date: 2014-11-13T23:55:15Z
CLIMATE-548 Vagrant references should not exist in easy-ocw ubuntu script
----
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] climate pull request: CLIMATE-548 Vagrant references should not ex...
Posted by MJJoyce <gi...@git.apache.org>.
Github user MJJoyce commented on the pull request:
https://github.com/apache/climate/pull/137#issuecomment-62992138
Hrm, it's probably not safe to assume that it will always be in ~ Lewis, you're right. We could explicitly check it out for them there or have them pass the location to "install" to.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] climate pull request: CLIMATE-548 Vagrant references should not ex...
Posted by OCWJenkins <gi...@git.apache.org>.
Github user OCWJenkins commented on the pull request:
https://github.com/apache/climate/pull/137#issuecomment-63929340
Merged build finished. Test Passed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] climate pull request: CLIMATE-548 Vagrant references should not ex...
Posted by lewismc <gi...@git.apache.org>.
Github user lewismc commented on the pull request:
https://github.com/apache/climate/pull/137#issuecomment-63855402
Passing it to install is the best I would say. I'll cook something up @MJJoyce but it won't be until just before the weekend. Chocca block right now!
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] climate pull request: CLIMATE-548 Vagrant references should not ex...
Posted by MJJoyce <gi...@git.apache.org>.
Github user MJJoyce commented on the pull request:
https://github.com/apache/climate/pull/137#issuecomment-65747266
:+1: Get that bugger in there
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] climate pull request: CLIMATE-548 Vagrant references should not ex...
Posted by OCWJenkins <gi...@git.apache.org>.
Github user OCWJenkins commented on the pull request:
https://github.com/apache/climate/pull/137#issuecomment-63929081
Merged build triggered. Test Failed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] climate pull request: CLIMATE-548 Vagrant references should not ex...
Posted by OCWJenkins <gi...@git.apache.org>.
Github user OCWJenkins commented on the pull request:
https://github.com/apache/climate/pull/137#issuecomment-62989248
Merged build started. Test Failed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] climate pull request: CLIMATE-548 Vagrant references should not ex...
Posted by OCWJenkins <gi...@git.apache.org>.
Github user OCWJenkins commented on the pull request:
https://github.com/apache/climate/pull/137#issuecomment-63929084
Merged build started. Test Failed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] climate pull request: CLIMATE-548 Vagrant references should not ex...
Posted by OCWJenkins <gi...@git.apache.org>.
Github user OCWJenkins commented on the pull request:
https://github.com/apache/climate/pull/137#issuecomment-65747062
Merged build started. Test Failed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] climate pull request: CLIMATE-548 Vagrant references should not ex...
Posted by OCWJenkins <gi...@git.apache.org>.
Github user OCWJenkins commented on the pull request:
https://github.com/apache/climate/pull/137#issuecomment-62989239
Merged build triggered. Test Failed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] climate pull request: CLIMATE-548 Vagrant references should not ex...
Posted by MJJoyce <gi...@git.apache.org>.
Github user MJJoyce commented on the pull request:
https://github.com/apache/climate/pull/137#issuecomment-63856491
:+1: let's get it in there before 0.5. I'll update the ticket so we remember.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] climate pull request: CLIMATE-548 Vagrant references should not ex...
Posted by OCWJenkins <gi...@git.apache.org>.
Github user OCWJenkins commented on the pull request:
https://github.com/apache/climate/pull/137#issuecomment-62989326
Merged build finished. Test Passed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] climate pull request: CLIMATE-548 Vagrant references should not ex...
Posted by lewismc <gi...@git.apache.org>.
Github user lewismc commented on the pull request:
https://github.com/apache/climate/pull/137#issuecomment-63855438
Would like to resolve this before 0.5 release
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] climate pull request: CLIMATE-548 Vagrant references should not ex...
Posted by OCWJenkins <gi...@git.apache.org>.
Github user OCWJenkins commented on the pull request:
https://github.com/apache/climate/pull/137#issuecomment-65747053
Merged build triggered. Test Failed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] climate pull request: CLIMATE-548 Vagrant references should not ex...
Posted by lewismc <gi...@git.apache.org>.
Github user lewismc commented on the pull request:
https://github.com/apache/climate/pull/137#issuecomment-63929134
@MJJoyce and folks,
please see most recent push.
Now hopefully prompts user for full path to where they currently have OCW downloaded. We then make this variable the basis for updating python path with correct paths to executables.
Please test and let me know how things are.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] climate pull request: CLIMATE-548 Vagrant references should not ex...
Posted by lewismc <gi...@git.apache.org>.
Github user lewismc commented on the pull request:
https://github.com/apache/climate/pull/137#issuecomment-65747812
commit c8d49eb324598f2901dac7e207f91e6b6c9b37cc
Merge: 9bcae94 3827882
Author: Lewis John McGibbney <le...@jpl.nasa.gov>
Date: Thu Dec 4 20:56:28 2014 -0800
Merge branch 'CLIMATE-548'
commit 38278829c68cda55583683e587eed322e9f31e65
Author: Lewis John McGibbney <le...@jpl.nasa.gov>
Date: Thu Dec 4 20:43:21 2014 -0800
CLIMATE-548 Vagrant references should not exist in easy-ocw ubuntu script
commit 8617af0c09cee2760bc2f967d4c29b1ba2476487
Author: Lewis John McGibbney <le...@jpl.nasa.gov>
Date: Thu Nov 20 22:18:59 2014 -0800
CLIMATE-548 Vagrant references should not exist in easy-ocw ubuntu script
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] climate pull request: CLIMATE-548 Vagrant references should not ex...
Posted by lewismc <gi...@git.apache.org>.
Github user lewismc commented on the pull request:
https://github.com/apache/climate/pull/137#issuecomment-65747064
Hi folks,
I've verified this patch against Ubuntu 14.04 VM and also documented within the script execution console output that the script will run A OK on that OS.
I'm going to commit this tonight unless I hear anything and then we can push a release. :) :) :)
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] climate pull request: CLIMATE-548 Vagrant references should not ex...
Posted by OCWJenkins <gi...@git.apache.org>.
Github user OCWJenkins commented on the pull request:
https://github.com/apache/climate/pull/137#issuecomment-65747150
Merged build finished. Test Passed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] climate pull request: CLIMATE-548 Vagrant references should not ex...
Posted by MJJoyce <gi...@git.apache.org>.
Github user MJJoyce commented on the pull request:
https://github.com/apache/climate/pull/137#issuecomment-63848716
How do we want to move forward with this @lewismc?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] climate pull request: CLIMATE-548 Vagrant references should not ex...
Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:
https://github.com/apache/climate/pull/137
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---