10BC0 Merge pull request #1250 from stonebig/master · winpython/winpython@a1d06ff · GitHub
[go: up one dir, main page]

Skip to content

Commit a1d06ff

Browse files
authored
Merge pull request #1250 from stonebig/master
fixing bad logic
2 parents 509f2cf + 73fb067 commit a1d06ff

File tree

1 file changed

+13
-3
lines changed

1 file changed

+13
-3
lines changed

winpython/utils.py

Lines changed: 13 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -215,7 +215,17 @@ def get_winpython_start_menu_folder(current=True):
215215
)
216216
return str(Path(folder) / 'WinPython')
217217

218-
218+
def remove_winpython_start_menu_folder(current=True):
219+
"""Create WinPython Start menu folder -- remove it if it already exists"""
220+
path = get_winpython_start_menu_folder(current=current)
221+
if Path(path).is_dir():
222+
try:
223+
shutil.rmtree(path, onerror=onerror)
224+
except WindowsError:
225+
print(
226+
f"Directory {path} could not be removed",
227+
file=sys.stderr,
228+
)
219229

220230
def create_winpython_start_menu_folder(current=True):
221231
"""Create WinPython Start menu folder -- remove it if it already exists"""
@@ -228,8 +238,8 @@ def create_winpython_start_menu_folder(current=True):
228238
f"Directory {path} could not be removed",
229239
file=sys.stderr,
230240
)
231-
else:
232-
os.mkdir(path)
241+
# create, or re-create !
242+
os.mkdir(path)
233243
return path
234244

235245

0 commit comments

Comments
 (0)
0