SMPlayer Portable Development Test 4 (DevTest 5) (I should say Testers Needed)

Uh oh....better take this one down lol, John was gonna do this one.

Corrupt

After multiple attempts to download both the installer and zip versions; reported as corrupt.

This looks pretty good, here's some changes to get it closed to an official release:

• Add an AppInfo.ini to App\AppInfo using AppInfo.ini from a recent release as a template.
• I'm not sure whether their package came compressed, or whether you UPXed it, but if you did, there's some other files that can still be UPXed to save a couple megs. If it runs okay after doing this, keep it. If it doesn't revert it. Download pangout.bat, upax.bat, and cleanup.bat. Save all of those files to C:\WINDOWS if you're using XP. Next, download UPX and copy UPX.exe to C:\WINDOWS and also download PNGOUT and save it to C:\WINDOWS. Next, open the command prompt cd to your app directory (Not App Portable, App\appname) and type upax wait till compression is done, them pangout and when all is done type cleanup.
• You can add your copyright below Johns in the source.
• Rename App\SMPlayer to App\smaplayer.
• The LegalCopyright entry in the source should be 'PortableApps.com and contributers' as apposed to John T. Haller.
• Below line 210 of the source, which is:

  ${ReplaceInFile} "$PROGRAMDIRECTORY\SMPlayer.ini" ':$LASTDRIVE:\' ':$CURRENTDRIVE:\'

  I don't think CURRENTDRIVE and LASTDRIVE should precede with a colon, only a slash as the last drive letter in the portable settings is also written with a colon.

• You can add your copyright below Johns in the readme as well, and change the website.
• ...Development_Test_1_en-us in the installer, don't forgot that though we all do.
• Your default settings file SMPlayer.ini with things like Screenshot dir set to ..\..\Data should be in App\DefaultData instead, so use the code from another app that uses defaultdata to copy the file from their is the settings aren't found in the Data or program dir.
• The icon needs work, I can help you with that on IRC or something, too tired right now.
• Add a help.html file, which also requires Other\Help. You can edit the file with something like KompoZer. Also, take the readme.txt from App\ from one of the new releases and put it in the same dir on your app.

That's basicially it, you did really great on this and I'm pretty impressed, good work!

Does this include the codecs pack? If so, it should be pulled from SourceForge ASAP as it includes numerous closed source binaries (which are a no no on SF). The main SMPlayer SF project has pulled theirs already.

doesn't work.
First you must change rmdir to RMDir and the homepath variable should be something like "C:$%HOMEPATH%\.dvdcss" but even that doesn't work. Maybe its because the folder isn't empty? I dunno. Maybe Wraithdu does

For me it works fine.

good work
NeoRame

wget didn't "get" the file. I can grab it fine using a browser so it's not my connection that the issue.

You need to add a notification to the user on what you're doing when wget runs. Use a popup box to let them know you're grabbing the CODECs for them. Users may close the box since you didn't inform them. How about launching a batch file in the \App\wget dir that tells the user what's going on with echo statements and then runs wget?

wget should be in the \App\wget directory, not as a "top level" directory. All files that execute have been put in the \App dir in the past.

EDIT: correction - wget DID get the file and it's in my $TEMP directory, but it didn't get extracted to $TEMP\windows-essential-20071007 and the contents of that dir wasn't copied over to the target directory as specified in the source code.