Skip to content

Comments

catch the securesystemslib exception add supply advice#877

Closed
lixuefeng2 wants to merge 1 commit intotheupdateframework:developfrom
lixuefeng2:for_repo
Closed

catch the securesystemslib exception add supply advice#877
lixuefeng2 wants to merge 1 commit intotheupdateframework:developfrom
lixuefeng2:for_repo

Conversation

@lixuefeng2
Copy link
Contributor

Signed-off-by: lixuefeng (Cloud) li.xuefeng@h3c.com

Please fill in the fields below to submit a pull request. The more information
that is provided, the better.

Fixes issue #:

Description of the changes being introduced by the pull request:
Sometimes for starter, the securesystemslib may failed to installed correctly. Init the repo shall get hard-to-understand exception message. This modification is to guide user to solve this problem.

Signed-off-by: lixuefeng (Cloud) <li.xuefeng@h3c.com>
@lixuefeng2
Copy link
Contributor Author

@sebastien , It seems the CI has problems? I don't know why it just only raise exception in python 3.6.
PermissionError: [WinError 32] The process cannot access the file because it is being used by another process: 'C:\projects\tuf-2aew1\tests\tmpv9tr3to2\TestReplayAttack_x_lr16lz\repository\metadata\timestamp.json'

@lukpueh
Copy link
Member

lukpueh commented May 27, 2019

The error indeed seems unrelated. @awwad, have you seen this before?

except (securesystemslib.exceptions.UnsupportedLibraryError) as e:
sys.stderr.write('Error: ' + str(e) + '\n')
sys.stderr.write('resolve this problem by:\n')
sys.stderr.write('sudo pip install securesystemslib[crypto,pynacl]\n')
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think it's better to point the user to the TUF installation instructions here.

@lukpueh
Copy link
Member

lukpueh commented Sep 17, 2019

Closing due to inactivity. Re-visit with #811.

@lukpueh lukpueh closed this Sep 17, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants