UncrustifyCheck: use stat instead of os.stat

[edk2-devel] [PATCH v1 0/1] UncrustifyCheck: use stat instead of os.stat

Joey Vagedes via groups.io posted 1 patch 3 months ago

Download series mbox

Failed in applying to current master (apply log)

.pytool/Plugin/UncrustifyCheck/UncrustifyCheck.py | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)

Expand all Fold all

[edk2-devel] [PATCH v1 0/1] UncrustifyCheck: use stat instead of os.stat

Posted by Joey Vagedes via groups.io 3 months ago

When UncrustifyCheck attempts to update the permissions of a file
( Only happens when a different error occurs ), it incorrectly uses
os.stat.S_IWRITE, which does not exist. The correct value is 
stat.S_IWRITE.

see os.chmod documentation: https://docs.python.org/3/library/os.html#os.chmod

Cc: Liming Gao <gaoliming@byosoft.com.cn>
Cc: Michael D Kinney <michael.d.kinney@intel.com>
Cc: Sean Brogan <sean.brogan@microsoft.com>

Joey Vagedes (1):
  .pytool/Plugin: UncrustifyCheck: use stat instead of os.stat

 .pytool/Plugin/UncrustifyCheck/UncrustifyCheck.py | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

-- 
2.40.1.vfs.0.0



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#114152): https://edk2.groups.io/g/devel/message/114152
Mute This Topic: https://groups.io/mt/103898980/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-