Bugzilla will be upgraded to version 5.0. The upgrade date is tentatively scheduled for 2 December 2018, pending final testing and feedback.
Bug 522267 - Windows: Qpid C++ pid_t and ssize_t 3rd-party compat
Windows: Qpid C++ pid_t and ssize_t 3rd-party compat
Status: CLOSED ERRATA
Product: Red Hat Enterprise MRG
Classification: Red Hat
Component: qpid-cpp (Show other bugs)
1.1.6
All Windows
urgent Severity high
: 1.3
: ---
Assigned To: Pete MacKinnon
Frantisek Reznicek
:
Depends On:
Blocks: 467890 506610
  Show dependency treegraph
 
Reported: 2009-09-09 18:44 EDT by Pete MacKinnon
Modified: 2015-11-15 20:11 EST (History)
5 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Compiling Qpid on Windows failed because Qpid defined the pid_t and ssize_t type declarations in public headers, which some third-party headers also define. This update moves these declarations to private headers or implementation code, with the result that compilation succeeds on Windows systems.
Story Points: ---
Clone Of:
Environment:
Last Closed: 2010-10-14 12:01:10 EDT
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---


Attachments (Terms of Use)


External Trackers
Tracker ID Priority Status Summary Last Updated
Red Hat Product Errata RHSA-2010:0773 normal SHIPPED_LIVE Moderate: Red Hat Enterprise MRG Messaging and Grid Version 1.3 2010-10-14 11:56:44 EDT

  None (edit)
Description Pete MacKinnon 2009-09-09 18:44:53 EDT
https://issues.apache.org/jira/browse/QPID-1951

These are currently declared in IntegerTypes.h but cause compile-time conflicts with other libraries (i.e., Grid) that also must declare these types. Not clear if they should be exposed as is.
Comment 3 Andrew Stitcher 2009-12-08 10:36:09 EST
These definitions should simply be removed from the windows code base.

It looks like the ssize_t definition is trivially removable from the windows code.
The pid_t looks used in more places and would be a little fiddlier
Comment 4 Andrew Stitcher 2009-12-08 10:37:39 EST
This issue needs an upstream qpid JIRA reported as the work is going to be done upstream - is there a reported JIRA?
Comment 5 Pete MacKinnon 2009-12-08 10:52:09 EST
https://issues.apache.org/jira/browse/QPID-1951
Comment 6 Frantisek Reznicek 2010-04-22 09:36:25 EDT
ssize_t and pid_t types are now removed from qpid/sys/windows/IntegerTypes.h header as suggested.

Reproduced on qpidc-0.5.829175-3.fc11.src.rpm, fixed lately on  qpid-cpp-mrg-0.7.935473-1.el5.src.rpm.

The change which fixed that is here:
http://svn.apache.org/viewvc/qpid/trunk/qpid/cpp/include/qpid/sys/windows/IntegerTypes.h?r1=822963&r2=890929&diff_format=h

-> VERIFIED
Comment 7 Pete MacKinnon 2010-10-05 09:49:28 EDT
    Technical note added. If any revisions are required, please edit the "Technical Notes" field
    accordingly. All revisions will be proofread by the Engineering Content Services team.
    
    New Contents:
Cause: Windows compilation of Qpid public headers with 3rd-party headers that define pid_t and ssize_t
Consequence: compilation fails
Fix: removal and/or movement to private headers or implementation code
Result: compilation succeeds
Comment 8 Pete MacKinnon 2010-10-05 09:50:30 EDT
    Technical note updated. If any revisions are required, please edit the "Technical Notes" field
    accordingly. All revisions will be proofread by the Engineering Content Services team.
    
    Diffed Contents:
@@ -1,4 +1,4 @@
 Cause: Windows compilation of Qpid public headers with 3rd-party headers that define pid_t and ssize_t
 Consequence: compilation fails
-Fix: removal and/or movement to private headers or implementation code
+Fix: removal and/or movement of pid_t, ssize_t declarations to private headers or implementation code
 Result: compilation succeeds
Comment 9 Douglas Silas 2010-10-05 14:48:37 EDT
    Technical note updated. If any revisions are required, please edit the "Technical Notes" field
    accordingly. All revisions will be proofread by the Engineering Content Services team.
    
    Diffed Contents:
@@ -1,4 +1 @@
-Cause: Windows compilation of Qpid public headers with 3rd-party headers that define pid_t and ssize_t
+Compiling Qpid on Windows failed because Qpid defined the pid_t and ssize_t type declarations, which some third-party headers also define. This update moves these declarations to private headers or implementation code, with the result that compilation succeeds on Windows systems.-Consequence: compilation fails
-Fix: removal and/or movement of pid_t, ssize_t declarations to private headers or implementation code
-Result: compilation succeeds
Comment 10 Douglas Silas 2010-10-05 14:49:50 EDT
    Technical note updated. If any revisions are required, please edit the "Technical Notes" field
    accordingly. All revisions will be proofread by the Engineering Content Services team.
    
    Diffed Contents:
@@ -1 +1 @@
-Compiling Qpid on Windows failed because Qpid defined the pid_t and ssize_t type declarations, which some third-party headers also define. This update moves these declarations to private headers or implementation code, with the result that compilation succeeds on Windows systems.+Compiling Qpid on Windows failed because Qpid defined the pid_t and ssize_t type declarations in public headers, which some third-party headers also define. This update moves these declarations to private headers or implementation code, with the result that compilation succeeds on Windows systems.
Comment 12 errata-xmlrpc 2010-10-14 12:01:10 EDT
An advisory has been issued which should help the problem
described in this bug report. This report is therefore being
closed with a resolution of ERRATA. For more information
on therefore solution and/or where to find the updated files,
please follow the link below. You may reopen this bug report
if the solution does not work for you.

http://rhn.redhat.com/errata/RHSA-2010-0773.html

Note You need to log in before you can comment on or make changes to this bug.