)]}'
{"tools/check_exec.sh":[{"author":{"_account_id":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},"change_message_id":"167b1975bb97dc78179326eccb10478fff5ca0c3","unresolved":false,"context_lines":[{"line_number":1,"context_line":"#!/bin/bash"},{"line_number":2,"context_line":""},{"line_number":3,"context_line":"EXECUTABLE\u003d`find $1 -type f -executable`"},{"line_number":4,"context_line":"if [ -n \"$EXECUTABLE\" ]; then"},{"line_number":5,"context_line":"    echo \"Error: Executable files found: \""},{"line_number":6,"context_line":"    echo $EXECUTABLE"}],"source_content_type":"text/x-sh","patch_set":1,"id":"da20952f_e3417eaa","line":3,"updated":"2015-08-28 19:13:45.000000000","message":"BSD find (e.g. OSX) doesn\u0027t support -executable.\n\nSuggest:\nEXECUTABLE\u003d`find $1 -type f -perm +111 `\nwhich appears to work with both BSD and Linux find.","commit_id":"bbfe9d931b6500acc09d1a93945622df1141b9b4"},{"author":{"_account_id":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},"change_message_id":"b3ef8b38937c9b6e3a8c20f5e546dfa35696aa5f","unresolved":false,"context_lines":[{"line_number":1,"context_line":"#!/bin/bash"},{"line_number":2,"context_line":""},{"line_number":3,"context_line":"EXECUTABLE\u003d`find $1 -type f -executable`"},{"line_number":4,"context_line":"if [ -n \"$EXECUTABLE\" ]; then"},{"line_number":5,"context_line":"    echo \"Error: Executable files found: \""},{"line_number":6,"context_line":"    echo $EXECUTABLE"}],"source_content_type":"text/x-sh","patch_set":1,"id":"da20952f_aead669c","line":3,"in_reply_to":"da20952f_93852fd0","updated":"2015-08-31 15:45:49.000000000","message":"Yuck.  FWIW, here\u0027s from the BSD find man page on OSX Yosemite:\n\n     -perm [-|+]mode\n             The mode may be either symbolic (see chmod(1)) or an octal number.  If the mode is symbolic, a starting value of zero is assumed and the mode sets or clears\n             permissions without regard to the process\u0027 file mode creation mask.  If the mode is octal, only bits 07777 (S_ISUID | S_ISGID | S_ISTXT | S_IRWXU | S_IRWXG |\n             S_IRWXO) of the file\u0027s mode bits participate in the comparison.  If the mode is preceded by a dash (``-\u0027\u0027), this primary evaluates to true if at least all of\n             the bits in the mode are set in the file\u0027s mode bits.  If the mode is preceded by a plus (``+\u0027\u0027), this primary evaluates to true if any of the bits in the\n             mode are set in the file\u0027s mode bits.  Otherwise, this primary evaluates to true if the bits in the mode exactly match the file\u0027s mode bits.  Note, the first\n             character of a symbolic mode may not be a dash (``-\u0027\u0027).","commit_id":"bbfe9d931b6500acc09d1a93945622df1141b9b4"},{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"694200a8d4516a4837d4db129094d6860c1d7849","unresolved":false,"context_lines":[{"line_number":1,"context_line":"#!/bin/bash"},{"line_number":2,"context_line":""},{"line_number":3,"context_line":"EXECUTABLE\u003d`find $1 -type f -executable`"},{"line_number":4,"context_line":"if [ -n \"$EXECUTABLE\" ]; then"},{"line_number":5,"context_line":"    echo \"Error: Executable files found: \""},{"line_number":6,"context_line":"    echo $EXECUTABLE"}],"source_content_type":"text/x-sh","patch_set":1,"id":"da20952f_93852fd0","line":3,"in_reply_to":"da20952f_e3417eaa","updated":"2015-08-31 15:36:42.000000000","message":"My find man page says:\n\n-perm +mode\n    This is no longer supported (and has been deprecated since 2005).  Use -perm /mode instead.\n\nWho knew this would be so complicated.  :)","commit_id":"bbfe9d931b6500acc09d1a93945622df1141b9b4"},{"author":{"_account_id":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},"change_message_id":"c5717c422047af05942527d01a29e190b60d25d7","unresolved":false,"context_lines":[{"line_number":1,"context_line":"#!/bin/bash"},{"line_number":2,"context_line":""},{"line_number":3,"context_line":"EXECUTABLE\u003d`find $1 -type f -perm /111`"},{"line_number":4,"context_line":"if [ -n \"$EXECUTABLE\" ]; then"},{"line_number":5,"context_line":"    echo \"Error: Executable files found: \""},{"line_number":6,"context_line":"    echo $EXECUTABLE"}],"source_content_type":"text/x-sh","patch_set":2,"id":"da20952f_a959e036","line":3,"updated":"2015-08-31 15:55:28.000000000","message":"Sadly, this is what I get on OSX BSD:\n\nThomass-MBP:cinder tbarron$ find ./tools -type f -perm /111\nfind: -perm: /111: illegal mode string\nThomass-MBP:cindertbarron$ find ./tools -type f -perm +111\n./tools/colorizer.py\n./tools/config/check_uptodate.sh\n./tools/config/generate_sample.sh\n./tools/enable-pre-commit-hook.sh\n./tools/generate_driver_list.py\n./tools/lintstack.py\n./tools/lintstack.sh\n./tools/with_venv.sh","commit_id":"cc349cf71f260044cc66e8b47fbc697be31c08f3"}]}