On 2015-12-01 15:05, Marcin Krol wrote:
Anyway - glen is right, rpm should reject that...

Perhaps we should go debian-like route and accept only patches that
apply cleanly? Fuzziness can cause code to be inserted/deleted/modified
in wrong places which can cause compile time errors, segfaults, broken
configs etc. I encountered such issues many times and most of our
patches apply with fuzzines or offsets.

But this case was not about fuzzines or invalid patch. Patches are allowed to contain trailing data – they might be part of an e-mail message. That is why blues' patch was accepted. The line ignored was technically outside of the patch.

Jacek
_______________________________________________
pld-devel-en mailing list
pld-devel-en@lists.pld-linux.org
http://lists.pld-linux.org/mailman/listinfo/pld-devel-en

Reply via email to