dennisvang / tufup

Automated updates for stand-alone Python applications.
MIT License
71 stars 1 forks source link

App name cannot contain whitespace #84

Closed dennisvang closed 8 months ago

dennisvang commented 8 months ago

Description

By design, TargetMeta does not support app names with whitespace.

However, this is not mentioned explicitly in the code, nor is it mentioned in the examples, nor in the readme.

Moreover, whitespace in the app name will cause version parsing to fail silently (only logged at DEBUG level).

This may lead to issues like https://github.com/dennisvang/tufup-example/issues/18, and it is also mentioned in https://github.com/dennisvang/tufup-example/issues/9.

Note that the restriction is implicit in the following lines from the README, and there's also a testcase for whitespace:

Archive filenames and patch filenames follow the pattern

<name>-<version><suffix>

where name is a short string that may contain alphanumeric characters, underscores, and hyphens, ...

Solution