Conversation

It's quite sobering realising a lot of shit code you see in mm exists because we just took it without _ANY_ review tags.

Joke.

3
0
0

IOW - no sign off. Review may have happened up to a point, but it stopped.

And that was taken as a 'merge this' signal.

Unreal.

1
0
0

@ljs How is that? Shouldn't some maintainer sign it off and send it to Linus on a PR?

1
0
0

vitaut 🀍❀️🀍 πŸ‡ΊπŸ‡¦

@ljs one weird trick to become a kernel developer

0
0
0

@ffmancera not if your maintainer is just merging anything.

1
0
0

@ffmancera in mm we have real maintainers and the guy who is taking everything and sending to Linus, they're not the same.

2
0
0

@ffmancera am working on trying to change this :)

0
0
0
@ljs The most absurd thing I've heared in that regard was: sometimes I "threaten" that I'll merge something if people don't review it.
1
0
3

@kdave sigh. But no tags required.

Review is 'people replied until they stopped replying at which point I just take it'.

Yeah things have got to change...

1
0
0

@kdave btw you can REALLY TELL who was relying on this prior to the great 'mm now actually has maintainers' change.

And there's tantrums...

0
0
1