Skip to content

Closes #13871#20035

Closed
IDisposable wants to merge 1 commit into
rust-lang:masterfrom
IDisposable:patch-1
Closed

Closes #13871#20035
IDisposable wants to merge 1 commit into
rust-lang:masterfrom
IDisposable:patch-1

Conversation

@IDisposable

Copy link
Copy Markdown

Removed quotes. Closes #13871

Removed copyrighted quotes. Closes rust-lang#13871
@rust-highfive

Copy link
Copy Markdown
Contributor

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @alexcrichton (or someone else) soon.

If any changes to this PR are deemed necessary, please add them as extra commits. This ensures that the reviewer can see what has changed since they last reviewed the code. The way Github handles out-of-date commits, this should also make it reasonably obvious what issues have or haven't been addressed. Large or tricky changes may require several passes of review and changes.

Please see CONTRIBUTING.md for more information.

@adrientetar

Copy link
Copy Markdown
Contributor

Not copyrighted...

@MattWindsor91

Copy link
Copy Markdown
Contributor

Not copyrighted, but seems a bit bourgeois to me. Call me a misguided optimisation weenie, but I'm not sure I'm fond of ~2KB of every standard-library Rust program I make being dedicated to an easter egg 😀

@MattWindsor91

Copy link
Copy Markdown
Contributor

Also, I remember being very confused at seeing this when a build of rustc failed with an abort, so I doubt it does users who aren't aware of its existence much good. It kinda seems almost flippant, actually.

@tomjakubowski

Copy link
Copy Markdown
Contributor

Even if the quotes were copyrighted (the one from Majora's Mask might be), this probably falls under fair use.

@IDisposable

Copy link
Copy Markdown
Author

Okay, so let's attack this from the other view-point... what possible value is added?

@sinistersnare

Copy link
Copy Markdown
Contributor

This is from the runtime, so in applications where the extra 2kb is necessary (embedded), this will not be put into the executable. And where the extra 2kb is not needed, then its a fun easter egg.

@MattWindsor91

Copy link
Copy Markdown
Contributor

@sinistersnare Depends entirely on the mood of whoever gets Lovecraft quotes dumped at them when an abort() happens.

@emberian

emberian commented Jan 8, 2015

Copy link
Copy Markdown
Contributor

@IDisposable can you change the commit message to remove mention of "copyrighted"? That's not the issue here (and I want to try and get this merged for the alpha...)

@liigo

liigo commented Jan 8, 2015

Copy link
Copy Markdown
Contributor

+1

@IDisposable

Copy link
Copy Markdown
Author

Done

@brson

brson commented Jan 11, 2015

Copy link
Copy Markdown
Contributor

@IDisposable the commit message still incorrectly claims this is a copyright issue.

@brson

brson commented Jan 11, 2015

Copy link
Copy Markdown
Contributor

I went ahead and did another one with a different commit message #20944

Thanks.

@brson brson closed this Jan 11, 2015
lnicola pushed a commit to lnicola/rust that referenced this pull request Jun 23, 2025
Add --color=always to test explorer command
lnicola pushed a commit to lnicola/rust that referenced this pull request Jun 30, 2025
Add --color=always to test explorer command
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.

"hello world" contains Lovecraft quotes

10 participants