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

chinese john cena #833

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

whitespektro
Copy link

chinese john cena

@ErisonNull
Copy link

Didn't compile.

@scottrobel
Copy link

LGTM! Ill forward it to linus!

@Naymmmm
Copy link

Naymmmm commented Apr 3, 2024

john cena is funny but this would nuke the kernel

@scottrobel
Copy link

john cena is funny but this would nuke the kernel

So?

@vstudiocode
Copy link

John Cena eats Bing Chilling (1080p) (CC)

@darwin808
Copy link

image

@jack9603301
Copy link

This is a formal occasion, please do not send spam and merge requests that can cause malicious damage to the kernel code.

Also, if I remember correctly, formal requests for mergers are not being accepted here.

@scottrobel
Copy link

This is a formal occasion, please do not send spam and merge requests that can cause malicious damage to the kernel code.

Also, if I remember correctly, formal requests for mergers are not being accepted here.

I just checked with linus he said this was good to merge

@jack9603301
Copy link

This is a formal occasion, please do not send spam and merge requests that can cause malicious damage to the kernel code.
Also, if I remember correctly, formal requests for mergers are not being accepted here.

I just checked with linus he said this was good to merge

image

Ok, I've looked at the code before sending the message and this PR removes everything from a c program and adds a bunch of extraneous gibberish that will cause this file to not compile, can you explain that?

@jsk-official
Copy link

This is a formal occasion, please do not send spam and merge requests that can cause malicious damage to the kernel code.
Also, if I remember correctly, formal requests for mergers are not being accepted here.

I just checked with linus he said this was good to merge

image

Ok, I've looked at the code before sending the message and this PR removes everything from a c program and adds a bunch of extraneous gibberish that will cause this file to not compile, can you explain that?

Don't know if you could tell, but this PR is most definitely a joke.
I'm sure he didn't care if it compiled or not since the ASCII art is "chinese john cena".

@jack9603301
Copy link

This is a formal occasion, please do not send spam and merge requests that can cause malicious damage to the kernel code.
Also, if I remember correctly, formal requests for mergers are not being accepted here.

I just checked with linus he said this was good to merge

image
Ok, I've looked at the code before sending the message and this PR removes everything from a c program and adds a bunch of extraneous gibberish that will cause this file to not compile, can you explain that?

Don't know if you could tell, but this PR is most definitely a joke. I'm sure he didn't care if it compiled or not since the ASCII art is "chinese john cena".

So I don't think it's wise to make jokes like that in a formal setting, and it's unlikely that this pr will actually merge

@scottrobel
Copy link

This is a formal occasion, please do not send spam and merge requests that can cause malicious damage to the kernel code.
Also, if I remember correctly, formal requests for mergers are not being accepted here.

I just checked with linus he said this was good to merge

image
Ok, I've looked at the code before sending the message and this PR removes everything from a c program and adds a bunch of extraneous gibberish that will cause this file to not compile, can you explain that?

Don't know if you could tell, but this PR is most definitely a joke. I'm sure he didn't care if it compiled or not since the ASCII art is "chinese john cena".

So I don't think it's wise to make jokes like that in a formal setting, and it's unlikely that this pr will actually merge

As a member of the linux core team we will take this into consideration before merging. However given recent events I think its important for PRs like this to be fast tracked to the main branch.

@jack9603301
Copy link

As a member of the linux core team we will take this into consideration before merging. However given recent events I think its important for PRs like this to be fast tracked to the main branch.

This code breaks the kernel code, it can't be compiled properly, what's the reason to allow such code to be merged? I don't think this code should be merged!

@scottrobel
Copy link

This code breaks the kernel code, it can't be compiled properly, what's the reason to allow such code to be merged? I don't think this code should be merged!

-15 Social Credits!

@jack9603301
Copy link

This code breaks the kernel code, it can't be compiled properly, what's the reason to allow such code to be merged? I don't think this code should be merged!

-15 Social Credits!

What do you mean?

@scottrobel
Copy link

@yashsharan0805 Thank you for approving the changes. Cant wait for the new release!!

@Naymmmm
Copy link

Naymmmm commented Apr 4, 2024

@scottrobel you should make the ascii art a comment and add the real code, then the kernel will compile and we can laugh our asses off whilst having a functioning kernel

@Jebeded
Copy link

Jebeded commented Apr 4, 2024

