Bugs, bugs, bugs:
- registry keys left behind:
HKCU\Software\Trolltech HKCU\Software\Trolltech\OrganizationDefaults HKCU\Software\Trolltech\OrganizationDefaults\Qt Factory Cache 4.6 HKCU\Software\Trolltech\OrganizationDefaults\Qt Factory Cache 4.6\com.trolltech.Qt.QImageIOHandlerFactoryInterface: HKCU\Software\Trolltech\OrganizationDefaults\Qt Factory Cache 4.6\com.trolltech.Qt.QImageIOHandlerFactoryInterface:\F: HKCU\Software\Trolltech\OrganizationDefaults\Qt Factory Cache 4.6\com.trolltech.Qt.QImageIOHandlerFactoryInterface:\F:\PortableApps HKCU\Software\Trolltech\OrganizationDefaults\Qt Factory Cache 4.6\com.trolltech.Qt.QImageIOHandlerFactoryInterface:\F:\PortableApps\LightscreenPortable HKCU\Software\Trolltech\OrganizationDefaults\Qt Factory Cache 4.6\com.trolltech.Qt.QImageIOHandlerFactoryInterface:\F:\PortableApps\LightscreenPortable\App HKCU\Software\Trolltech\OrganizationDefaults\Qt Factory Cache 4.6\com.trolltech.Qt.QImageIOHandlerFactoryInterface:\F:\PortableApps\LightscreenPortable\App\Lightscreen HKCU\Software\Trolltech\OrganizationDefaults\Qt Factory Cache 4.6\com.trolltech.Qt.QImageIOHandlerFactoryInterface:\F:\PortableApps\LightscreenPortable\App\Lightscreen\imageformats HKCU\Software\Trolltech\OrganizationDefaults\Qt Plugin Cache 4.6.false HKCU\Software\Trolltech\OrganizationDefaults\Qt Plugin Cache 4.6.false\F: HKCU\Software\Trolltech\OrganizationDefaults\Qt Plugin Cache 4.6.false\F:\PortableApps HKCU\Software\Trolltech\OrganizationDefaults\Qt Plugin Cache 4.6.false\F:\PortableApps\LightscreenPortable HKCU\Software\Trolltech\OrganizationDefaults\Qt Plugin Cache 4.6.false\F:\PortableApps\LightscreenPortable\App HKCU\Software\Trolltech\OrganizationDefaults\Qt Plugin Cache 4.6.false\F:\PortableApps\LightscreenPortable\App\Lightscreen HKCU\Software\Trolltech\OrganizationDefaults\Qt Plugin Cache 4.6.false\F:\PortableApps\LightscreenPortable\App\Lightscreen\imageformats
- DefaultDirectory (line 234-237): It checks for Documents\Music and sets Documents\Pictures as default. Makes no sense.
- WriteScreenshotDirectory (line 223-232): On every launch, it replaces the root of the target path by the current driveletter, even if the portable app hasn't moved. Say Lightscreen Portable is on your harddrive d:, but you want the target directory e:\screenshots. You don't move anything, but the launcher always sets the target directory to d:\screenshots, even if this folder doesn't exist. That's annoying. (and has been reported here too)
- CheckRunning (line 249): Should check "$PROGRAMEXECUTABLE" and not "${DEFAULTEXE}"
... to be continued ... :evil:
The Qt mess. Time to put support for clearing it up into PAL (after Beta 2 I think).
I am a Christian and a developer and moderator here.
“A soft answer turns away wrath, but a harsh word stirs up anger.” – Proverbs 15:1
The launcher for KVIrc 4 Portable handles all these entries since it's Qt as well. Hopefully they're fixed soon....
The Qt-Mess is a 1.01 regression. Version 1.00 leaves no registry keys behind.
It's the same launcher as 1.0. So, it's the app itself. I didn't test it, just did a quick update because David was busy.
Sometimes, the impossible can become possible, if you're awesome!
Portable regression. Version 1.0 uses "Qt blah blah 4.5" and the launcher handles that, but version 1.01 uses "Qt blah blah 4.6".
I say no to quick updates, especially to all Qt apps.
So the app itself changed, not the launcherm as I didn't update nor test the launcher. I just did a quick repack when asked by the app dev. Right or wrong, that's all I did. The other bugs you point out were present in 1.0 as well, as it's the same launcher.
I'll either not touch it until David has time. Or ditch it and just use the PA.c launcher on it when Qt handling is better baked in, because it's basically a more than partial rewrite to fix all the bugs anyway. I'll probably ditch writing out the default path and ditch all options other than splash and additional parameters, which should handle the other bugs.
Sometimes, the impossible can become possible, if you're awesome!
With the tip of the repository of the PortableApps.com Launcher, the Qt keys should be able to be cleaned up with this INI section:
I am a Christian and a developer and moderator here.
“A soft answer turns away wrath, but a harsh word stirs up anger.” – Proverbs 15:1
wouldn't you need quotes around that?
Too many lonely hearts in the real world
Too many bridges you can burn
Too many tables you can't turn
Don't wanna live my life in the real world
Everything after the = is assigned to that variable. You only need it for leading or trailing spaces, I believe.
Sometimes, the impossible can become possible, if you're awesome!
I was having a look in the source of the launcher and noticed that it was deleting keys from "Qt Factory Cache 4.5" when in fact it should be deleting "Qt Factory Cache 4.6". I went in and simply changed the '5' to a '6'. I have not tested this but it seems that it should work.
May the Shwartz be with you
Points 1,2 and 4 could be fixed within one minute.
This will be fixed as part of all the apps being revved.
Point 3 will probably be a WONTFIX. Apps are meant to utilize data and settings on the same drive, not between drives.
Sometimes, the impossible can become possible, if you're awesome!
Wouldn't it increase performance? Adjust paths only if $EXEDIR has changed?
It's a single write operation. And I don't think the launcher tracks path changes.
Keeping apps and data on the same drive is a universal rule with portable apps anyway, so this is really a non-issue.
Sometimes, the impossible can become possible, if you're awesome!
As Gord just re-reported this (removed as a duplicate) would you like to update it to PAL and fix the bug?
Sometimes, the impossible can become possible, if you're awesome!
Working on it at the moment.
I've sent John an update for Lightscreen that fixes this bug, along with updating to use PAL.
[EDIT] Bump - trying to clear up some old bugs. Can the revision get released?
This is fixed in the new Lightscreen Portable release.
Sometimes, the impossible can become possible, if you're awesome!