Bug 740439

Summary: Medium: cib: Fix compilation of ACL code
Product: [Fedora] Fedora Reporter: Andrew Beekhof <abeekhof>
Component: pacemakerAssignee: Andrew Beekhof <andrew>
Status: CLOSED CURRENTRELEASE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: medium    
Version: 15CC: andrew, fdinitto, lhh
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
URL: https://github.com/ClusterLabs/pacemaker/issues/1
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2011-09-22 12:27:28 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description Andrew Beekhof 2011-09-22 04:11:21 UTC 
 Filed by: gao-yan at 2011-09-05 03:24:27 (Closed: 2011-09-05 19:59:28)
 Patch: https://github.com/ClusterLabs/pacemaker/pull/1.patch

 | Hi Andrew,
 | 
 | The following commit breaks compilation of ACL code:
 | 
 | commit 8b1b087144f60c569083156c7de3f3c915dd910f
 | Author: Andrew Beekhof <andrew>
 | Date:   Fri Sep 2 20:09:36 2011 +1000
 | 
 |     Low: Core: Remove use of the xml_prop_iter macros - they confuse parsers, analyisers and indent tools
 | 
 | The commit fixes this.

 gao-yan at 2011-09-05 20:00:49
 > Curious. Shouldn't it be automatically closed?

 russellb at 2011-09-06 05:27:10
 > I think it will only get automatically closed if the pull request is merged by clicking merge in the pull request.  If you merge manually, you have to come back and close the pull request.

 gao-yan at 2011-09-06 05:59:48
 > Makes sense. Thanks for the info!

 beekhof at 2011-09-06 15:16:09
 > Yeah, in this case I used the fork queue.
 >
Comment 1 Andrew Beekhof 2011-09-22 12:27:28 UTC 
Testing BZ import from github.