fgerlits commented on a change in pull request #1076:
URL: https://github.com/apache/nifi-minifi-cpp/pull/1076#discussion_r647489726
##########
File path: docker/test/integration/minifi/core/DockerTestDirectoryBindings.py
##########
@@ -77,6 +77,7 @@ def put_file_contents(file_abs_path, contents):
logging.info('Writing %d bytes of content to file: %s', len(contents),
file_abs_path)
with open(file_abs_path, 'ab') as test_input_file:
test_input_file.write(contents)
+ os.chmod(file_abs_path, 0o0777)
Review comment:
Yeah, it looks like the 0777 syntax was valid in Python 2 but it got
removed in Python 3. Maybe someone pointed out to them that having both `0123`
and `0o123` conflicts with "There's Only One Way To Do It" -- and then they
picked the wrong one to keep. :D
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
