ScrollPrize / vesuvius

Python library for accessing Vesuvius Challenge data
MIT License
13 stars 1 forks source link

vesuvius.accept_terms creates a bad path to the agreement file. #6

Open righthalfplane opened 3 days ago

righthalfplane commented 3 days ago

vesuvius.accept_terms creates a good path to the agreement file on MacOS 12.7.3 vesuvius.accept_terms creates a bad path to the agreement file on Ubuntu 24.10 and Windows. On Ubuntu 24.10 the path is to a python2 type location.

On Windows 10 the path is 'C:\Program Files\Python310\lib\site-packages\vesuvius\setup\agreement.txt' and that is not even a legal file name path. The path should actually be in the "C:\Users\Owner\AppData\Roaming\Python\Python310\site-packages\" area - since that it where everything else has been put. "C:/Program Files/Python310/lib/site-packages/vesuvius/setup/agreement.txt" would be a global location for the file - except that a local install of the vesuvius was done.

Here is the Windows 10 traceback -

Traceback (most recent call last): File "C:\Program Files\Python310\lib\runpy.py", line 196, in _run_module_as_main return _run_code(code, main_globals, None, File "C:\Program Files\Python310\lib\runpy.py", line 86, in _run_code exec(code, run_globals) File "c:\users\owner\appdata\roaming\python\python310\Scripts\vesuvius.accept_terms.exe__main__.py", line 7, in File "C:\Users\Owner\AppData\Roaming\Python\Python310\site-packages\vesuvius\setup\accept_terms.py", line 61, in main display_terms_and_conditions(args.yes) File "C:\Users\Owner\AppData\Roaming\Python\Python310\site-packages\vesuvius\setup\accept_terms.py", line 52, in display_terms_and_conditions save_agreement() File "C:\Users\Owner\AppData\Roaming\Python\Python310\site-packages\vesuvius\setup\accept_terms.py", line 24, in save_agreement with open(agreement_file_path, 'w+') as file: PermissionError: [Errno 13] Permission denied: 'C:\Program Files\Python310\lib\site-packages\vesuvius\setup\agreement.txt'