Bug #27456 decimal.h is not installed with public include files
Submitted: 27 Mar 2007 0:38 Modified: 27 Apr 2007 17:30
Reporter: John David Duncan
Status: Closed
Category:Server: Packaging Severity:S3 (Non-critical)
Version:5.0, 5.1 OS:Any (all)
Assigned to: Joerg Bruehe Target Version:

[27 Mar 2007 0:38] John David Duncan
Description:
The public mysql strings library (libmystrings) includes functions such as bin2decimal
and decimal2string that work with the MySQL 5.0 "new" decimal type -- but these functions
cannot be used as a public API because the function prototypes for them (and the
declaration of the decimal_t structure) are not in any public include files. 

In the mysql source tree, these declarations are in include/decimal.h, but this file is
not installed with a build (i.e. it is not in /usr/local/mysql/include/mysql) 

In the past this was not an issue because there was no need to use these functions
outside the mysql server.  But now you need them for development using the NDB API, so
you can work with DECIMAL columns that are stored in NDB. 

How to repeat:
N/A.
[27 Mar 2007 1:03] Miguel Solorzano
Thank you for the bug report.
[12 Apr 2007 21:51] Bugs System
A patch for this bug has been committed. After review, it may
be pushed to the relevant source trees for release in the next
version. You can access the patch from:

  http://lists.mysql.com/commits/24422

ChangeSet@1.2445, 2007-04-12 21:50:56+02:00, joerg@trift2. +1 -0
  Header file "decimal.h" needs to be delivered with binary packages:
  
  Bug#27456    decimal.h is not installed with public include files
[17 Apr 2007 14:29] Joerg Bruehe
Will be used in the 5.0.40 build,

but is not yet pushed to 5.1 (expect: 5.1.18).
[17 Apr 2007 21:37] Paul DuBois
Noted in 5.0.40 changelog.

The decimal.h header file was incorrectly omitted from binary
distributions.

Changing status of report to Patch Queued pending push into
5.1.x.
[27 Apr 2007 11:22] Bugs System
Pushed into 5.1.18-beta
[27 Apr 2007 11:24] Bugs System
Pushed into 5.0.42
[27 Apr 2007 17:30] Paul DuBois
Noted in 5.1.18 changelog.