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)



Subject: Makefile.PL doesn't detect Debian 6 GCC 4.4.5 as supported
Download (untitled) / with headers
text/plain 337b
# 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.
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>
Download (untitled) / with headers
text/plain 1015b
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!
Download (untitled) / with headers
text/plain 1.6k
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!
Download (untitled) / with headers
text/plain 535b
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'?
Download (untitled) / with headers
text/plain 289b
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
Download (untitled) / with headers
text/plain 549b
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
Download (untitled) / with headers
text/plain 676b
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.
Download (untitled) / with headers
text/plain 274b
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.


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.