Lightscreen Portable Development Test 7 (Testers Needed) *Outdated*

this probably should have been a continuation of the first thread, instead of a new thread...glad to see the update though

it's not a big deal, but i was just wondering why the pics would be stored in Data\settings. pics aren't settings; i would think they could go in the main Data folder, which would also make them easier for users to find. the pictures folder was a good idea btw -to either you or patrick. yes i would vote for the screenshots folder in pictures

setting an ini file to tell you if it's the first run or not isn't really necessary. All you have to do is a IfFileExists "$SETTINGSDIRECTORY\lightscreen_portable.reg" command and that will do the exact same thing without all the ini read/write code. Also, just curious: what happens if someone actually selects the "Start Lightscreen with Windows" option? i would think that could cause problems but maybe not. if so, then it might be a good idea to undo any changes that the option does, if they select it, on script exit.

For the Development Test page, can I use your real name, or should I keep it set to your user ID?

Placing the LightscreenPortable.ini in the main folder does nothing. I cannot disable the splashscreen.

Edit: I also took a screenshot with the timestamp as the file name and I got this: screenshot.1207938157.jpg Those just look like random numbers to me. The date for this picture is April 11, 2008 at 2:20pm so shouldn't it look something like this: screenshot.04.11.2008.1420.jpg? It seems like this doesn't work correctly.

I found 2 registry entries left on the system after running Lightscreen. The are both located under HKCU\Software\Trolltech\OrganizationDefaults. The launcher should handle this as well and it should also work with the ini.

Can you get your "Alternative Download" uploaded? skydrive and dropio and similar services are blocked here at work and I'd really like to test your changes.

Just a question, what was the purpose of writing your own installer?

The ini works now, but in order for this to be more "official" the ini should be in the root directory. This would work the same way that all the other portable apps work. If you look at the code for the 7-Zip launcher you'll see what I mean.

I'm by no means a great developer. In fact, there's some code I'm absolutely not proud of. But . . . I offer help when I can- just beware to take everything with a grain of salt.

IfFileExists "$EXEDIR\Data\${DEFAULTSETTINGSPATH}\${NAME}Settings.ini" INIEXISTS NoINI

Is checking into your data folder- it should just be the $EXEDIR.

NoINI:
			;== Create INI File
				CreateDirectory "$EXEDIR\Data\${DEFAULTSETTINGSPATH}"
				WriteINIStr "$EXEDIR\Data\${DEFAULTSETTINGSPATH}\${NAME}Settings.ini" "${NAME}Settings" "ShowSplashScreen" "true"
			;== Create Folders If The Dont Exist
				CreateDirectory "$EXEDIR\Data"
				CreateDirectory "$EXEDIR\Data\settings"
				GoTo INIEXISTS

I'm a little confused why you hardcoded the ini creation into your launcher.
That's all I can see as far as your splash problem. Just stick a copy of the ini in the source directory.

As far as the reg import/export/backup, I stop there. I don't have any experience in that area. But I will take a look and see if I see any Obvious errors.
Questions:
Do you all think the image directory for the screenshots should be Documents\Pictures\Screenshots? Yes- but only if checks to see if that directory is present. Else- you are creating a bunch of folders which somepeople may not have. And I agree that it's annoying.

I just solved your DisplaySplashScreen issue! your LightScreenPortable.ini had

[LightscreenPortable]
Lightscreen=App\Lightscreen
SettingsDirectory=Data\settings
LightscreenExecutable=lightscreen.exe
AdditionalParameters=
DisableSplashScreen=true

but it should have been

[LightscreenPortable]
LightscreenDirectory=App\Lightscreen
SettingsDirectory=Data\settings
LightscreenExecutable=lightscreen.exe
AdditionalParameters=
DisableSplashScreen=true

Also, you had some unneeded code in your launcher. I have removed it, and will upload my fixes momentarily.

Edit: Here it is. Good work on getting this going, it's a great application. I added another entry to LightscreenPortable.ini as well, the user can now set a custom screenshot path via the ini. Relative paths are accepted because the path is created relative to $EXEDIR

I haven't checked the registry at all, but it works well in vista ultimate sp1, and doesn't leave anything obvious left on the hard drive

Looking good now I'd say this is just about ready to go. May I suggest you contact the author of the base application and mention the issue with appending the timestamp to the file name? It should be giving something like year.month.day.time, but it looks like it's just a random series of numbers.

I made another quick fix to fix something that was an issue on my part. The IfErrors NoINI that I told you to move yesterday, really was in the right spot the first time. Should I upload my fix, or can you make another release that corrects it?

I've just tested Dev Test 7 in Vista Ultmate SP1 and XP Pro sp2 and I couldn't find any obvious problems.

I might as well put my 2 cents in for the default save folder now (but if it's too late just ignore it). I would prefer to see it default to Document\Pictures

To be the same as the other apps on this site, the reg key should be 'lightscreen.reg' instead of 'lightscreen_portable.reg' This is a very minor change and after this, I don't think there would be any additional changes until the original author fixes the timestamp issue.