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

Multiple files not working #33

Closed
sdekrijger opened this issue Feb 6, 2015 · 1 comment
Closed

Multiple files not working #33

sdekrijger opened this issue Feb 6, 2015 · 1 comment

Comments

@sdekrijger
Copy link

Hi,
There seems to be an issue with
$ pyminifier --dest-dir=/dev/shm ./*.py

The only output is:

Created by pyminifier (https://github.com/liftoff/pyminifier)

On commit 2ed2eaf it's working fine, after that it's the above.

Br,
Sjef.

liftoff added a commit that referenced this issue Feb 6, 2015
…fier` will work just like running '/usr/local/bin/pyminifier'.

Fixed issue #33 where minifying multiple files into a destination dir (`--destdir`) was not working properly.  Thanks Sjef de Krijger for emailing me a patch.  His patch also fixed the `--prepend` option which wasn't working properly when minifying/obfuscating multiple files.
__init__.py:  Added some of the recently-added features to the global docstring.
__init__.py:  Added a docstring to the `pyminify()` function.
__main__.py:  Added a docstring to the `main()` function.
analyze.py:  The `enumerate_imports()` and `enumerate_global_imports()` functions now skip 'from' import lines like, `from foo import bar` since those aren't being handled correctly by the `enumerate_local_modules()` function inside of obfuscate.py.  This is just a temporary measure until I get the logic in order for differentiating between "from whatever import foo" and having a "foo.py" in the same directory.
obfuscate.py:  Fixed a bug where variables being passed as arguments (right of an equal sign) were not being obfuscated when they should be.
All over:  Modernized some strings here and there by converting them to use `format()` and cleaned up a few things to improve code readability/presentation.
@liftoff
Copy link
Owner

liftoff commented Feb 6, 2015

This should be all fixed now. Can you grab the latest code and confirm? Thanks.

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

No branches or pull requests

2 participants