nsis script to make windows installer #432

Merged
erinn merged 3 commits from nsi into master 2020-11-28 01:55:31 +00:00
Owner
No description provided.
Member

Drone Build Status: success

https://build.openprivacy.ca/cwtch.im/ui/864

Drone Build Status: success https://build.openprivacy.ca/cwtch.im/ui/864
erinn requested changes 2020-11-28 01:06:51 +00:00
@ -0,0 +9,4 @@
Unicode True
# define the name of the installer
Outfile "cwtch-installer.exe"

can we get this on build.openprivacy.ca?

can we get this on build.openprivacy.ca?
Author
Owner

yup that's the plan. Once merged I can add a nsis docker image to the server and work on configuring a drone stage to generate it and upload it :)

yup that's the plan. Once merged I can add a nsis docker image to the server and work on configuring a drone stage to generate it and upload it :)
erinn marked this conversation as resolved
@ -0,0 +20,4 @@
InstallDir "$PROGRAMFILES\Cwtch"
;Get installation folder from registry if available
InstallDirRegKey HKCU "Software\Cwtch" ""

does MUI generate an uninstaller? can users remove cwtch via the windows add/remove programs panel after installing it?

does MUI generate an uninstaller? can users remove cwtch via the windows add/remove programs panel after installing it?
Author
Owner

nsis can. this config doesn't do that. that's gonna be another project. Ideally a day, but you saw how well this one went... so.. #TODO?

nsis can. this config doesn't do that. that's gonna be another project. Ideally a day, but you saw how well this one went... so.. #TODO?
erinn marked this conversation as resolved
@ -0,0 +48,4 @@
; Pages --------
!define MUI_WELCOMEPAGE_TITLE "Welcome to the Cwtch installer"

we'll have to figure out how to translate these...

we'll have to figure out how to translate these...
Author
Owner

NSIS has languages and you can I believe tag things with languages... but yeah. that'll be ... a later project as well

NSIS has languages and you can I believe tag things with languages... but yeah. that'll be ... a later project as well
erinn marked this conversation as resolved
@ -0,0 +53,4 @@
Cwtch is a platform for building consentful, decentralized, untrusted infrastructure using metadata resistant group communication applications. Currently there is a selfnamed instant messaging prototype app that is driving development and testing. Many Further apps are planned as the platform matures."
!define MUI_FINISHPAGE_TITLE "Enjoy Cwtch"
!define MUI_FINISHPAGE_RUN $INSTDIR/ui.exe

quotes?

quotes?
Author
Owner

apparently don't need them

apparently don't need them
erinn marked this conversation as resolved
@ -0,0 +54,4 @@
!define MUI_FINISHPAGE_TITLE "Enjoy Cwtch"
!define MUI_FINISHPAGE_RUN $INSTDIR/ui.exe
!define MUI_FINISHPAGE_TEXT "You can keep uptodate on Cwtch and report any issues you have at https://cwtch.im"

nit: up-to-date

nit: up-to-date
Author
Owner

fixed

fixed
@ -0,0 +56,4 @@
!define MUI_FINISHPAGE_RUN $INSTDIR/ui.exe
!define MUI_FINISHPAGE_TEXT "You can keep uptodate on Cwtch and report any issues you have at https://cwtch.im"
!define MUI_FINISHPAGE_LINK "https://cwtch.im"
!define MUI_FINISHPAGE_LINK_LOCATION "https://cwtch.im"

the app itself avoids making clickable links... do we want to apply that policy to the installer as well?

the app itself avoids making clickable links... do we want to apply that policy to the installer as well?
@ -0,0 +73,4 @@
Section
# define the output path for this file
SetOutPath $INSTDIR

quotes?

quotes?
Author
Owner

doesn't need them

doesn't need them
erinn marked this conversation as resolved
@ -0,0 +83,4 @@
# create a shortcut named "new shortcut" in the start menu programs directory
# presently, the new shortcut doesn't call anything (the second field is blank)

outdated comment?

outdated comment?
Author
Owner

fixed

fixed
@ -0,0 +87,4 @@
CreateShortcut "$SMPROGRAMS\Cwtch.lnk" "$INSTDIR\ui.exe"
;Store installation folder
WriteRegStr HKCU "Software\Cwtch" "" $INSTDIR

does $INSTDIR potentially need quotes? this ref to ui.exe has quotes but there's one above that does not

does $INSTDIR potentially need quotes? this ref to ui.exe has quotes but there's one above that does not
Author
Owner

doesn't need it, its in my registry fine

doesn't need it, its in my registry fine
erinn marked this conversation as resolved
Member

Drone Build Status: success

https://build.openprivacy.ca/cwtch.im/ui/866

Drone Build Status: success https://build.openprivacy.ca/cwtch.im/ui/866
erinn approved these changes 2020-11-28 01:55:23 +00:00
erinn merged commit d4e5546079 into master 2020-11-28 01:55:31 +00:00
Member

Drone Build Status: success

https://build.openprivacy.ca/cwtch.im/ui/868

Drone Build Status: success https://build.openprivacy.ca/cwtch.im/ui/868
This repo is archived. You cannot comment on pull requests.
No description provided.