Ticket #10322 (closed: fixed)
Mantid does not start if there is an inaccessible dir in the save or search paths
Reported by: | Martyn Gigg | Owned by: | Nick Draper |
---|---|---|---|
Priority: | critical | Milestone: | Release 3.3 |
Component: | Framework | Keywords: | |
Cc: | Blocked By: | ||
Blocking: | #10484 | Tester: | Federico M Pouzols |
Description
Observed on Windows 7 where an inaccessible network drive was set as the defaultsave.directory. Mantid takes a long time to start and then throws up an Python initialization error saying there has been a file access problem.
Change History
comment:1 Changed 6 years ago by Nick Draper
- Owner set to Nick Draper
- Status changed from new to assigned
comment:2 Changed 6 years ago by Nick Draper
- Status changed from assigned to inprogress
re #10322 Added error handling around a directory checking routine
Changeset: da8423bf22cc99bfd7e3a194cfc19ceea310dea0
comment:3 Changed 6 years ago by Nick Draper
- Status changed from inprogress to verify
- Resolution set to fixed
Instructions to test are aimed at Windows (you might be able to do something similar on linux)
To Test
- Share a folder on your computer as z:
- Map a drive to that share
- Go back to the folder, select properties, sharing, advanced sharing and untick the box that says share this folder.
- You now have an invalid network drive
- Open Mantid.user.properties and set DefaultSave.Directory = Z:/
- Open Mantidplot, it should open successfully, and python should still work
- Undo the mess you have made!
comment:4 Changed 6 years ago by Nick Draper
re #10322 Added the exception include, strange win didn't need this
Changeset: 56a69933773fcf6590d7a3bab0c1ca7cfa33ad12
comment:6 Changed 6 years ago by Federico M Pouzols
- Status changed from verify to verifying
- Tester set to Federico M Pouzols
comment:7 Changed 6 years ago by Federico M Pouzols
- Status changed from verifying to closed
I did the suggested mess test (Windows 7) and it works. It handles the mess gracefully, the broken path/drive is ignored.
The new code looks clear and does the job.
comment:8 Changed 6 years ago by FedeMPouzols
Merge remote-tracking branch 'origin/feature/10322_Mantid_will_not_start_with_bad_path'
Full changeset: 92355232fca526480e698b2f6f4a231e57cf9570