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

Unhandled Exception when Updating #4

Closed
GoogleCodeExporter opened this issue Jun 15, 2015 · 4 comments
Closed

Unhandled Exception when Updating #4

GoogleCodeExporter opened this issue Jun 15, 2015 · 4 comments

Comments

@GoogleCodeExporter
Copy link

When boar can't update a file, an unhandled exception occurs and stops the 
process.

What steps will reproduce the problem?
1. Attept to update a directory containing files with the System and Hidden 
attributes.
2. Observe the exception.

What is the expected output? What do you see instead?
I expect the action taken to be configurable, (ie. Can't update Thumbs.db, 
Skip/All/Ignore/Fail)


The Traceback is as follows:
Updating: photos/2009/100CANON/Thumbs.db
Traceback (most recent call last):
  File "C:\Python26\boar\boar", line 498, in <module>
    main()
  File "C:\Python26\boar\boar", line 470, in main
    cmd_update(args[1:])
  File "C:\Python26\boar\boar", line 280, in cmd_update
    wd.update(new_revision = options.revision)
  File "C:\Python26\boar\workdir.py", line 141, in update
    fetch_blob(front, b['md5sum'], target_abspath, overwrite = True)
  File "C:\Python26\boar\workdir.py", line 482, in fetch_blob
    f = open(target_path, "wb")
IOError: [Errno 13] Permission denied: 
u'E:\\Pictures\\photos/2009/100CANON/Thumbs.db'

What platform are you using? (Windows XP, Windows 7, Linux, ...)
Windows 7 Ultimate

What version of Python are you using?
Python 2.6.5

What version of boar are you using? (Mercurial change id or daily build date)
Daily Build 2/13


Original issue reported on code.google.com by fleeingt...@gmail.com on 21 Feb 2011 at 3:00

@GoogleCodeExporter
Copy link
Author

Agreed. This needs to be handled more gracefully. I think a quick fix will be 
to add a "--force" option, to simply ignore all errors, to allow for a way to 
proceed for users bumping into this issue. Not so elegant, but it will at least 
be quick to implement. I'll also open a new issue regarding the best way to 
handle these kinds of errors.

Original comment by ekb...@gmail.com on 24 Feb 2011 at 3:01

  • Changed state: Accepted

@GoogleCodeExporter
Copy link
Author

Fixed in changeset 296:a3acaa6dc936

There is now a --ignore option to 'update' that makes the process continue 
after errors. Also, any errors are presented in a nicer way (not as stack 
dumps). 

Please verify that the fix works.

Original comment by ekb...@gmail.com on 27 Feb 2011 at 4:26

  • Changed state: Fixed

@GoogleCodeExporter
Copy link
Author

I'm currently on vacation, but I will have a chance to try the new fix
on Monday. Thanks!

Original comment by fleeingt...@gmail.com on 2 Mar 2011 at 2:13

@GoogleCodeExporter
Copy link
Author

Assuming verified.

Original comment by ekb...@gmail.com on 20 Aug 2011 at 9:48

  • Changed state: Verified

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant