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

[SUGGESTION] Release Binary Should Log To STDERR instead of to install directory #59

Closed
lilleswing opened this issue Aug 17, 2022 · 1 comment

Comments

@lilleswing
Copy link
Contributor

In https://github.com/rdk/p2rank/blob/develop/distro/prank the program logs to a logfile, that is default inside the installation directory. I believe this is the run-script distributed in the release tarball.

When installing this on HPC systems often the installation directory is not writable by the users running the software leading to errors. In the case that it is writable by users then there can be thrashing and race conditions in the logfile if multiple jobs are writing to it at once.

It might be better to distribute the version that logs to stdout instead of a logfile in the install directory.

@lilleswing lilleswing changed the title Release Log To STDOUT instead of to install directory Release Binary Should Log To STDERR instead of to install directory Aug 17, 2022
@lilleswing lilleswing changed the title Release Binary Should Log To STDERR instead of to install directory [SUGGESTION] Release Binary Should Log To STDERR instead of to install directory Aug 17, 2022
@rdk
Copy link
Owner

rdk commented Aug 23, 2022

Thanks for the suggestion. I happen to agree and wanted to change it for some time.

It will be changed in the next release.

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

No branches or pull requests

2 participants