Skip Menu |

This queue is for tickets about the Config-JSON CPAN distribution.

Report information
The Basics
Id: 67411
Status: resolved
Priority: 0/
Queue: Config-JSON

People
Owner: Nobody in particular
Requestors: mosesamaro [...] gmail.com
Cc:
AdminCc:

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



Subject: Bad comment at line 520
Some compilers, in particular on ancient horrible systems such as AIX that no one should ever... ever... ever use, require that you don't use the // style single line comment syntax. This breaks the build on AIX, unless you've somehow gotten GCC to be the default compiler for the system. There's a // style comment in Mouse.c and Mouse.xs. You can find these comments with the following command: find . -type f | xargs grep "\/\/" Thank you for writing this module (which I'm going to fix and use on AIX myself). Thanks in advance for a fix for this as well.
From: mosesamaro [...] gmail.com
On Tue Apr 12 12:15:34 2011, MosesAmaro wrote: Show quoted text
> Some compilers, in particular on ancient horrible systems such as AIX > that no one should ever... ever... ever use, require that you don't use > the // style single line comment syntax. > > This breaks the build on AIX, unless you've somehow gotten GCC to be the > default compiler for the system. > > There's a // style comment in Mouse.c and Mouse.xs. You can find these > comments with the following command: > > find . -type f | xargs grep "\/\/" > > Thank you for writing this module (which I'm going to fix and use on AIX > myself). Thanks in advance for a fix for this as well.
Jesus, I'm an idiot. The problem is in Mouse, which is a dependency for Config::JSON. I should not have posted this here. Please reject this bug with apologies. =(
No worries. BTW, Mouse isn't necessarily a prereq. We use Any::Moose, so if you prefer you can use Moose. That might get rid of the problem you're seeing.