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

Fixer cannot overwrite source file #5

Closed
CommonGuy opened this issue Jan 26, 2017 · 1 comment
Closed

Fixer cannot overwrite source file #5

CommonGuy opened this issue Jan 26, 2017 · 1 comment
Labels

Comments

@CommonGuy
Copy link

The appcache-manifest-fixer cannot overwrite the source file. If you attempt to do it, the source file will be empty afterwards.

I call the appcache-manifest-fixer like this:

appcache-manifest-fixer index.html -o index.html

Instead of adding the manifest-attribute, index.html is now completely empty.

@mysticatea
Copy link
Owner

Thank you for this report. And I'm sorry for my late.

appcache-manifest-fixer command was designed to use to make production code. So I did not think that the output file path is the same as the input file path.

I will add assertion check for this to avoid lost files.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants