Skip Menu |
 

This queue is for tickets about the Crypt-IDEA CPAN distribution.

Report information
The Basics
Id: 55158
Status: resolved
Priority: 0/
Queue: Crypt-IDEA

People
Owner: DPARIS [...] cpan.org
Requestors: kmx [...] cpan.org
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: (no value)



Subject: Win32 patch - support for mingw-w64 gcc toolchain
Hi,

could you please confider the following path to _idea.c

  #ifdef WIN32
+ #include <windows.h>
  #include <wininet.h>
  # else
  #include <netinet/in.h>
  # endif

The poposed patch works nice with:
- 32bit gcc3 by mingw.org
- 32bit gcc4 by mingw-w64.sf.net
- 64bit gcc4 by mingw-w64.sf.net

Thanks in advance for any feedback.

--
kmx
Download (untitled) / with headers
text/plain 406b
I will specify that while the current module works with the mingw32-gcc3 toolchain that Strawberry Perl is currently using without patching, we plan to move to the two aforementioned gcc4 toolchain with the 5.12.x.x versions, and this module is used as part of the "crypto toolchain" that Module::Signature relies on. So this is a blocker for those versions. Please release a fix with the included patch.
Subject: Re: [rt.cpan.org #55158] Win32 patch - support for mingw-w64 gcc toolchain
Date: Tue, 02 Mar 2010 22:42:19 -0500
To: bug-Crypt-IDEA [...] rt.cpan.org
From: Dave Paris <dparis [...] w3works.com>
Download (untitled) / with headers
text/plain 1.3k
Mr. Jewell, First, I'm quite capable of deducing the conclusion you state below. Second, the ticket was opened less than three hours prior to your comment. I would request an apology for your final tone, implying I have no plans on issuing a subsequent, patched version and in the future, attempt to restrain yourself for 24-48 hours after the submission of a ticket. This gives the module maintainer an ample, if minimal, amount of time to adequately patch, test, and update. Or at least reply to the ticket. If you find this to be offensive, it is merely an in-kind reply. Followups will not be responded to. I will update CPAN with a tested, patched module once a full regression test is passed on all main (Win32, *nix, OS X, Solaris, etc) platforms. Regards, -dsp Curtis Jewell via RT wrote: Show quoted text
> Queue: Crypt-IDEA > Ticket <URL: https://rt.cpan.org/Ticket/Display.html?id=55158 > > > I will specify that while the current module works with the mingw32-gcc3 > toolchain that Strawberry Perl is currently using without patching, we > plan to move to the two aforementioned gcc4 toolchain with the 5.12.x.x > versions, and this module is used as part of the "crypto toolchain" that > Module::Signature relies on. > > So this is a blocker for those versions. Please release a fix with the > included patch.
Hi Dave,

Show quoted text
> I will update CPAN with a tested, patched module once a full
> regression test is passed on all main (Win32,  *nix, OS X, Solaris,
> etc) platforms.

I am not sure if you have noticed that my proposed 1-line patch is all inside #ifdef _WIN32 ... #endif so perhaps it has not such a significant impact on *nix, OS X, Solaris and other platforms.

Anyway, thaks for your prompt feedback (even though slightly heated).

--
kmx
Download (untitled) / with headers
text/plain 299b
I'm sorry about the tone. I know I'm not the most tactful person out there. I meant to be informative as to why we ask without adding a perjorative "jump on your case tone". Thanks for both your work, and for letting me know that I did tone that badly. I'll be more careful in the future. --Curtis
Hi Dave,

can I help somehow with getting the one-line-patch from my first post into Crypt-IDEA release?

--
kmx

From: vvm [...] tut.by
Download (untitled) / with headers
text/plain 232b
On Tue Mar 02 18:43:34 2010, KMX wrote: Show quoted text
> could you please confider the following path to _idea.c > > #ifdef WIN32 > + #include <windows.h> > #include <wininet.h> > # else > #include <netinet/in.h> > # endif
Patch work Ok . . .
Hi Dave,

can I help somehow with getting the one-line-patch from my first post into Crypt-IDEA release?

If you give me co-maint I can handle Crypt-IDEA release.

Please give me any kind of feedback.

--
kmx
Subject: Re: [rt.cpan.org #55158] Win32 patch - support for mingw-w64 gcc toolchain
Date: Tue, 18 Oct 2011 06:29:16 -0400
To: bug-Crypt-IDEA [...] rt.cpan.org
From: Dave Paris <dparis [...] w3works.com>
Download (untitled) / with headers
text/plain 477b
I'll patch it later today and send it to you for testing. Thanks! On Tue, Oct 18, 2011 at 2:16 AM, kmx via RT <bug-Crypt-IDEA@rt.cpan.org> wrote: Show quoted text
>       Queue: Crypt-IDEA >  Ticket <URL: https://rt.cpan.org/Ticket/Display.html?id=55158 > > > Hi Dave, > > can I help somehow with getting the one-line-patch from my first post into > Crypt-IDEA release? > > If you give me co-maint I can handle Crypt-IDEA release. > > Please give me any kind of feedback. > > -- > kmx > >
This ticket has been resolved in release 1.09 which has been uploaded to PAUSE. -dsp


This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.