View Single Post
Old 10-05-2017, 01:57 PM   #233
DiapDealer
Grand Sorcerer
DiapDealer ought to be getting tired of karma fortunes by now.DiapDealer ought to be getting tired of karma fortunes by now.DiapDealer ought to be getting tired of karma fortunes by now.DiapDealer ought to be getting tired of karma fortunes by now.DiapDealer ought to be getting tired of karma fortunes by now.DiapDealer ought to be getting tired of karma fortunes by now.DiapDealer ought to be getting tired of karma fortunes by now.DiapDealer ought to be getting tired of karma fortunes by now.DiapDealer ought to be getting tired of karma fortunes by now.DiapDealer ought to be getting tired of karma fortunes by now.DiapDealer ought to be getting tired of karma fortunes by now.
 
DiapDealer's Avatar
 
Posts: 28,760
Karma: 206758686
Join Date: Jan 2010
Device: Nexus 7, Kindle Fire HD
It looks to be OK at first glance. You may want to add a try/except clause around the os.remove() call so that it can fail gracefully, though. An exception will be raised on Windows if the file in question happens to be open. Better safe than sorry.

Also ... I'm sure there's probably more "allowable" unmanifested files that will only become apparent after you've released your plugin (not that I can think of any offhand). A way for the user to exempt specific files (or file-types) might stave off future trouble.
DiapDealer is offline   Reply With Quote