Skip to content

Conversation

@penguinpee
Copy link
Contributor

  1. Duplicate output of results
  2. Missing white space in description

1. Duplicate output of results
2. Missing white space in description
@penguinpee
Copy link
Contributor Author

penguinpee commented Jul 5, 2025

The duplication looks as follows:

$ fastflix -m /tmp/foo.png 
'/tmp/foo.png' : image/png
'/tmp/foo.png' : image/png

It's due to command_line_entry() being called twice.

The missing white space is due toe argparse concatenating the two strings:

$ fastflix --help
usage: fastflix [-h] [-m] [-v] files [files ...]

puremagic is a pure python file identification module.It looks for matching magic numbers in the file to locate the file type.

Adding a space at the end of the first string would also work.

@penguinpee
Copy link
Contributor Author

Oh, btw, why is the command called fastflix? Just curious.

@cdgriffith cdgriffith changed the base branch from master to develop July 5, 2025 14:19
@cdgriffith
Copy link
Owner

Oh, btw, why is the command called fastflix? Just curious.

That would be because I'm an idiot, and when changing from setup.py to pyproject.toml yesterday I copied plenty from my other project, and missed that change!

Thank you for letting me know about this!

@cdgriffith cdgriffith merged commit cfe7458 into cdgriffith:develop Jul 5, 2025
@cdgriffith cdgriffith mentioned this pull request Jul 5, 2025
cdgriffith added a commit that referenced this pull request Jul 5, 2025
* Fix #112 command line tool text (thanks to Sandro)

---------

Co-authored-by: Sandro <shfu29r4bu@liamekaens.com>
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