Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 336385 - [AIX] Swap build order of libtptpConfig and libprocessControlUtil
Summary: [AIX] Swap build order of libtptpConfig and libprocessControlUtil
Status: CLOSED FIXED
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: TPTP (show other bugs)
Version: unspecified   Edit
Hardware: PC AIX
: P3 critical (vote)
Target Milestone: ---   Edit
Assignee: Mike Reid CLA
QA Contact: Kathy Chan CLA
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 335859
  Show dependency tree
 
Reported: 2011-02-04 13:40 EST by Mike Reid CLA
Modified: 2016-05-05 10:38 EDT (History)
3 users (show)

See Also:
kathy: pmc_approved? (oec)
kathy: pmc_approved? (ernest)
kathy: pmc_approved? (kathy)
jgwest: pmc_approved+
jerome.bozier: pmc_approved+

Attachments
Swap build order (2.64 KB, patch)
2011-02-04 13:41 EST, Mike Reid CLA 		no flags Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Mike Reid CLA 2011-02-04 13:40:39 EST 
The fix for bug 335859 introduced a build break on AIX. Change to library dependencies also requires a swap in the build order of tptpConfig and processControlUtil libraries that was not noticed at the time the original fix was created.

Opening this bug to request PMC approval on build order change.
Comment 1 Mike Reid CLA 2011-02-04 13:41:21 EST 
Created attachment 188349 [details]
Swap build order

Patch attached.
Comment 2 Mike Reid CLA 2011-02-04 13:43:55 EST 
I would like to request that the following defect be considered for PMC
approval for 4.7.2.

1.  Explain why you believe this is a stop-ship defect. How does the defect
manifest itself, and how will users of TPTP / consuming products be affected if
the defect is not fixed?

Downstream products cannot build on AIX platform without this fix.

2. Is there a work-around? If so, why do you believe the work-around is
insufficient?

No.

3. Is this a regression or API breakage? Explain.

Regression; introduced by bug 335859.

4. Does this require new API?

No.

5. Who performed the code review?

Jonathan West

6. Is there a test case attached to the bugzilla record?

n/a

7. What is the nature of the fix?  What is the risk associated with this fix?

Build order change. Low risk.

8. Is this fix related to any standards that TPTP adheres to? If so, who has
validated that the fix continues to adhere to the standard?

No.
Comment 3 Kathy Chan CLA 2011-02-04 13:48:49 EST 
Requesting PMC approval for TPTP 4.7.2.
Comment 4 Mike Reid CLA 2011-02-04 14:23:02 EST 
Patch checked into HEAD w/ PMC approval.
Comment 5 Mike Reid CLA 2011-04-04 11:00:37 EDT 
Closing.