On 2007-05-17, Gary Johnson <[EMAIL PROTECTED]> wrote:
> On 2007-02-15, Frodak Baksik <[EMAIL PROTECTED]> wrote:
> > On 2/15/07, Chris Sutcliffe wrote:
> > > > > Also, is there anything I can do to help get the original
> > > > > patch accepted?
> > > >
> > > > Ask a few people to try it out and report their results here.
> > >
> > > I'll give it a shot. Is there somewhere I can grab the patch from, or
> > > should I go through the list archives to find it?
>
> > Here are all the changes in a single patch.
> > I'm also posting this to the cygwin-apps mailing list, so if anyone
> > over there could try it out would be nice.
>
> I just applied this patch to the 7.1 source. The patch to
> proto/os_mswin.pro failed, but it was easy to fix manually. The
> problem was that the extern prefix has been removed from the 7.1
> declarations. Hunk #2 of the patch to os_win32.c failed, apparently
> because that change was already made to the 7.1 source, so I left
> that file with only hunk #1 applied.
It looks like I goofed when applying the patch that way. I just
applied the patch to another system, but edited the patch first to
get rid of the failures instead of trying to edit the files
afterwards. All I had to do to the proto/winclip.pro section was to
remove "extern " from the start of all the lines. While editing the
os_win32.c section, I discovered that 'patch' was missing the first
hunk in the target and applying the first hunk of the patch to the
second hunk in the target, causing the application of the second
hunk of the patch to fail. This was cause by the reformatting of a
comment from 7.0 to 7.1. I changed the comment in the patch to
match the comment in the 7.1 source and the entire patch applied
successfully.
Regards,
Gary
--
Gary Johnson | Agilent Technologies
[EMAIL PROTECTED] | Mobile Broadband Division
| Spokane, Washington, USA
