On Jan 29, 2013, at 12:06 PM, Jack Howarth <howa...@bromo.med.uc.edu> wrote:
> 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. > db48 and db53 also needed the patch. All three should now build with Xcode 4.6. 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
