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

ValueError: options should not be an empty list #1

Open
shvchk opened this issue Oct 15, 2022 · 1 comment · May be fixed by #2
Open

ValueError: options should not be an empty list #1

shvchk opened this issue Oct 15, 2022 · 1 comment · May be fixed by #2

Comments

@shvchk
Copy link

shvchk commented Oct 15, 2022

% sudo ntfs-rw
Traceback (most recent call last):
  File "/Users/x/.local/bin/ntfs-rw", line 8, in <module>
    sys.exit(main())
  File "/Users/x/.local/pipx/venvs/osx-ntfs-rw/lib/python3.10/site-packages/ntfs_rw/main.py", line 53, in main
    disk = select_disk()
  File "/Users/x/.local/pipx/venvs/osx-ntfs-rw/lib/python3.10/site-packages/ntfs_rw/main.py", line 47, in select_disk
    selected, _ = pick(disks)
  File "/Users/x/.local/pipx/venvs/osx-ntfs-rw/lib/python3.10/site-packages/pick/__init__.py", line 211, in pick
    picker: Picker = Picker(
  File "<string>", line 12, in __init__
  File "/Users/x/.local/pipx/venvs/osx-ntfs-rw/lib/python3.10/site-packages/pick/__init__.py", line 50, in __post_init__
    raise ValueError("options should not be an empty list")
ValueError: options should not be an empty list
shvchk added a commit to shvchk/osx-ntfs-rw that referenced this issue Oct 15, 2022
@shvchk shvchk linked a pull request Oct 15, 2022 that will close this issue
@shvchk
Copy link
Author

shvchk commented Oct 15, 2022

Also, disk type can contain spaces:

1:       Microsoft Basic Data ⁨easystore⁩               2.0 TB     disk6s1

This means result of a split in src/ntfs_rw/main.py#L18 might not always be as expected.

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 a pull request may close this issue.

1 participant