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
MIME-Version: 1.0
Content-Type: text/html; charset="UTF-8"
X-Mailer: MIME-tools 5.427 (Entity 5.427)
Content-Disposition: inline
Content-Transfer-Encoding: binary
X-RT-Original-Encoding: utf-8
Content-Length: 503
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
MIME-Version: 1.0
X-Mailer: MIME-tools 5.427 (Entity 5.427)
Content-Disposition: inline
Content-Type: text/plain; charset="UTF-8"
Message-ID: <rt-3.8.HEAD-12689-1267583823-1200.55158-0-0 [...] rt.cpan.org>
Content-Transfer-Encoding: binary
X-RT-Original-Encoding: utf-8
Content-Length: 406
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.
MIME-Version: 1.0
X-Spam-Status: No, hits=-0.0 required=8.0 tests=SPF_HELO_PASS
In-Reply-To: <rt-3.8.HEAD-12689-1267583823-673.55158-5-0 [...] rt.cpan.org>
References: <RT-Ticket-55158 [...] rt.cpan.org> <rt-3.8.HEAD-12689-1267583823-673.55158-5-0 [...] rt.cpan.org>
Message-ID: <4B8DDA9B.7010401 [...] w3works.com>
Content-Type: text/plain; charset=UTF-8; format=flowed
X-RT-Original-Encoding: utf-8
Received: from la.mx.develooper.com (x1.develooper.com [207.171.7.70]) by diesel.bestpractical.com (Postfix) with SMTP id 217934D8002 for <bug-Crypt-IDEA [...] rt.cpan.org>; Tue, 2 Mar 2010 22:43:13 -0500 (EST)
Received: (qmail 11280 invoked by uid 103); 3 Mar 2010 03:43:13 -0000
Received: from x16.dev (10.0.100.26) by x1.dev with QMQP; 3 Mar 2010 03:43:13 -0000
Received: from mail.w3works.com (HELO mail.w3works.com) (64.211.196.10) by 16.mx.develooper.com (qpsmtpd/0.80) with ESMTP; Tue, 02 Mar 2010 19:42:37 -0800
Received: (qmail 10276 invoked by uid 511); 3 Mar 2010 03:42:25 -0000
Received: from 64.211.196.127 by mail.w3works.net (envelope-from <dparis [...] w3works.com>, uid 503) with qmail-scanner-1.24 (clamdscan: 0.91.2/4671. spamassassin: 3.0.2. Clear:RC:1(64.211.196.127):. Processed in 0.111374 secs); 03 Mar 2010 03:42:25 -0000
Received: from zuul.w3works.com (HELO ?192.168.1.219?) (dave [...] w3works.com [...] 64.211.196.127) by mail.w3works.com with SMTP; 3 Mar 2010 03:42:24 -0000
Delivered-To: cpan-bug+Crypt-IDEA [...] diesel.bestpractical.com
Subject: Re: [rt.cpan.org #55158] Win32 patch - support for mingw-w64 gcc toolchain
User-Agent: Thunderbird 2.0.0.23 (Windows/20090812)
Return-Path: <dparis [...] w3works.com>
X-Spam-Check-BY: 16.mx.develooper.com
X-Original-To: bug-Crypt-IDEA [...] rt.cpan.org
Date: Tue, 02 Mar 2010 22:42:19 -0500
X-Spam-Level: *
To: bug-Crypt-IDEA [...] rt.cpan.org
Content-Transfer-Encoding: 7bit
From: Dave Paris <dparis [...] w3works.com>
RT-Message-ID: <rt-3.8.HEAD-12689-1267587804-1459.55158-0-0 [...] rt.cpan.org>
Content-Length: 1335
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.
MIME-Version: 1.0
In-Reply-To: <rt-3.8.HEAD-12689-1267587804-1459.55158-0-0 [...] rt.cpan.org>
X-Mailer: MIME-tools 5.427 (Entity 5.427)
Content-Disposition: inline
References: <RT-Ticket-55158 [...] rt.cpan.org> <rt-3.8.HEAD-12689-1267583823-673.55158-5-0 [...] rt.cpan.org> <4B8DDA9B.7010401 [...] w3works.com> <rt-3.8.HEAD-12689-1267587804-1459.55158-0-0 [...] rt.cpan.org>
Content-Type: text/html; charset="UTF-8"
Message-ID: <rt-3.8.HEAD-12687-1267608734-1037.55158-0-0 [...] rt.cpan.org>
Content-Transfer-Encoding: binary
X-RT-Original-Encoding: utf-8
Content-Length: 522
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
MIME-Version: 1.0
In-Reply-To: <rt-3.8.HEAD-12689-1267587804-1459.55158-0-0 [...] rt.cpan.org>
X-Mailer: MIME-tools 5.427 (Entity 5.427)
Content-Disposition: inline
References: <RT-Ticket-55158 [...] rt.cpan.org> <rt-3.8.HEAD-12689-1267583823-673.55158-5-0 [...] rt.cpan.org> <4B8DDA9B.7010401 [...] w3works.com> <rt-3.8.HEAD-12689-1267587804-1459.55158-0-0 [...] rt.cpan.org>
Content-Type: text/plain; charset="UTF-8"
Message-ID: <rt-3.8.HEAD-12682-1267693147-898.55158-0-0 [...] rt.cpan.org>
Content-Transfer-Encoding: binary
X-RT-Original-Encoding: utf-8
Content-Length: 299
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
MIME-Version: 1.0
In-Reply-To: <rt-3.8.HEAD-12689-1267587804-1459.55158-0-0 [...] rt.cpan.org>
X-Mailer: MIME-tools 5.427 (Entity 5.427)
Content-Disposition: inline
References: <RT-Ticket-55158 [...] rt.cpan.org> <rt-3.8.HEAD-12689-1267583823-673.55158-5-0 [...] rt.cpan.org> <4B8DDA9B.7010401 [...] w3works.com> <rt-3.8.HEAD-12689-1267587804-1459.55158-0-0 [...] rt.cpan.org>
Content-Type: text/html; charset="UTF-8"
Message-ID: <rt-3.8.HEAD-2372-1278958367-26.55158-0-0 [...] rt.cpan.org>
Content-Transfer-Encoding: binary
X-RT-Original-Encoding: utf-8
Content-Length: 160
Hi Dave,

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

--
kmx

MIME-Version: 1.0
X-Mailer: MIME-tools 5.427 (Entity 5.427)
Content-Disposition: inline
Content-Type: text/plain; charset="UTF-8"
Message-ID: <rt-3.8.HEAD-18810-1308142709-142.55158-0-0 [...] rt.cpan.org>
Content-Transfer-Encoding: binary
From: vvm [...] tut.by
X-RT-Original-Encoding: utf-8
Content-Length: 232
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 . . .
MIME-Version: 1.0
In-Reply-To: <rt-3.8.HEAD-18810-1308142709-142.55158-0-0 [...] rt.cpan.org>
X-Mailer: MIME-tools 5.427 (Entity 5.427)
Content-Disposition: inline
References: <rt-3.8.HEAD-18810-1308142709-142.55158-0-0 [...] rt.cpan.org>
Content-Type: text/html; charset="UTF-8"
Message-ID: <rt-3.8.HEAD-2601-1318918606-261.55158-0-0 [...] rt.cpan.org>
Content-Transfer-Encoding: binary
X-RT-Original-Encoding: utf-8
Content-Length: 267
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
From mr.mused [...] w3works.com Tue Oct 18 06: 29:24 2011
MIME-Version: 1.0
X-Spam-Status: No, score=-6.235 tagged_above=-99.9 required=10 tests=[BAYES_00=-1.9, RCVD_IN_DNSWL_HI=-5, SPF_SOFTFAIL=0.665] autolearn=ham
In-Reply-To: <rt-3.8.HEAD-2601-1318918606-731.55158-5-0 [...] rt.cpan.org>
X-Spam-Flag: NO
References: <RT-Ticket-55158 [...] rt.cpan.org> <rt-3.8.HEAD-18810-1308142709-142.55158-5-0 [...] rt.cpan.org> <rt-3.8.HEAD-2601-1318918606-731.55158-5-0 [...] rt.cpan.org>
X-Virus-Scanned: Debian amavisd-new at bestpractical.com
Message-ID: <CAAF0DAaGTeFx7ddZMtipgmuo2zJzt_Wh1bpOJHqHtUNhe2jfhw [...] mail.gmail.com>
Content-Type: text/plain; charset="utf-8"
X-RT-Original-Encoding: utf-8
X-Spam-Score: -6.235
Received: from localhost (localhost [127.0.0.1]) by hipster.bestpractical.com (Postfix) with ESMTP id 93746240C32 for <cpan-bug+Crypt-IDEA [...] hipster.bestpractical.com>; Tue, 18 Oct 2011 06:29:24 -0400 (EDT)
Received: from hipster.bestpractical.com ([127.0.0.1]) by localhost (hipster.bestpractical.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id 1n0Ifiq4zuRD for <cpan-bug+Crypt-IDEA [...] hipster.bestpractical.com>; Tue, 18 Oct 2011 06:29:22 -0400 (EDT)
Received: from la.mx.develooper.com (x1.develooper.com [207.171.7.70]) by hipster.bestpractical.com (Postfix) with SMTP id 96A29240C30 for <bug-Crypt-IDEA [...] rt.cpan.org>; Tue, 18 Oct 2011 06:29:22 -0400 (EDT)
Received: (qmail 5716 invoked by uid 103); 18 Oct 2011 10:29:21 -0000
Received: from x16.dev (10.0.100.26) by x1.dev with QMQP; 18 Oct 2011 10:29:21 -0000
Received: from mail-gy0-f178.google.com (HELO mail-gy0-f178.google.com) (209.85.160.178) by 16.mx.develooper.com (qpsmtpd/0.80/v0.80-19-gf52d165) with ESMTP; Tue, 18 Oct 2011 03:29:19 -0700
Received: by gyh4 with SMTP id 4so410187gyh.9 for <bug-Crypt-IDEA [...] rt.cpan.org>; Tue, 18 Oct 2011 03:29:16 -0700 (PDT)
Received: by 10.236.116.98 with SMTP id f62mr1990827yhh.48.1318933756313; Tue, 18 Oct 2011 03:29:16 -0700 (PDT)
Received: by 10.236.208.2 with HTTP; Tue, 18 Oct 2011 03:29:16 -0700 (PDT)
Delivered-To: cpan-bug+Crypt-IDEA [...] hipster.bestpractical.com
Subject: Re: [rt.cpan.org #55158] Win32 patch - support for mingw-w64 gcc toolchain
Return-Path: <mr.mused [...] w3works.com>
X-Spam-Check-BY: 16.mx.develooper.com
X-Original-To: cpan-bug+Crypt-IDEA [...] hipster.bestpractical.com
X-RT-Mail-Extension: crypt-idea
X-Google-Sender-Auth: D_qb5_dPILJhc0zX2iqBZmNjJ-I
Sender: mr.mused [...] w3works.com
Date: Tue, 18 Oct 2011 06:29:16 -0400
X-Spam-Level:
X-Originating-Ip: [69.205.123.67]
To: bug-Crypt-IDEA [...] rt.cpan.org
Content-Transfer-Encoding: quoted-printable
From: Dave Paris <dparis [...] w3works.com>
RT-Message-ID: <rt-3.8.HEAD-2602-1318933765-1150.55158-0-0 [...] rt.cpan.org>
Content-Length: 477
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 > >
MIME-Version: 1.0
X-Mailer: MIME-tools 5.504 (Entity 5.504)
Content-Disposition: inline
X-RT-Interface: Web
Content-Type: text/plain; charset="utf-8"
Message-ID: <rt-4.0.13-16564-1374595041-1712.55158-0-0 [...] rt.cpan.org>
Content-Transfer-Encoding: binary
X-RT-Original-Encoding: utf-8
X-RT-Encrypt: 0
X-RT-Sign: 0
Content-Length: 85
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.