payload: {"push_id":3354522567,"size":3,"distinct_size":3,"ref":"refs/heads/macos-clang","head":"d41dc83761eca701ec285b5f93096b554f82fd8d","before":"fdc264fbdccf26546f1f94d8c60946175fb91446","commits":[{"sha":"6ed46315d65a0a02a849d9252abd5746739bde81","author":{"name":"Christopher Collins","email":"931fdb982a80bb744c6b95a3e1cdb3f421ab63fd@apache.org"},"message":"unittests: Fix some \"missing initializer\" warnings\n\nThese warnings are reported by clang.","distinct":true,"url":"https://api.github.com/repos/ccollins476ad/mynewt-core/commits/6ed46315d65a0a02a849d9252abd5746739bde81"},{"sha":"59a268c48a55fbcf61555d1d30c469660cd0aebe","author":{"name":"Christopher Collins","email":"931fdb982a80bb744c6b95a3e1cdb3f421ab63fd@apache.org"},"message":"compiler/sim: Use YAML sequences for flags\n\nThe sim compiler used to specify a set of flags as a single long string.\nThis mostly works, but it prevents newt from processing each flag\nindividually.  Newt needs to inspect each flag to prevent conflicting\nflags (e.g., two different -O switches).\n\nThe solution is to specify flags as a YAML sequence instead of a string.\nSome other compiler packages have already been converted to sequences;\nthis commit only applies to the sim compiler.","distinct":true,"url":"https://api.github.com/repos/ccollins476ad/mynewt-core/commits/59a268c48a55fbcf61555d1d30c469660cd0aebe"},{"sha":"d41dc83761eca701ec285b5f93096b554f82fd8d","author":{"name":"Christopher Collins","email":"931fdb982a80bb744c6b95a3e1cdb3f421ab63fd@apache.org"},"message":"compiler/sim: Use `gcc` in macOS, not `gcc-5`\n\nIn macOS, the sim compiler package invokes the `gcc-5` binary.  This is\nnot ideal - gcc5 is rather old now, so most mac users won't have it on\ntheir machine.\n\nThe reason the compiler package does not just specify `gcc` is that\nmacOS ships with an \"imposter\" gcc binary: an outdated version of clang\nwith the filename \"gcc\".  Until recently, much of the unit test code\nwould not build with clang, so `gcc-5` was specified as a workaround.\n\nTo get the code building with clang, this commit adds some compiler\nflags in macOS:\n\n    * -Wno-missing-braces\n    * -Wmissing-field-initializers\n\nThe first flag silences a spurious warning caused by a bug in clang\n(\"suggest braces around initialization of subobject\").  This bug has\nsupposedly been fixed in the latest clang:\nhttps://bugs.llvm.org/show_bug.cgi?id=21629.  However, it is difficult\nto replace the version of clang that ships with macOS, so I don't think\nwe should require users to upgrade their compiler.\n\nThe second flag is not strictly necessary - it just scales back the\neffects of the first flag.  It re-adds legitimate warnings that were\nsilenced by the first flag.","distinct":true,"url":"https://api.github.com/repos/ccollins476ad/mynewt-core/commits/d41dc83761eca701ec285b5f93096b554f82fd8d"}]}
other: {"actor":{"display_login":"ccollins476ad"}}