Skip to content

Prevent failure when no hex files in download_path#1

Merged
tomviner merged 1 commit intotomviner:masterfrom
Geekfish:bugfix/handle-no-hex
Dec 22, 2015
Merged

Prevent failure when no hex files in download_path#1
tomviner merged 1 commit intotomviner:masterfrom
Geekfish:bugfix/handle-no-hex

Conversation

@Geekfish
Copy link
Contributor

Some times I would clean up my downloads folder and upyflashed would grumpily fail. This will keep it running and log instead.

@tomviner
Copy link
Owner

Thanks Eleni!

tomviner added a commit that referenced this pull request Dec 22, 2015
Prevent failure when no hex files in download_path
@tomviner tomviner merged commit d2bed4e into tomviner:master Dec 22, 2015
@tomviner
Copy link
Owner

@Geekfish you may want to install the latest from master/pypi as I found the short circuit loop left my lap(top) getting a bit hot! So I refactored the loop to ensure the sleep always happens.

@Geekfish
Copy link
Contributor Author

Oops 🙈 !! Thanks @tomviner !

@Geekfish
Copy link
Contributor Author

Actually I'm having some trouble installing the latest version from pypi:

$ pip install -U upyflashed --no-cache-dir
Collecting upyflashed
  Downloading upyflashed-1.1.13.tar.gz
    Complete output from command python setup.py egg_info:
    Traceback (most recent call last):
      File "<string>", line 20, in <module>
      File "/private/var/folders/ll/_r6fwzs12mbg115frb4rlk6r0000gn/T/pip-build-LpzGDE/upyflashed/setup.py", line 15, in <module>
        long_description=read('README.rst') + read('CHANGES.rst'),
      File "/private/var/folders/ll/_r6fwzs12mbg115frb4rlk6r0000gn/T/pip-build-LpzGDE/upyflashed/setup.py", line 6, in read
        return open(os.path.join(os.path.dirname(__file__), fname)).read()
    IOError: [Errno 2] No such file or directory: '/private/var/folders/ll/_r6fwzs12mbg115frb4rlk6r0000gn/T/pip-build-LpzGDE/upyflashed/CHANGES.rst'

    ----------------------------------------
Command "python setup.py egg_info" failed with error code 1 in /private/var/folders/ll/_r6fwzs12mbg115frb4rlk6r0000gn/T/pip-build-LpzGDE/upyflashed

tomviner added a commit that referenced this pull request Dec 22, 2015
@tomviner
Copy link
Owner

@Geekfish thanks for the report! try again now

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