Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

killdir error message when exporting mac os x package from distribute game #1084

Closed
ohrrpgce-bugbot opened this issue Jun 30, 2016 · 1 comment
Labels
bug Yeah... that's broken file I/O Low-level file access problems, often OS-specific or OPENFILE internals os: mac Specific to Macs packaging Packaging/installing the engine or games (Distribute Game) rel: dwimmercrafty Present in dwimmercrafty 2017-08-31 / D+1 2017-09-22 rel: New in ... Introduced in the tagged "rel:" release/nightlies resolved: works for me Couldn't be reproduced or disappeared on its own

Comments

@ohrrpgce-bugbot
Copy link

[sf#2025]

When you export a Mac OS X app from the Distribute Game menu, there is an error message that says:

"killdir: refusing to delete directory '/home/james/.ohrrpgce/_gameplayer/OHRRPGCE-Game.app'"

https://i.imgur.com/581gPy4.png

As far as I can tell, this error does not actually break anything, it is just annoying (?)

I am guessing that this was probably the only code that actually relied on killdir's recently eliminated recursive delete feature.

From: @bob-the-hamster
Reported version: Nightly WIP (Dwimmercrafty)
Operating system: Linux

@ohrrpgce-bugbot ohrrpgce-bugbot added bug Yeah... that's broken os: mac Specific to Macs labels Mar 14, 2020
@rversteegen rversteegen added file I/O Low-level file access problems, often OS-specific or OPENFILE internals packaging Packaging/installing the engine or games (Distribute Game) rel: dwimmercrafty Present in dwimmercrafty 2017-08-31 / D+1 2017-09-22 rel: New in ... Introduced in the tagged "rel:" release/nightlies labels Mar 15, 2020
@rversteegen
Copy link
Contributor

Actually, recursive deletion was never removed from killdir. That error message was added in r7997 (24e91ed)

+  ' For safety. (You ought to pass absolute paths.) Check
+  ' writability so we don't recurse if started from e.g. /home until
+  ' we hit something deletable (this happened to me)!
+  IF LEN(directory) < 5 ORELSE diriswriteable(directory) = NO THEN
+   showerror "killdir: refusing to delete directory '" & directory & "'"
+   EXIT SUB
+  END IF

(Since reworded to "killdir: refusing suspicious attempt to delete directory")

But no sign of the error now, 3 1/2 years later, so I'll close this.

@rversteegen rversteegen added the resolved: works for me Couldn't be reproduced or disappeared on its own label Mar 15, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Yeah... that's broken file I/O Low-level file access problems, often OS-specific or OPENFILE internals os: mac Specific to Macs packaging Packaging/installing the engine or games (Distribute Game) rel: dwimmercrafty Present in dwimmercrafty 2017-08-31 / D+1 2017-09-22 rel: New in ... Introduced in the tagged "rel:" release/nightlies resolved: works for me Couldn't be reproduced or disappeared on its own
Projects
None yet
Development

No branches or pull requests

2 participants