Skip to content
This repository has been archived by the owner on Mar 3, 2023. It is now read-only.

--log-file parameter didn't create log file #15144

Closed
kenorb opened this issue Aug 2, 2017 · 5 comments · Fixed by #21446
Closed

--log-file parameter didn't create log file #15144

kenorb opened this issue Aug 2, 2017 · 5 comments · Fixed by #21446

Comments

@kenorb
Copy link

kenorb commented Aug 2, 2017

Description

I'm running Atom with --log-file parameter, but the log file isn't created. Tested on macOS Sierra.

Steps to Reproduce

$ /Applications/Atom.app/Contents/MacOS/Atom --help | grep log
  -l, --log-file             Log all output to file.  [string]
$ /Applications/Atom.app/Contents/MacOS/Atom -l ~/.atom/atom.log
[warn] kq_init: detected broken kqueue; not using.: Undefined error: 0
[warn] kq_init: detected broken kqueue; not using.: Undefined error: 0
[warn] kq_init: detected broken kqueue; not using.: Undefined error: 0
$ ls -la ~/.atom/
# The log file is not there.

Reproduces how often: All the time.

Versions

$ atom --version
Atom    : 1.18.0
Electron: 1.3.15
Chrome  : 52.0.2743.82
Node    : 6.5.0
$ apm --version
apm  1.18.1
npm  3.10.10
node 6.9.5 x64
python 2.7.10
git 2.13.3
@50Wliu 50Wliu added the bug label Aug 2, 2017
@rsese
Copy link
Contributor

rsese commented Aug 2, 2017

Reproduced with Atom 1.18.0 on macOS 10.12.6 and Atom 1.17.2 on Ubuntu 16.04.

@rsese rsese added the triaged label Aug 2, 2017
@lee-dohm
Copy link
Contributor

We should probably just remove this option.

@freeman42x
Copy link

It is still not working on current version of Atom.

@sadick254
Copy link
Contributor

--log-file option seem only to used in the context of running tests. This use case doesn't provide much benefits to a developer debugging.

buildTerminalReporter = (logFile, resolveWithExitCode) ->

Moving forward in my opinion we should deprecate the --log-file option since having a log file could be achieved by redirecting the output to a file.

# Unix based systems
atom --dev --foreground --wait > debug.log

# Windows 
atom --dev --foreground --wait >> debug.log

@sadick254
Copy link
Contributor

Since the log-file option has been there for a long time. I think it would be better to clarify its usage other than deprecating it.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants