https://bugzilla.redhat.com/show_bug.cgi?id=2335013
--- Comment #8 from Gwyn Ciesla gwync@protonmail.com --- You're very welcome!
Well, both methods work, but I tend towards patching for two reasons.
One, if the change is no longer needed, you won't have a spurious sed command lingering forever. Two, if the change needs adjusting, the build will break, and you'll know and can refactor the patch to make sure it's still appropriate, etc.
The only time I really do changes to files in the spec is if it's a change I know will always be needed, especially if a patch would require constant updating due to a date string or something.
The above is for file changes. For directories that need removing, do that in the spec.
Make sense?