tyler.durden
/
audk
mirror of https://github.com/acidanthera/audk.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

.pytool/Plugin: UncrustifyCheck: use stat instead of os.stat 

The UncrustifyCheck plugin passes os.stat.S_IWRITE to os.chmod, when
attempting to change file permissions. os.stat.S_IWRITE does not exist
as os.stat is a function. The correct value is stat.S_IWRITE.

Signed-off-by: Joey Vagedes <joey.vagedes@gmail.com>
Cc: Liming Gao <gaoliming@byosoft.com.cn>
Cc: Michael D Kinney <michael.d.kinney@intel.com>
Cc: Sean Brogan <sean.brogan@microsoft.com>
Reviewed-by: Michael D Kinney <michael.d.kinney@intel.com>
This commit is contained in:
Joey Vagedes 2024-01-22 14:41:15 -08:00 committed by Savva Mitrofanov
parent 69dd3faa16
commit 30334126c7
1 changed files with 2 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
.pytool/Plugin/UncrustifyCheck/UncrustifyCheck.py
View File

@ -12,6 +12,7 @@ import logging
import os import os
import pathlib import pathlib
import shutil import shutil
import stat
import timeit import timeit
from edk2toolext.environment import version_aggregator from edk2toolext.environment import version_aggregator
from edk2toolext.environment.plugin_manager import PluginManager from edk2toolext.environment.plugin_manager import PluginManager
@ -628,7 +629,7 @@ class UncrustifyCheck(ICiBuildPlugin):
""" """
Private function to attempt to change permissions on file/folder being deleted. Private function to attempt to change permissions on file/folder being deleted.
""" """
os.chmod(path, os.stat.S_IWRITE) os.chmod(path, stat.S_IWRITE)
func(path) func(path)
for _ in range(3): # retry up to 3 times for _ in range(3): # retry up to 3 times