Skip Menu |

This queue is for tickets about the DBD-SQLite CPAN distribution.

Report information
The Basics
Id: 94207
Status: resolved
Priority: 0/
Queue: DBD-SQLite
People
Owner: Nobody in particular
Requestors: zefram [...] fysh.org
Cc:
AdminCc:
Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: (no value)

History Show all quoted text
  Wed Mar 26 09:02:06 2014 zefram [...] fysh.org - Ticket created
Subject: overrides optimisation level badly
Date: Wed, 26 Mar 2014 13:01:50 +0000
To: bug-DBD-SQLite [...] rt.cpan.org
From: Zefram <zefram [...] fysh.org>
DBD-SQLite has some logic in Makefile.PL that attempts to override the configured C compilation optimisation flags. It does it badly, not entirely successfully, but in any case impedes the purpose of that Perl configuration option. I think it shouldn't be overriding this at all. If it is indeed necessary on some platform, then a much more narrowly tailored override should be implemented instead. I wouldn't be terribly surprised at some compiler needing to have the optimisation dialled down a bit on sqlite3.c, merely because of its abnormal size. If that's the cause, another option would be to split it up. -zefram
  Tue Aug 04 07:27:58 2015 ishigaki [...] cpan.org - Correspondence added
On Wed Mar 26 22:02:06 2014, zefram@fysh.org wrote: Show quoted text
> DBD-SQLite has some logic in Makefile.PL that attempts to override the > configured C compilation optimisation flags. It does it badly, not > entirely successfully, but in any case impedes the purpose of that Perl > configuration option. I think it shouldn't be overriding this at all. > If it is indeed necessary on some platform, then a much more narrowly > tailored override should be implemented instead. > > I wouldn't be terribly surprised at some compiler needing to have the > optimisation dialled down a bit on sqlite3.c, merely because of its > abnormal size. If that's the cause, another option would be to split > it up. > > -zefram
Removed OPTIMIZE in 1.47_01 and 1.48. Thanks.
  Tue Aug 04 07:27:59 2015 The RT System itself - Status changed from 'new' to 'open'
  Tue Aug 04 07:28:01 2015 ishigaki [...] cpan.org - Status changed from 'open' to 'resolved'