Opened 8 years ago

Closed 8 years ago

Last modified 8 years ago

#1907 closed defect (fixed)

Problem compiling BIND 10 on MacOS X 10.7

Reported by: cas Owned by: jinmei
Priority: medium Milestone: Sprint-20120515
Component: build system Version:
Keywords: Cc:
CVSS Scoring: Parent Tickets:
Sensitive: no Defect Severity: N/A
Sub-Project: DNS Feature Depending on Ticket:
Estimated Difficulty: 4 Add Hours to Ticket: 0
Total Hours: 5.5 Internal?: no

Description

I'm trying to compile BIND 10 20120329 on MacOS X. Python3 and log4cplus has been installed from HomeBrew? (http://mxcl.github.com/homebrew/) and hitting this error:

configure:15291: g++ -c -g -O2 -Wall -Wextra -Wwrite-strings -Woverloaded-virtual -Wno-sign-compare -Werror -Werror -Wno-unused-parameter -I/usr/local/Cellar/python3/3.2.3/include/py
thon3.2m -I/usr/local/Cellar/python3/3.2.3/include/python3.2m conftest.cpp >&5
cc1plus: warnings being treated as errors
In file included from /usr/local/Cellar/python3/3.2.3/include/python3.2m/Python.h:111,

from conftest.cpp:24:

/usr/local/Cellar/python3/3.2.3/include/python3.2m/modsupport.h:27: warning: 'PyArg_ParseTuple' is an unrecognized format function type
configure:15291: $? = 1

full config.log attached

Subtickets

Attachments (4)

config.log (39.7 KB) - added by cas 8 years ago.
pyconfig.diff (437 bytes) - added by jinmei 8 years ago.
python3.rb.diff (475 bytes) - added by jinmei 8 years ago.
configure.diff (524 bytes) - added by jinmei 8 years ago.

Download all attachments as: .zip

Change History (18)

Changed 8 years ago by cas

comment:1 Changed 8 years ago by jreed

There may be a problem with the homebrew install of Python3 which makes it use the other installed python 2 headers?

comment:2 Changed 8 years ago by jelte

  • Milestone changed from New Tasks to Sprint-20120515

comment:3 Changed 8 years ago by jinmei

  • Owner changed from UnAssigned to jinmei
  • Status changed from new to accepted

comment:4 Changed 8 years ago by jinmei

I encountered this problem myself, worked around it by tweaking an
installed file under /usr/local/include/python3.2m and then forgot it.

Now I looked into it more closely. It actually seems to be a bug (or
at least a suboptimal setup) of HomeBrew. A short term workaround is
to build BIND 10 suppressing -Werror:

./configure --without-werror

comment:5 follow-up: Changed 8 years ago by jreed

Can you provide a diff for the local include file(s)?

And who will report upstream to HomeBrew? packagers?

Changed 8 years ago by jinmei

Changed 8 years ago by jinmei

comment:6 in reply to: ↑ 5 Changed 8 years ago by jinmei

Replying to jreed:

Can you provide a diff for the local include file(s)?

See the attached file. The file to be patched is /usr/local/include/python3.2m/pyconfig.h.

I also think a patch (also attached) to the python3 formula (located under
/usr/local/Library/Formula) will fix it at the HomeBrew? side (but not
even tried it yet).

And who will report upstream to HomeBrew? packagers?

I planned to report it as part of this task, but HomeBrew? seems to
have a high bar for a bug reporter, so it will take some time to meet
the prerequisites (not technically difficult but time consuming).
https://github.com/mxcl/homebrew/wiki/Reporting-Bugs

Now that we seem to have higher priority tasks I'm going to release
this ticket for now. If someone has time to report this issue or try
the patch for HomeBrew? that would be great.

comment:7 Changed 8 years ago by jinmei

  • Owner changed from jinmei to UnAssigned
  • Status changed from accepted to assigned

comment:8 Changed 8 years ago by jinmei

  • Owner changed from UnAssigned to jinmei
  • Status changed from assigned to accepted

comment:9 Changed 8 years ago by jinmei

Okay, I've reported the problem to Homebrew with a suggested patch:
https://github.com/mxcl/homebrew/issues/12194

I also updated the system notes page for Mac OS X:
http://bind10.isc.org/wiki/SystemNotesMacOSX

For this ticket, I propose simply including a link to this note to the
./configure error message. See the attached diff (configure.diff).

Please review it. If okay, I'll merge the diff and close the ticket.

Changed 8 years ago by jinmei

comment:10 Changed 8 years ago by jinmei

  • Owner changed from jinmei to UnAssigned
  • Status changed from accepted to reviewing

comment:11 follow-up: Changed 8 years ago by muks

  • Owner changed from UnAssigned to jinmei

Looks good. You can commit it as-is, but I'd remove the 'also' from 'see also' text.

comment:12 in reply to: ↑ 11 Changed 8 years ago by jinmei

Replying to muks:

Looks good. You can commit it as-is, but I'd remove the 'also' from 'see also' text.

Committed the patch with the suggested change.

Thanks for the review, closing the ticket.

comment:13 Changed 8 years ago by jinmei

  • Resolution set to fixed
  • Status changed from reviewing to closed

comment:14 Changed 8 years ago by jinmei

  • Total Hours changed from 0 to 5.5
Note: See TracTickets for help on using tickets.