CAUTION Somebody broke "bench" last night

ankush · August 8, 2022, 4:22pm

I actually didn’t fully understand the question and assumed it was about annoyance that warning causes.

Most if not all bench commands don’t have stable stdout, so using stdout in scripts like this is bound to break.

Feel free to fix it if it has been bothering you for so long, here’s the reference code that you need to modify:

frappe/bench/blob/develop/bench/cli.py#L46


      
          verbose = False
          is_envvar_warn_set = None
          from_command_line = False  # set when commands are executed via the CLI
          bench.LOG_BUFFER = []
          
          change_uid_msg = "You should not run this command as root"
          src = os.path.dirname(__file__)
          SKIP_MODULE_TRACEBACK = ("click",)
          
          
          @contextmanager
          def execute_cmd(check_for_update=True, command: str = None, logger: Logger = None):
          	if check_for_update:
          		atexit.register(check_latest_version)
          
          	try:
          		yield
          	except BaseException as e:
          		return_code = getattr(e, "code", 1)
          
          		if isinstance(e, Exception):

Quite a lot coming from someone whose every other message is passive-aggressive. I am enlightened, thanks!

**CAUTION** Somebody broke "bench" last night

CAUTION Somebody broke "bench" last night