PortableApps.com Platform and Suite 1.6 have been released.

Error in the Sunbird Launcher ?

Submitted by tsyr2ko on November 25, 2006 - 12:45am

I am looking in the SunbirdPortable.nsi and i think there is a problem in it.

At line 117, we have ReadEnvStr $APPDATAPATH "APPDATA"
And, after that, on line 198, there is something like this
IfFileExists "$APPDATA\Mozilla\Sunbird\*.*" CheckIfRunning

Assuming the syntax of ReadEnvStr
the Environment variable will be stored into $APPDATAPATH variable

So the line 198 (and some others near line 383) would be
IfFileExists "$APPDATAPATH\Mozilla\Sunbird\*.*" CheckIfRunning

No ?? Is this correct for you ??

Correct me if i'm wrong.


( categories: )

Hmm

Sounds right.
----
Ryan McCue
Cube Games
Life is like a sewer. What you get out of it depends on what you put into it.
(Tom Lehrer)

"If you're not part of the solution, you're part of the precipitate."

Looks

like a bug to me.

EDIT: Although I think you can call $APPDATA straight off and it will give you the correct dir, so maybe the readenvstr line is unnecessary.

Yours

Steve Lamerton

The Liberta Project

'...and do the other things, not because they are easy, but because they are hard...' JFK

Right

Your right, there is a constant for $APPDATA, that's why the NSIS compiler
don't stop on errors.

It's not a important bug, just the ReadEnvStr line is obsolete

----
Sorry for my english, it's not my first language :/

--

Sorry for my english, it's not my first language :/

Another interrogation

I think it was not necessary to create a new topic for this, so i post here.

Into the CheckIfRunning label, it checks for multiple instances.

On line 207, a test with IfFileExists on file parent.lock is done
but , i think, not correctly implemented.

IfFileExists "$PROFILEDIRECTORY\parent.lock" "" WarnAnotherInstance
StrCpy $SECONDARYLAUNCH "true"
Goto RunProgram

If the file really exists, the jump would go to WarnAnotherInstance and not go to
on the bottom line.

For tests, i edit installer and delete the "" between file path and "WarnAnotherInstance"
and it works as it should be now.

--

Sorry for my english, it's not my first language :/

--

Sorry for my english, it's not my first language :/