You are here

[Fixed] Lightscreen Portable 1.01 TrollTech Registry Keys

19 posts / 0 new
Last post
Bart.S
Offline
Last seen: 7 months 2 weeks ago
Developer
Joined: 2008-07-23 07:56
[Fixed] Lightscreen Portable 1.01 TrollTech Registry Keys

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:

Chris Morgan
Chris Morgan's picture
Offline
Last seen: 8 years 9 months ago
Joined: 2007-04-15 21:08
Hmm...

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

Gizmokid2005
Gizmokid2005's picture
Offline
Last seen: 3 months 3 weeks ago
Developer
Joined: 2007-01-17 19:24
Gotta love it.

The launcher for KVIrc 4 Portable handles all these entries since it's Qt as well. Hopefully they're fixed soon....

Bart.S
Offline
Last seen: 7 months 2 weeks ago
Developer
Joined: 2008-07-23 07:56
Regression

The Qt-Mess is a 1.01 regression. Version 1.00 leaves no registry keys behind.

John T. Haller
John T. Haller's picture
Offline
Last seen: 7 hours 14 min ago
AdminDeveloperModeratorTranslator
Joined: 2005-11-28 22:21
Same Launcher

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!

Bart.S
Offline
Last seen: 7 months 2 weeks ago
Developer
Joined: 2008-07-23 07:56
Nope

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.

John T. Haller
John T. Haller's picture
Offline
Last seen: 7 hours 14 min ago
AdminDeveloperModeratorTranslator
Joined: 2005-11-28 22:21
Right

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!

Chris Morgan
Chris Morgan's picture
Offline
Last seen: 8 years 9 months ago
Joined: 2007-04-15 21:08
[QtKeysCleanup]

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:

[QtKeysCleanup]
1=Qt Factory Cache 4.6\com.trolltech.Qt.QImageIOHandlerFactoryInterface:
2=Qt Plugin Cache 4.6.false

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

OliverK
OliverK's picture
Offline
Last seen: 2 years 9 months ago
Developer
Joined: 2007-03-27 15:21
wouldn't you need quotes

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

John T. Haller
John T. Haller's picture
Offline
Last seen: 7 hours 14 min ago
AdminDeveloperModeratorTranslator
Joined: 2005-11-28 22:21
Nope

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!

App Guy
Offline
Last seen: 10 years 5 months ago
Joined: 2008-03-16 14:24
Possible solution for registry problem

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

Bart.S
Offline
Last seen: 7 months 2 weeks ago
Developer
Joined: 2008-07-23 07:56
Sure that works.

Points 1,2 and 4 could be fixed within one minute.

John T. Haller
John T. Haller's picture
Offline
Last seen: 7 hours 14 min ago
AdminDeveloperModeratorTranslator
Joined: 2005-11-28 22:21
Fix Coming

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!

Bart.S
Offline
Last seen: 7 months 2 weeks ago
Developer
Joined: 2008-07-23 07:56
Performance?

Wouldn't it increase performance? Adjust paths only if $EXEDIR has changed?

John T. Haller
John T. Haller's picture
Offline
Last seen: 7 hours 14 min ago
AdminDeveloperModeratorTranslator
Joined: 2005-11-28 22:21
Not Really

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!

John T. Haller
John T. Haller's picture
Offline
Last seen: 7 hours 14 min ago
AdminDeveloperModeratorTranslator
Joined: 2005-11-28 22:21
PAL Update, Gord?

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!

Gord Caswell
Gord Caswell's picture
Offline
Last seen: 4 months 6 days ago
DeveloperModerator
Joined: 2008-07-24 18:46
Already on it

Working on it at the moment.

Gord Caswell
Gord Caswell's picture
Offline
Last seen: 4 months 6 days ago
DeveloperModerator
Joined: 2008-07-24 18:46
Sent

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?

John T. Haller
John T. Haller's picture
Offline
Last seen: 7 hours 14 min ago
AdminDeveloperModeratorTranslator
Joined: 2005-11-28 22:21
Fixed in 1.01 Rev 2

This is fixed in the new Lightscreen Portable release.

Sometimes, the impossible can become possible, if you're awesome!

Log in or register to post comments