apr-bugs mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From bugzi...@apache.org
Subject DO NOT REPLY [Bug 45513] New: Wrong pid_t_fmt detection for Solaris
Date Thu, 31 Jul 2008 11:27:09 GMT

           Summary: Wrong pid_t_fmt detection for Solaris
           Product: APR
           Version: HEAD
          Platform: Sun
        OS/Version: Solaris
            Status: NEW
          Keywords: PatchAvailable
          Severity: normal
          Priority: P2
         Component: APR
        AssignedTo: bugs@apr.apache.org
        ReportedBy: rainer.jung@kippdata.de

Created an attachment (id=22336)
 --> (https://issues.apache.org/bugzilla/attachment.cgi?id=22336)
Fix APR_PID_T_FMT on Solaris

Revision 553652 of configure.in introduced changes in printf format detection
for size_t and ssize_t to branch 1.3 and trunk. There were two unwanted side
effects, that break detection of format for pid_t on Solaris (and lead to lots
of Compilation warnings for httpd 2.2.9).

Problem 1: In the block setting pid_t_fmt for Solaris it got changed from 

   pid_t_fmt='#define APR_PID_T_FMT "ld"'



That type of change was wanted for size_t and ssize_t, but is unwanted for
pid_t, since later on in configure.in, the define style is still used, and the
way pid_t_fmt is included in apr.h.in was not adjusted.

Problem 2: the whole block of special system handling concerning format
characters was moved around. Before it was located under the lines were the
defaults for pid_t_fmt had been set, now it is located above, so any special
setting gets overwritten by the defaults.

The attached patch fixed it. Same file for 1.3 and trunk.

Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

To unsubscribe, e-mail: bugs-unsubscribe@apr.apache.org
For additional commands, e-mail: bugs-help@apr.apache.org

View raw message