GNOME Bugzilla – Bug 639615
Improve error message for non-existing and non-readable files
Last modified: 2011-06-25 15:55:56 UTC
Created attachment 178403 [details] [review] Patch that will improve the error message for the mentioned cases Just the other day, I tried to import some files into PiTiVi from a remotely mounted file-system. The error I got while trying to import that file was "Pipeline didn't want to go to PAUSED.". Fifteen minutes later, I finally figured that the file on the remote file-system didn't had +r for my user account. The attached patch should help users facing a similar issue in the future.
Review of attachment 178403 [details] [review]: ::: pitivi/discoverer.py @@ +416,3 @@ + if self.error: + self.info("Error: " + self.error) + elif not os.access(self.current_uri, os.R_OK): Why not just self._finishAnalysis("Couln't load file: " + self.error) ?
Will be merged "as-is" in my 0.14-localization-issues branch. I think the reason he wrote the code that way is to match the rest of the code in discoverer.py. Patrick, thanks for your contribution!
commit 74520f3e1a43b4b6b231013eed48e66385ea64b1 Author: Jean-François Fortin Tam <nekohayo@gmail.com> Date: Fri Jun 24 22:43:48 2011 -0400 Improve error message for non-existing and non-readable files