mctwist / addonpacker Goto Github PK

Currently when an error occurs, it will just exit the script without telling the user what went wrong. Adding some sort of way to tell the user what happened would reduce the pain to find out what actually went wrong.

A suggestion is to add a name validation. This means that whenever a name for an archive is incorrect, it will either prompt the user for a change, or just output and ignore it.

The reason for this script is mostly to make it easy to use and handle. However, it might come down that supporting two types of scripts might make it quite troublesome for the user to just "use" it without having to rely on third party programs to run it.

One could create an executable for each platform and still give out a pack.cmd to pick one of them, unless the user knows what they want, which in most cases is not true.

I think that using Python along with PyInstaller could be a start to look first.

This would also make it a lot easier to handle the rough differences from bash and batch which will be more prominent when more features are added. An example could be the lack of realpath, which already exists in Python. That function is required for bash as 7zip somehow cannot handle relative paths.

Even better, it is therefore possible to ditch 7zip altogether and make the script even easier to install as Python already have a library for it that we might have to do ourselves, but in the end it is much easier to modify.