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

Add --log-driver=none to irssi run command. #426

Merged
merged 1 commit into from
Dec 15, 2015

Conversation

psftw
Copy link
Contributor

@psftw psftw commented Dec 15, 2015

No description provided.

--read-only -v $HOME/.irssi:/home/user/.irssi \
-v /etc/localtime:/etc/localtime \
irssi
```
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why'd you axe this section?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

what value does it add?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think the idea was to point out that ~/.irssi is the only place irssi tries to mutate by default.

@psftw psftw force-pushed the irssi-log-driver-none branch from 3d90d74 to 59c21ed Compare December 15, 2015 15:57
@psftw
Copy link
Contributor Author

psftw commented Dec 15, 2015

attempt to fix CI by actually using markdownfmt.sh

@psftw psftw force-pushed the irssi-log-driver-none branch from 59c21ed to 2844b06 Compare December 15, 2015 16:06
@psftw
Copy link
Contributor Author

psftw commented Dec 15, 2015

alright, one more time without README.md getting updated. maybe I'll learn what I'm doing one day.

@tianon
Copy link
Member

tianon commented Dec 15, 2015

LGTM

cc @jfrazelle

@jessfraz
Copy link
Contributor

nice LGTM

tianon added a commit that referenced this pull request Dec 15, 2015
Add --log-driver=none to irssi run command.
@tianon tianon merged commit ff504cf into docker-library:master Dec 15, 2015
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

3 participants