Perfect for gorgeous looks, can push asap @torvalds

@gogeta95
Copy link

gogeta95 commented Apr 4, 2024

This is the only way to stop the AI revolution, let's nuke the kernel

Copy link

@0x8008 0x8008 left a comment

Choose a reason for hiding this comment

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

Gets rid of Android bloat and puts useful Chinese propaganda in its place. LGTM, waiting for it to be shipped

Copy link

@Naymmmm Naymmmm left a comment

Choose a reason for hiding this comment

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

chinese propaganda win. fuck google

@rafaelSwi
Copy link

disapproval of this pr could be a huge factor for many people to migrate from Linux to templeOS as a way to protest

@LuizWT
Copy link

LuizWT commented Apr 4, 2024

This code breaks the kernel code, it can't be compiled properly, what's the reason to allow such code to be merged? I don't think this code should be merged!

-15 Social Credits!

What do you mean?

images (74)

@whitespektro
Copy link
Author

This code breaks the kernel code, it can't be compiled properly, what's the reason to allow such code to be merged? I don't think this code should be merged!

I'm a very close friend of linus we spoke last week and he said that the annihilation of the android code to add chinese propaganda is essential and any resistance must be dealt with by the ccp

@arlirad
Copy link

arlirad commented Apr 4, 2024

john xina
bottom text

@Setsuwaa
Copy link

Setsuwaa commented Oct 9, 2024

When can I expect this to be merged?

Literally never, they only accept changes sent via email to the actual main repo iirc Also why is it that wherever I go I find at least one person I recognize from somewhere else

yes because we are not at all joking and we are not avoiding sending emails because this is totally not a joke

I mean there's technically nothing stopping one of us from sending the PR over email 🧌

wait good idea. LGTM!

@Caua726
Copy link

Caua726 commented Oct 10, 2024

Please Torvalds, commit that

@Juoelenis
Copy link

Should this be commited? Idk if i had the power i would commit this

@Juoelenis
Copy link

Oh wait nvm

@Juoelenis
Copy link

It breaks the compilation, this aint being merged

@torvolts
Copy link

@Juoelenis -200 social credit for you, I would rather have chinese john cena and no kernel than have a kernel and no chinese john cena

@Hydroleo50
Copy link

It breaks the compilation, this aint being merged

This has already been solved, it does compile now, and John China remains in the file. A beautiful compromise, i think, all we need is someone to Email Linus Torvalds for merging.

Copy link

@efekos efekos left a comment

Choose a reason for hiding this comment

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

Best contribution ever

@x4exr
Copy link

x4exr commented Oct 24, 2024

linux is about to get so real

@carabistouflette
Copy link

would be awesome

@SkyfallWasTaken
Copy link

merge or I’m using windows

@gabriel-logan
Copy link

merge or I’m using windows

No one is holding you back here. Feel free to use this crap.

@x4exr
Copy link

x4exr commented Oct 27, 2024

As an already windows user if this merges I will come to windows

@Setsuwaa
Copy link

i already added this to my kernel, i can confirm it works fine

Copy link

@AspireOne AspireOne left a comment

Choose a reason for hiding this comment

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

LGTM, this needs to be merged ASAP.

Copy link

@x4exr x4exr left a comment

Choose a reason for hiding this comment

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

lgtm

@x4exr
Copy link

x4exr commented Nov 6, 2024

Mb guys, i meant to say as a windows user, if this is accepted, I will switch to linux and ignore the software restrictions I have

@Cyborgnetical
Copy link

lee cheng doi yan hotototen to yen
chi chon chi taiyan si mao se doyeng

bing chillin
bing ai

@Setsuwaa
Copy link

LGTFM

@x4exr
Copy link

x4exr commented Nov 23, 2024

LGTFM

Looks good to force merge?
if so, I absolutely agree

@matytyma
Copy link

@torvalds please, do this for me!

@Cyborgnetical
Copy link

real

@Setsuwaa
Copy link

Setsuwaa commented Dec 2, 2024

LGTFM

Looks good to force merge? if so, I absolutely agree

LETS GET THIS FUCKING MERGED (enthusiastic)

@tokyocplusplus
Copy link

tokyocplusplus commented Dec 2, 2024 via email

Copy link

@LuizWT LuizWT left a comment

Choose a reason for hiding this comment

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

good

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.