Re: [Rails-devel] delurking with a quick ReportWindow fix

[brett l. <wak...@gm...>]

On Sat, Nov 22, 2008 at 9:22 AM, Erik Vos <eri...@hc...> wrote:
>> The most recent information that I had is that "unified diff"
>> was still the preferred format for beginning contributions.
>> If that should be changed, just let me know.
>>
>> I don't see how to save the contents of an Eclipse "source
>> compare" view in any meaningful way.  I must add that
>> whenever a non-coding task can be performed in Cygwin/Emacs
>> instead of Eclipse, I almost always do so. :-)
>
> Eclipse can create unified diffs via (right-click)Team/Create Patch, see
> example below.
> The paths in this patch do have date/time, so that can't have been the
> problem after all.
> And this patch can be applied via Team/Apply Patch.
>
> Perhaps it's more important that there is no "orig" in the paths; indeed I
> also had to remove that stuff and make both paths equal to have Eclipse
> accept the previous patches.
>

This is likely the issue.  The patch command has a flag to specify the
number of directories to strip off the patch's location.

> N.B. The below patch only removes some dead code. I have committed it.
>
> Erik
>

---Brett