Hi,
Ok, my script review:
PREFIX=
"smbx"
That's a bit short, to avoid any possibility of collisions script_checker.sh recommends at least 5 characters, but there's no good reason nowadays not to use full names like SuperMarioBrosX
POL_Wine_PrefixCreate
"1.7.45"
You create lots of variables, you could have used one for the Wine version used...
POL_SetupWindow_pulsebar
"Installing base applications with winetricks"
"Installing"
All user-oriented messages should support localization (http://wiki.playonlinux.com/index.php/Scripting_-_Chapter_10:_Script_Translation)
cd
"$REPERTOIRE/ressources/"
...
POL_SetupWindow_download
"Downloading $appname"
"$shortname"
"$download"
...
rm
"$REPERTOIRE/ressources/$zipname"
Several things here, $REPERTOIRE is deprecated and one should use $POL_USER_ROOT instead, and POL_SetupWindow_download has been superseded by POL_Download; But most importantly since the resources cache is never trimmed, scripts should only put reusable files there, validated thru the use of a hash as in POL_Download_Resource. That's exactly the wrong place for a temporary file, either put it in $POL_USER_ROOT/tmp, or use POL_System_TmpCreate.
bash
winetricks -q vb6run
Use of winetricks is scripts has been obsoleted since at least when I learned about PlayOnLinux, almost four years ago. It can't say before that. Anyway, all those "verbs" have their "components" counterparts, just use POL_Call POL_Install_vbrun6, etc.
mkdir
-p
"$REPERTOIRE/wineprefix/$shortname/drive_c/$shortname"
Don't hardcode virtual drive path $POL_USER_ROOT/wineprefix/$PREFIX, use $WINEPREFIX instead
unzip -o
"$REPERTOIRE/ressources/$zipname"
-d
"$REPERTOIRE/wineprefix/$shortname/drive_c/$shortname"
You can use POL_System_unzip instead of unzip, the added benefit is that in case of error in unzip, the user will be warned instead of the error being silently ignored.
That's it, but the use of winetricks and or the resources cache for temporary files are both no-gos for adding it to the official scripts repository.
Regards,
Pierre.