-
Notifications
You must be signed in to change notification settings - Fork 34
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Windows #7
Open
claudiabm777
wants to merge
5
commits into
e-mission:master
Choose a base branch
from
claudiabm777:windows
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Windows #7
Changes from all commits
Commits
Show all changes
5 commits
Select commit
Hold shift + click to select a range
c965acd
new windows configuration to run phone and server
claudiabm777 605676c
fixing some extra issues like spaces, etc
claudiabm777 4d455dc
fixing some extra issues like spaces, etc
claudiabm777 027d13c
new windows configuration to run phone and server fixing spaces an style
claudiabm777 22903b8
script on new folder called scripts
claudiabm777 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,54 @@ | ||
# How to get the code running on Windows for testing | ||
There are some different steps to follow in order to run the e-mission server and the e-mission phone app in Windows from the command prompt. In this document, we will highlight the steps in the main documentation page (https://github.com/e-mission/e-mission-server#dependencies) where there are different steps to perform. | ||
|
||
Before start following the steps above, the first thing you want to do is to create this path in your computer: | ||
``` | ||
C:\var\tmp | ||
``` | ||
## e-mission server | ||
### Python distribution | ||
After installing and configuring Anaconda, there is the following step "*Setup the emission environment*". Instead of running the command stated in this step, please run the following commands into your command prompt: | ||
|
||
``` | ||
conda env update --name emission --file setup/environment36.yml | ||
activate emission | ||
pip install six --upgrade | ||
``` | ||
|
||
Note that every time you need to run the code, you have to run the command ```activate emission```. Also, note that you must be at the root of the project in order to access the directory ```setup/``` in the first command stated before. | ||
|
||
### Development | ||
In the development section, we will change the step 3 "*Start the server*". Instead of doing that, first, you have to create or update the following environment variable on windows: | ||
|
||
- Name of the variable: PYTHONPATH | ||
- Value of the variable: C:\MyPath\Anaconda3;C:\MyPath\e-mission-server | ||
|
||
Note that in the field "*Value of the variable*" there are two paths separated by a semicolon. The first one is the path on your computer to the folder in which you installed Anaconda. The second is the path in your computer where you have the root folder of the server code. | ||
|
||
After doing so, you have to copy the e-mission.bat script (located at scripts/overview/running_project_windows) into the root of your project and run the following command: | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I would be happy to have There actually used to be an |
||
|
||
``` | ||
e-mission.bat . emission/net/api/cfc_webapp.py | ||
``` | ||
|
||
Yes, with that dot in the middle... | ||
So, note that every time you see the command ```./e-mission-py.bash``` in the documentation, you need to run instead ```e-mission.bat . ``` (with the dot after ".bat"). | ||
## e-mission phone | ||
### Installation | ||
In step 4 of the installation called "Setup the config" you have run following commands: | ||
|
||
``` | ||
cd bin | ||
node configure_xml_and_json.js "serve" | ||
``` | ||
|
||
You need to change the property "setup-serve" in the files: | ||
- package.json | ||
- package.cordobabuild.json | ||
- package.serve.json | ||
You have to change it to look like the following: | ||
|
||
``` | ||
"setup-serve": "node bin/download_settings_controls.js" | ||
``` | ||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,5 @@ | ||
@ECHO OFF | ||
setlocal | ||
set PYTHONPATH=%1 | ||
python %2 %3 | ||
endlocal |
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
there are steps required for both server and phone. So maybe something like
or something like that