On Tue, Jan 29, 2013 at 11:55:45AM -0500, Daniel Johnson wrote:
>
> On Jan 29, 2013, at 11:16 AM, Alexander Hansen <alexanderk.han...@gmail.com>
> wrote:
>
> > On 1/29/13 9:12 AM, Jack Howarth wrote:
> >> On Tue, Jan 29, 2013 at 08:20:42AM -0700, Alexander Hansen wrote:
> >>> On 1/29/13 7:49 AM, Jack Howarth wrote:
> >>>> Using a clean bootstrap of fink fink-0.34.5 on Mac OS X 10.8 with
> >>>> Xcode 4.6 and
> >>>> its associated Command Line Tools installed, I am seeing a failure in
> >>>> the build of
> >>>> db51-aes-5.1.29-2 at...
> >>>>
> >>>> clang -c -I. -I../src -Os ../src/btree/bt_compare.c -fno-common -DPIC
> >>>> -o .libs/bt_compare.o
> >>>> In file included from ../src/btree/bt_compare.c:46:
> >>>> In file included from ./db_int.h:999:
> >>>> In file included from ../src/dbinc/mutex.h:15:
> >>>> In file included from ../src/dbinc/mutex_int.h:12:
> >>>> ../src/dbinc/atomic.h:179:19: error: definition of builtin function
> >>>> '__atomic_compare_exchange'
> >>>> static inline int __atomic_compare_exchange(
> >>>> ^
> >>>> 1 error generated.
> >>>>
> >>>> This looks like the issue discussed in...
> >>>>
> >>>> http://lists.cs.uiuc.edu/pipermail/cfe-dev/2012-April/020973.html
> >>>> http://lists.cs.uiuc.edu/pipermail/cfe-dev/2012-April/020974.html
> >>>>
> >>>> Jack
> >>>>
> >>>
> >>> Since this is a maintained package, I'll cc the maintainer.
> >>
> >> The attached patch derived from
> >> http://unix.derkeiler.com/Mailing-Lists/FreeBSD/current/2012-05/msg00058.html
> >> solves the build problem on Clang 4.2.
> >> Jack
> >> ps The net change to the info file is...
> >>
> >> --- db51-aes.info 2012-01-17 12:19:49.000000000 -0500
> >> +++ /sw/fink/10.7/local/main/finkinfo/db51-aes.info 2013-01-29
> >> 11:06:57.000000000 -0500
> >> @@ -1,7 +1,7 @@
> >> Info4: <<
> >> Package: db51-aes%type_pkg[-64bit]
> >> Version: 5.1.29
> >> -Revision: 2
> >> +Revision: 3
> >> Type: -64bit (boolean)
> >> Architecture: ( %type_raw[-64bit] = -64bit ) powerpc, ( %type_raw[-64bit]
> >> = -64bit ) i386
> >> CustomMirror: <<
> >> @@ -161,6 +161,8 @@
> >> Homepage: http://www.oracle.com/database/berkeley-db/index.html
> >> Maintainer: Daniel Johnson <dan...@daniel-johnson.org>
> >> Source-MD5: a94ea755ab695bc04f82b94d2e24a1ef
> >> +PatchFile: %n.patch
> >> +PatchFile-MD5: 1ed24665950881ecab778163aa986fa1
> >>
> >> InfoTest: TestScript: cd test/micro; sh test_micro || exit 2
> >> <<
> >>
> >
> > Does it really need an increased revision?
> >
>
> I don't think so. This is simply a build-time issue and shouldn't affect the
> finished product. I won't have time to do anything with this until tonight,
> but if someone wants to commit this it's fine with me. Does this issue only
> affect db51? What about db53 or db48?
>
It should be transparent to the pre-clang 4.2 since all we are doing is
renaming the underscored
subroutine and its usage in db51. Note the comment on this usage of underscored
calls in db in the
FreeBSD mailing list posting with the patch...
> This is because bdb5 tries to redefine gcc's builtin atomic functions,
> which are also supported by clang (this is a feature :). It is rather
> dumb to use names starting with underscores for this, since they are
> reserved.
Jack
> Daniel
------------------------------------------------------------------------------
Master Visual Studio, SharePoint, SQL, ASP.NET, C# 2012, HTML5, CSS,
MVC, Windows 8 Apps, JavaScript and much more. Keep your skills current
with LearnDevNow - 3,200 step-by-step video tutorials by Microsoft
MVPs and experts. ON SALE this month only -- learn more at:
http://p.sf.net/sfu/learnnow-d2d
_______________________________________________
Fink-devel mailing list
Fink-devel@lists.sourceforge.net
List archive:
http://news.gmane.org/gmane.os.apple.fink.devel
Subscription management:
https://lists.sourceforge.net/lists/listinfo/fink-devel
