customize windows install for this product #28
Conversation
|
I have no idea what I am looking at here. Can you walk me through this on Monday @marcparadise ? |
marcparadise
force-pushed
the
windows-build-fix
branch
2 times, most recently
from
73adc64
to
bff41ab
Compare
| @@ -58,7 +58,7 @@ | |||
|
|
|||
| <Directory Id="TARGETDIR" Name="SourceDir"> | |||
| <Directory Id="ProgramFilesFolder"> | |||
| <Directory Id="INSTALLLOCATION" Name="opscode"> | |||
| <Directory Id="INSTALLLOCATION" Name="chef"> | |||
There was a problem hiding this comment.
I think this should be opscode as the installation path will be c:\opscode\chef-workstaiton
There was a problem hiding this comment.
That's what we've been trying to fix - to install to a proper location Program Files\Chef/Workstation. We've tabled that for now as it was turning into a rabbit warren...
| @@ -58,7 +58,7 @@ | |||
|
|
|||
| <Directory Id="TARGETDIR" Name="SourceDir"> | |||
| <Directory Id="ProgramFilesFolder"> | |||
There was a problem hiding this comment.
The example I followed has this as <Directory Id="WindowsVolume">
There was a problem hiding this comment.
That comes from wanting to install to C:/Opscode instead of C:/Program Files - the Id ProgramFilesFolder will resolve to "C:\Program Files".
marcparadise
force-pushed
the
windows-build-fix
branch
from
7358d1d
to
bb6f5fd
Compare
The default installation ran into some errors. This change includes a custom wxs build script that does the right things for workstation. At this time windows installations will default to C:/opscode/chef-workstation; however we intend to come back and use C:\Program Files\Chef\Workstation if omnibus can be modified to better support it.
marcparadise
force-pushed
the
windows-build-fix
branch
from
bb6f5fd
to
bcda7a9
Compare
| @@ -17,14 +17,20 @@ | |||
| name "chef-workstation" | |||
| friendly_name "Chef Workstation" | |||
| maintainer "Chef Software, Inc. <maintainers@chef.io>" | |||
| homepage "TBD" | |||
| homepage "https://chef.sh" | |||
|
|
|||
| license "Apache-2.0" | |||
| license_file "../LICENSE" | |||
|
|
|||
| # Defaults to C:/chef-workstation on Windows | |||
| # and /opt/chef-workstation on all other platforms | |||
There was a problem hiding this comment.
explain why we're being explicit
| dependency "clean-static-libs" | ||
| dependency "version-manifest" |
| @@ -35,6 +35,7 @@ | |||
| dependency "bundler" | |||
| dependency "ruby" | |||
| dependency "appbundler" | |||
| dependency "version-manifest" | |||
| --> | ||
| <% if fastmsi %> | ||
| <SetProperty Id="FastUnzip" | ||
| Value="FASTZIPDIR=[INSTALLLOCATION];FASTZIPAPPNAME=chef_workstation" |
There was a problem hiding this comment.
Went over this with @marcparadise - Looks good! Always love reading XML
marcparadise
force-pushed
the
windows-build-fix
branch
from
7005938
to
9ee504d
Compare
No description provided.