Opened 8 years ago

Closed 8 years ago

Last modified 8 years ago

#519 closed defect (fixed)

compilation error with clang

Reported by: Peter Johansson Owned by: Peter Johansson
Priority: major Milestone: svndigest 0.10
Component: build Version: 0.9.6
Keywords: Cc:

Description

macport maintainer reported that svndigest doesn't compile with clang (which is default with Xcode).

As indicated by macport changeset 97081:

# ./utility.h:315:56: error: redefinition of default argument 

the error is in yat/utility.h. This has already been fixed upstream in yat as well as in svndigest trunk.

Change History (7)

comment:1 Changed 8 years ago by Peter Johansson

(In [1499]) refs #519. Update to latest yat/utility.h. Limit the fetch to one file only as a full fetch seemed intrusive (and belong in trunk).

comment:2 Changed 8 years ago by Jari Häkkinen

(In [1501]) Addresses #519. clang++ requires include utility.h

Last edited 8 years ago by Jari Häkkinen (previous) (diff)

comment:3 Changed 8 years ago by Peter Johansson

(In [1502]) refs #519. add missing include and prefer using std::string

comment:4 Changed 8 years ago by Jari Häkkinen

(In [1503]) Addresses #519. Resolves compiler issue described in http://blog.llvm.org/2009/12/dreaded-two-phase-name-lookup.html

comment:5 Changed 8 years ago by Jari Häkkinen

Resolution: fixed
Status: newclosed

Both clang++ and g++ is happy with the current code.

comment:6 Changed 8 years ago by Jari Häkkinen

Both trunk and 0.9.x-stable was fixed.

comment:7 Changed 8 years ago by Peter Johansson

Milestone: svndigest 0.9.7svndigest 0.10

Milestone svndigest 0.9.7 deleted

Note: See TracTickets for help on using tickets.