It *seems* that empty patches submitted (erroneously) on the issue queue are uploaded and tested like valid ones, and obviously accepted.
While technically correct since empty patches can never fail, this seems like a waste of resources, and should probably result in an error, since an empty patch does nothing.
Comments
Comment #1
boombatower commentedPerhaps we should look into this..fairly trivial to change, but also probably doesn't occur to often. My only concern is that the patch command accepts an empty patch and thus invalidating the patch only occurs if the patch command throws errors. Add an empty check seems odd.
Perhaps get some opinions on this.
Comment #2
boombatower commentedUnless this becomes a prevalent issue I say we leave it alone.