Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Create a log file of the install to aid debugging #64

Open
borpin opened this issue Oct 25, 2019 · 2 comments
Open

Create a log file of the install to aid debugging #64

borpin opened this issue Oct 25, 2019 · 2 comments
Labels
enhancement New feature or request

Comments

@borpin
Copy link
Contributor

borpin commented Oct 25, 2019

When running the scripts, I currently tee it to a log file.

./main.sh 2>&1 | tee install.log

I suggest that this is done by default to aid debugging if issues occur.
@borpin borpin mentioned this issue Oct 25, 2019
Closed
@TrystanLea TrystanLea added the enhancement New feature or request label Jan 23, 2020
@borpin
Copy link
Contributor Author

borpin commented Feb 10, 2023

Add to the documentation

CDuffers pushed a commit to CDuffers/EmonScripts that referenced this issue Mar 24, 2023
In repsonse to issue (enhancement) openenergymonitor#64
d9316c9 
The install.sh and main.sh have been updated to copy stdout and stderr to both the terminal and also a log file.
The script echos the log file name at the beginning and end of the script
'tee -a' used in case the scripts are run multiple times
In addition main.sh is changed to move the shell directive to the top of the script and to remove superfluous code after the "exit 0" in the final if statement. Also removed the exit to enable the script output to be shown using one command after the if
@CDuffers CDuffers mentioned this issue Mar 25, 2023
Merged
@CDuffers
Copy link
Contributor

PR #182 submitted to fulfil this request

TrystanLea added a commit that referenced this issue Oct 18, 2023
@TrystanLea
Merge pull request #182 from CDuffers/logging
6c09f8d 
Enhancement to log init.sh and main.sh install scripts from issue #64
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@TrystanLea @borpin @CDuffers