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.

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

Use STDOUT for logs from engine and auto searches #654

Merged
merged 3 commits into from
Feb 26, 2021

Conversation

shadowspawn
Copy link
Collaborator

@shadowspawn shadowspawn commented Feb 19, 2021

Pull Request

Problem

Do not expect any output from this:

n --quiet --download engine >/dev/null

Issue: #653

Solution

The auto and engine related routines were logging to STDERR to allow returning function result on STDOUT. This made it awkward to suppress routine logging when running commands using target version of auto or engine.

Refactor to log to STDOUT and internally return function result in global.

ChangeLog

  • consistently log to STDOUT

@shadowspawn shadowspawn changed the title Use STDOUT for logs Use STDOUT for logs from engine and auto searches Feb 19, 2021
@shadowspawn shadowspawn merged commit c7aaa3d into develop Feb 26, 2021
@shadowspawn shadowspawn deleted the feature/stdout-for-logs branch February 27, 2021 01:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant