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.

Do I install the codecs like I would to MPlayer? (as in download the codecs provided by mplayer?) Or do you have your own codecs pack somewhere else?

You might want to remove the "latest version" link. I almost clicked it but then I saw the Rev 4 link.

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

vts-files... when i open a dvd from my harddrive, i have to select in the menu, which vts-file i want to play. therefore the player stops playing the film after this only file although a dvd-film consits of several vts-files. what do i have to adjust, that the smplayer plays the whole film without brakes such as the windows media player and other media player do. (by the way: the same problem i have with the kmplayer)

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.