How do you remove unneeded patch files?
Stephen Montgomery-Smith
stephen at math.missouri.edu
Wed Jul 25 05:43:01 UTC 2007
Erwin Lansing wrote:
> On Tue, Jul 24, 2007 at 02:41:04PM -0500, Paul Schmehl wrote:
>> I'm working on a port upgrade. The existing port has a number of patch
>> files in FILESDIR that are no longer needed. (The changes they made have
>> been incorporated into the distro.) What's the appropriate way to submit
>> those changes? Normally, a send-pr submission will contain the patches for
>> existing port files. There is no patch for these. They need to be removed.
>>
>> I could do this: diff -Naur files/original_patch_file files/empty_file, but
>> I don't know if that removes the file or simply replaces it with an empty
>> one.
>>
> The result of using that patch would be an empty file. This is also
> alright to submit it that way, but as a courtesy to the committer (and
> also to make sure he doesn't forget) it would be good to mention in the
> PR that file/xyz and files/abc should be removed completely.
You really should be unambiguous and clear in explaining that the files
are to be deleted. I have had a couple of experiences where ports I
maintain did not have the files deleted, and I had to remind the committer.
Stephen
More information about the freebsd-ports
mailing list