Skip Menu |

This queue is for tickets about the Math-Int128 CPAN distribution.

Report information
The Basics
Id: 92174
Status: resolved
Priority: 0/
Queue: Math-Int128
People
Owner: Nobody in particular
Requestors: abraxxa [...] cpan.org
Cc:
AdminCc:
Bug Information
Severity: (no value)
Broken in: 0.13
Fixed in: (no value)

History Show all quoted text
  Wed Jan 15 12:31:09 2014 abraxxa [...] cpan.org - Ticket created
Subject: Makefile.PL doesn't detect Debian 6 GCC 4.4.5 as supported
# cc --version cc (Debian 4.4.5-8) 4.4.5 Copyright (C) 2010 Free Software Foundation, Inc. This is free software; see the source for copying conditions. There is NO warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. Maybe use the detection code from Math::Int64 instead of whitelisting compiler/version combos.
  Wed Jan 15 12:36:21 2014 autarch [...] urth.org - Correspondence added
Subject: Re: [rt.cpan.org #92174] Makefile.PL doesn't detect Debian 6 GCC 4.4.5 as supported
Date: Wed, 15 Jan 2014 11:36:12 -0600 (CST)
To: Alexander Hartmaier via RT <bug-Math-Int128 [...] rt.cpan.org>
From: Dave Rolsky <autarch [...] urth.org>
On Wed, 15 Jan 2014, Alexander Hartmaier via RT wrote: Show quoted text
> Wed Jan 15 12:31:09 2014: Request 92174 was acted upon. > Transaction: Ticket created by ABRAXXA > Queue: Math-Int128 > Subject: Makefile.PL doesn't detect Debian 6 GCC 4.4.5 as supported > Broken in: 0.13 > Severity: (no value) > Owner: Nobody > Requestors: abraxxa@cpan.org > Status: new > Ticket <URL: https://rt.cpan.org/Ticket/Display.html?id=92174 > > > > # cc --version > cc (Debian 4.4.5-8) 4.4.5 > Copyright (C) 2010 Free Software Foundation, Inc. > This is free software; see the source for copying conditions. There is NO > warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. > > Maybe use the detection code from Math::Int64 instead of whitelisting compiler/version combos.
I'm planning to move both of these modules to use Config::AutoConf. I think it's just a better practice to probe for feature capabilities rather than versions. If you could email me a box of tuits, that'd be great!
  Wed Jan 15 12:36:21 2014 The RT System itself - Status changed from 'new' to 'open'
  Sun Feb 02 07:02:11 2014 abraxxa [...] cpan.org - Correspondence added
Am Mi 15. Jan 2014, 12:36:21, autarch@urth.org schrieb: Show quoted text
> On Wed, 15 Jan 2014, Alexander Hartmaier via RT wrote: >
> > Wed Jan 15 12:31:09 2014: Request 92174 was acted upon. > > Transaction: Ticket created by ABRAXXA > > Queue: Math-Int128 > > Subject: Makefile.PL doesn't detect Debian 6 GCC 4.4.5 as > > supported > > Broken in: 0.13 > > Severity: (no value) > > Owner: Nobody > > Requestors: abraxxa@cpan.org > > Status: new > > Ticket <URL: https://rt.cpan.org/Ticket/Display.html?id=92174 > > > > > > > # cc --version > > cc (Debian 4.4.5-8) 4.4.5 > > Copyright (C) 2010 Free Software Foundation, Inc. > > This is free software; see the source for copying conditions. There > > is NO > > warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR > > PURPOSE. > > > > Maybe use the detection code from Math::Int64 instead of whitelisting > > compiler/version combos.
> > I'm planning to move both of these modules to use Config::AutoConf. I > think it's just a better practice to probe for feature capabilities > rather > than versions. > > If you could email me a box of tuits, that'd be great!
I looked into it but as I have almost no C knowledge I'm not sure what is the right way to test for a supported compiler. I guess it's checking for the uint128_t type by using Config::AutoConf->check_type but I couldn't get it to work. Neither the docs or http://grep.cpan.me/?q=Config%3A%3AAutoConf show an example of check_type usage. I might be able to work on it if you support me but I'm not sure if that's less work for you than doing it yourself. You can ping me on irc too (abraxxa). Cheers!
  Thu Oct 30 06:15:48 2014 salva [...] cpan.org - Correspondence added
On Wed Jan 15 12:31:09 2014, ABRAXXA wrote: Show quoted text
> # cc --version > cc (Debian 4.4.5-8) 4.4.5 > Copyright (C) 2010 Free Software Foundation, Inc. > This is free software; see the source for copying conditions. There > is NO > warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR > PURPOSE. > > Maybe use the detection code from Math::Int64 instead of whitelisting > compiler/version combos.
That compiler should be properly detected with the current implementation. Could you attach the output of 'cc --version --verbose'?
  Fri Nov 28 14:01:00 2014 haarg [...] haarg.org - Correspondence added
This is also a problem with the clang provided by Apple on OS X. Removing the checks from Makefile.PL allows it to compile and test successfully. $ cc --version --verbose Apple LLVM version 6.0 (clang-600.0.54) (based on LLVM 3.5svn) Target: x86_64-apple-darwin14.0.0 Thread model: posix
  Tue Dec 02 03:41:32 2014 salva [...] cpan.org - Correspondence added
El Vie Nov 28 14:01:00 2014, haarg escribió: Show quoted text
> This is also a problem with the clang provided by Apple on OS X. > Removing the checks from Makefile.PL allows it to compile and test > successfully. > > $ cc --version --verbose > Apple LLVM version 6.0 (clang-600.0.54) (based on LLVM 3.5svn) > Target: x86_64-apple-darwin14.0.0 > Thread model: posix
Graham, could you confirm that the clang version detection is still failing with the latest development version of Math::Int128 (0.16_02)? https://metacpan.org/release/SALVA/Math-Int128-0.16_02
  Fri Dec 12 08:25:47 2014 haarg [...] haarg.org - Correspondence added
On Tue Dec 02 03:41:32 2014, SALVA wrote: Show quoted text
> El Vie Nov 28 14:01:00 2014, haarg escribió:
> > This is also a problem with the clang provided by Apple on OS X. > > Removing the checks from Makefile.PL allows it to compile and test > > successfully. > > > > $ cc --version --verbose > > Apple LLVM version 6.0 (clang-600.0.54) (based on LLVM 3.5svn) > > Target: x86_64-apple-darwin14.0.0 > > Thread model: posix
> > Graham, > > could you confirm that the clang version detection is still failing > with the latest development version of Math::Int128 (0.16_02)? > > https://metacpan.org/release/SALVA/Math-Int128-0.16_02
The development version installs for me without issue.
  Sat Dec 13 13:59:18 2014 DROLSKY [...] cpan.org - Correspondence added
The latest dev release, 0.16_03, now detects the type using Config::AutoConf. If anyone reading this ticket could give it a shot and let me know if it works, I'd greatly appreciate that. The goal is to have this just work on every compiler that supports a 128-bit int type.
  Sun Feb 08 11:38:08 2015 salva [...] cpan.org - Status changed from 'open' to 'resolved'