{"action":"created","issue":{"url":"https://api.github.com/repos/TauLabs/TauLabs/issues/1542","labels_url":"https://api.github.com/repos/TauLabs/TauLabs/issues/1542/labels{/name}","comments_url":"https://api.github.com/repos/TauLabs/TauLabs/issues/1542/comments","events_url":"https://api.github.com/repos/TauLabs/TauLabs/issues/1542/events","html_url":"https://github.com/TauLabs/TauLabs/pull/1542","id":66013453,"number":1542,"title":"Build: add FLIGHT_BUILD_CONF environment variable","user":{"login":"mlyle","id":90903,"avatar_url":"https://avatars.githubusercontent.com/u/90903?v=3","gravatar_id":"","url":"https://api.github.com/users/mlyle","html_url":"https://github.com/mlyle","followers_url":"https://api.github.com/users/mlyle/followers","following_url":"https://api.github.com/users/mlyle/following{/other_user}","gists_url":"https://api.github.com/users/mlyle/gists{/gist_id}","starred_url":"https://api.github.com/users/mlyle/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mlyle/subscriptions","organizations_url":"https://api.github.com/users/mlyle/orgs","repos_url":"https://api.github.com/users/mlyle/repos","events_url":"https://api.github.com/users/mlyle/events{/privacy}","received_events_url":"https://api.github.com/users/mlyle/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":{"login":"solidgoldbomb","id":71638,"avatar_url":"https://avatars.githubusercontent.com/u/71638?v=3","gravatar_id":"","url":"https://api.github.com/users/solidgoldbomb","html_url":"https://github.com/solidgoldbomb","followers_url":"https://api.github.com/users/solidgoldbomb/followers","following_url":"https://api.github.com/users/solidgoldbomb/following{/other_user}","gists_url":"https://api.github.com/users/solidgoldbomb/gists{/gist_id}","starred_url":"https://api.github.com/users/solidgoldbomb/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/solidgoldbomb/subscriptions","organizations_url":"https://api.github.com/users/solidgoldbomb/orgs","repos_url":"https://api.github.com/users/solidgoldbomb/repos","events_url":"https://api.github.com/users/solidgoldbomb/events{/privacy}","received_events_url":"https://api.github.com/users/solidgoldbomb/received_events","type":"User","site_admin":false},"milestone":null,"comments":6,"created_at":"2015-04-02T21:19:25Z","updated_at":"2015-04-05T17:46:35Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/TauLabs/TauLabs/pulls/1542","html_url":"https://github.com/TauLabs/TauLabs/pull/1542","diff_url":"https://github.com/TauLabs/TauLabs/pull/1542.diff","patch_url":"https://github.com/TauLabs/TauLabs/pull/1542.patch"},"body":"It's a knob, with the values 'default' (which is the default), 'release',\r\nand 'debug'.  In debug, DEBUG is set to YES in the flight firmwares.  In\r\ndebug and default, compilation caching is allowed (and ccache is smart\r\nenough to cache default/debug separately because it includes command line\r\narguments in its hash).  In release, there is no compilation caching (and\r\njenkins/release build scripts should be updated to use this mode).\r\n\r\nThis parallels the GCS_BUILD_CONF environment variable which also disables\r\ncompilation caching for GCS build."},"comment":{"url":"https://api.github.com/repos/TauLabs/TauLabs/issues/comments/89818412","html_url":"https://github.com/TauLabs/TauLabs/pull/1542#issuecomment-89818412","issue_url":"https://api.github.com/repos/TauLabs/TauLabs/issues/1542","id":89818412,"user":{"login":"solidgoldbomb","id":71638,"avatar_url":"https://avatars.githubusercontent.com/u/71638?v=3","gravatar_id":"","url":"https://api.github.com/users/solidgoldbomb","html_url":"https://github.com/solidgoldbomb","followers_url":"https://api.github.com/users/solidgoldbomb/followers","following_url":"https://api.github.com/users/solidgoldbomb/following{/other_user}","gists_url":"https://api.github.com/users/solidgoldbomb/gists{/gist_id}","starred_url":"https://api.github.com/users/solidgoldbomb/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/solidgoldbomb/subscriptions","organizations_url":"https://api.github.com/users/solidgoldbomb/orgs","repos_url":"https://api.github.com/users/solidgoldbomb/repos","events_url":"https://api.github.com/users/solidgoldbomb/events{/privacy}","received_events_url":"https://api.github.com/users/solidgoldbomb/received_events","type":"User","site_admin":false},"created_at":"2015-04-05T17:46:35Z","updated_at":"2015-04-05T17:46:35Z","body":"@mlyle This is much better.  Thanks for making the changes.  I'm not thrilled about the DEBUG enable being in firmware-defs.mk since it mixes policy with the basic tool setup in firmware-defs but I can see why you would prefer this to replicating in each target makefile.\r\n\r\nThis version is OK to merge as soon as Jenkins gives it a clean bill of health.\r\n\r\nI've poked @lilvinz to see if he can check on the memory exhaust error in Jenkins.\r\n"}}