Why do you expect us to tell you what you're doing wrong if you're not even telling us what "exception" you get?

Also, you were advised to use Archive::Zip. Why do you shell out to the zip program instead?