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.
Visit the Community page
Join our forums
Subscribe to our email newsletter
Subscribe with RSS
Follow us on Facebook
Follow us on LinkedIn
Follow us on Twitter
Follow us on Mastodon
Follow us on BlueSky
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."
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
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 :/
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 :/