Skip to content

on linux don't declare we installed to the system store#18

Merged
FiloSottile merged 1 commit intoFiloSottile:masterfrom
adamdecaf:clarify-linux-text
Jun 29, 2018
Merged

on linux don't declare we installed to the system store#18
FiloSottile merged 1 commit intoFiloSottile:masterfrom
adamdecaf:clarify-linux-text

Conversation

@adamdecaf
Copy link
Copy Markdown
Contributor

No description provided.

Comment thread truststore_linux.go Outdated

func (m *mkcert) installPlatform() {
log.Fatalf("-install is not yet supported on Linux 😣\nYou can manually install the root certificate at %q in the meantime.", filepath.Join(m.CAROOT, rootName))
log.Println(" -install is not yet fully supported on Linux")
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

What do you have against emojis 😁

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Whoops. Fixed. They don't show up the best in my emacs config. :-/

@adamdecaf adamdecaf force-pushed the clarify-linux-text branch from 2290192 to 4920cdc Compare June 29, 2018 19:44
@FiloSottile FiloSottile merged commit 073ee25 into FiloSottile:master Jun 29, 2018
@adamdecaf adamdecaf deleted the clarify-linux-text branch June 29, 2018 20:06
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.

2 participants