{"push_id":1462622189,"size":8,"distinct_size":0,"ref":"refs/heads/code-fixes","head":"fde48f5538520bd98da42b4b9f60b5074c61bc2d","before":"e641ea465b03b2d2502d592f7fb19b2a4e648958","commits":[{"sha":"0f17495c80f3517ae0e30d9384fc6448f365390a","author":{"name":"Fred Wright","email":"dbfaf6a80dfd937969ef4cfe4989863cfe354fce@fwright.net"},"message":"Reverts \"Retroactive NEWS addition (for v3.15).\"\n\nThe original change was based on a misconception about why Navika GSV\nhandling improved in v3.15, which was actually due to the\nalready-documented signal ID fix.\n\nAlso fixes a typo in the original text.\n\n(cherry picked from commit 4418bc6be13aa372a372c41c996437b321b4248e)","distinct":false,"url":"https://api.github.com/repos/fhgwright/gpsd/commits/0f17495c80f3517ae0e30d9384fc6448f365390a"},{"sha":"9abf6f9c83b865e389e7fa6fe0044c7556800598","author":{"name":"Fred Wright","email":"dbfaf6a80dfd937969ef4cfe4989863cfe354fce@fwright.net"},"message":"Fixes some build failures of test_gpsmm.\n\nThe clock_gettime() function was originally a \"second-tier\" library\nfunction in Linux, implemented in librt and requiring \"-lrt\" when\nlinking.  Later versions moved it to libc, with the \"-lrt\" being\nunnecessary.  The build recipe for test_gpsmm here was failing to\naccount for the optional need for the \"-lrt\", causing it to fail\nto build on debian Wheezy.\n\nTESTED:\nNow passes \"scons build-all check\" on a Beaglebone Black running\nDebian Wheezy, where test_gpsmm formerly failed to build.  Also\nretested on OSX 10.9 (where librt doesn't exist) for good measure.","distinct":false,"url":"https://api.github.com/repos/fhgwright/gpsd/commits/9abf6f9c83b865e389e7fa6fe0044c7556800598"},{"sha":"f3f99af3ccfb84f7b5c5f95e7068fd662428e226","author":{"name":"Fred Wright","email":"dbfaf6a80dfd937969ef4cfe4989863cfe354fce@fwright.net"},"message":"Fixes gratuitous rebuild in non-git directories.\n\nThe code that generates the version string for development versions\ntries to identify the specific code state in one of two ways.  When\nthe build is within a git repo, it uses \"git describe\" output.  When\nthe build is not within a git repo, it uses the date and time.\n\nBefore this change, the applicable date and time was the current date\nand time, causing revision.h and everything that depends on it to be\nrebuilt every time, even if no sources changed.  This change causes it\nto use the latest mtime among the relevant sources instead, thus still\nhaving the desired effect without unnecessary changes in the version\nstring.\n\nSince git doesn't preserve mtimes, the resulting date/time may reflect\nthe last checkout of the relevant files rather than the last actual\nchange, but this is still a vast improvement.  And recent versions of\ngit avoid clobbering the mtimes of files that are not actually altered\nby the checkout.\n\nThis change also alters the version format somewhat in this case, both\nto incorporate the actual version number, and to eliminate some\ntruncation which was probably slightly incorrect, anyway.\n\nA sample new format is:\n\n\t\"3.17~dev-2016-09-24T19:06:46\"\n\nTESTED:\nTested both git and non-git cases.  Did not test the non-\"dev\" case,\nbut that code path is untouched.","distinct":false,"url":"https://api.github.com/repos/fhgwright/gpsd/commits/f3f99af3ccfb84f7b5c5f95e7068fd662428e226"},{"sha":"c173a5ecc51395225955df858315b0872d0a00b0","author":{"name":"Fred Wright","email":"dbfaf6a80dfd937969ef4cfe4989863cfe354fce@fwright.net"},"message":"Increases debug level to 3 in debug build.\n\nDebug level 3 makes macro definitions available to the debugger, which\ncan be useful.  Given that debug=yes is only used when necessary,\nthere's no reason to be stingy with the debug info when it is.\n\nTESTED:\nRan a debug build and observed \"-g3\" in the command lines.","distinct":false,"url":"https://api.github.com/repos/fhgwright/gpsd/commits/c173a5ecc51395225955df858315b0872d0a00b0"},{"sha":"40b8f6657f3766926900c7c552b0c25972d0230a","author":{"name":"Fred Wright","email":"dbfaf6a80dfd937969ef4cfe4989863cfe354fce@fwright.net"},"message":"Removes two redundant or troublesome compile flags.\n\nThe two flags -Wmissing-prototypes and -Wmissing-declarations were\nbeing both conditionally included after a validity test, and then\nunconditionally included later.  In cases where they were acceptable,\nthat resulted in doubly including them, and in cases where they\nwere unacceptable, that meant breaking the build.\n\nThe fix is simply to remove the unconditional adds.  This is a\nnecessary but not sufficient condition for building with CC=c99\nto work.\n\nTESTED:\nRan \"scons build-all check\" on OSX, and verified that the two\nflags were still present in the appropriate commands, but now\nonly once.  Also verified that building with CC=c99 no longer\nfails in a manner related to those flags.","distinct":false,"url":"https://api.github.com/repos/fhgwright/gpsd/commits/40b8f6657f3766926900c7c552b0c25972d0230a"},{"sha":"116c0b548aa8ce5430c9f19a40534b1056268f5b","author":{"name":"Fred Wright","email":"dbfaf6a80dfd937969ef4cfe4989863cfe354fce@fwright.net"},"message":"Tweaks xgps and xgpsspeed UIs.\n\nThis makes a couple of minor improvements to the xgps[speed] UIs:\n\n1) If a target (host/port/device) is specified, it is included in the\nwindow title.  This is especially useful when running multiple\ninstances pointing at different targets.\n\n2) Interprets blank host and/or port fields in host:port:device\nnotation as the default values.  This allows, e.g., specifying a\ndevice without having to explicitly specify the default host/port.\n\nAlso replaces all hardcoded '2947' instances with gps.GPSD_PORT.\n\nTESTED:\nTried various arguments, including using the option forms in\nxgpsspeed.  Also tried a long string of leading zeroes on the port\nnumber to verify that a ridiculously long target string is taken in\nstride.","distinct":false,"url":"https://api.github.com/repos/fhgwright/gpsd/commits/116c0b548aa8ce5430c9f19a40534b1056268f5b"},{"sha":"1564c089aac91f11104207f0f14125ce96e86037","author":{"name":"Sanjeev","email":"b17fb696c4f996fbd087b4f8f655dbda573454af@gmail.com"},"message":"Improve wording while reporting Fix Type","distinct":false,"url":"https://api.github.com/repos/fhgwright/gpsd/commits/1564c089aac91f11104207f0f14125ce96e86037"},{"sha":"fde48f5538520bd98da42b4b9f60b5074c61bc2d","author":{"name":"Fred Wright","email":"dbfaf6a80dfd937969ef4cfe4989863cfe354fce@fwright.net"},"message":"Moves compiler warning Configure setup later.\n\nThis change defers the conditional enabling of various compiler\nwarnings until after various other Configure checks are performed,\nfor a couple of reasons:\n\n1) It reduces the noise in config.log, by not having warnings\ncome from the more substantive Configure checks.\n\n2) It *may* avoid the CheckFunc problem with -Wstrict_prototypes\nwithout having to rewrite it, though that hasn't been confirmed.\n\nTESTED:\nRan \"scons build-all check\" with this change successfully.  Also\ncompared before and after versions of config.log, seeing expected\nimprovements and nothing unreasonable.","distinct":false,"url":"https://api.github.com/repos/fhgwright/gpsd/commits/fde48f5538520bd98da42b4b9f60b5074c61bc2d"}]}
{"actor":{"display_login":"fhgwright"}}