watery01 / libyuv

Automatically exported from code.google.com/p/libyuv
0 stars 0 forks source link

Mac build files incomplete #71

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
gyp build on Mac does not build.
In the context of chromium is must be building, but not stand alone.
Likely missing DEPS files entries?

Original issue reported on code.google.com by fbarch...@google.com on 20 Aug 2012 at 11:49

GoogleCodeExporter commented 9 years ago
with a pending CL for DEPS, theres also a minor build error

/Users/fbarchard/src/libyuv/trunk/util/compare.cc:22:34: error: data argument 
not used by format string [-Werror,-Wformat-extra-args]

Original comment by fbarch...@google.com on 27 Aug 2012 at 7:29

GoogleCodeExporter commented 9 years ago
r325 fixes compare.cc

A bug in yasm md5.c also breaks on clang:
third_party/yasm/source/patched-yasm/libyasm/md5.c:166:31:{166:16-166:19}{166:31
-166:34}: error: argument to 'sizeof' in 'mems
et' call is the same expression as the destination; did you mean to dereference 
it? [-Werror,-Wsizeof-pointer-memaccess]
        memset(ctx, 0, sizeof(ctx));    /* In case it's sensitive */
               ~~~            ^~~
1 error generated.
This is fixed in a more recent yasm, to be updated with DEPS.
If patched locally, it builds

gclient runhooks
xcodebuild -project libyuv.xcodeproj -configuration Debug
xcodebuild -project libyuv.xcodeproj -configuration Release
xcodebuild -project libyuv_test.xcodeproj -configuration Debug
xcodebuild -project libyuv_test.xcodeproj -configuration Release

Original comment by fbarch...@chromium.org on 28 Aug 2012 at 12:08

GoogleCodeExporter commented 9 years ago
r327 updates DEPS to include clang and update yasm.

Original comment by fbarch...@google.com on 28 Aug 2012 at 3:09

GoogleCodeExporter commented 9 years ago
r329 updates the documentation on how to build for OSX
http://code.google.com/p/libyuv/wiki/GettingStarted
unittests pass on OSX (except version number, which will be updated to r330)

Original comment by fbarch...@google.com on 28 Aug 2012 at 4:12