Issue 686: Implement a better system for assigning a version which is not web installed

Assigned To:Guest
Opened:2019-10-02 by Brendan MacLean
Changed:2021-11-30 by Brendan MacLean
Resolved:2021-11-30 by matt.chambers42
Closed:2021-11-30 by Brendan MacLean
2019-10-02 Brendan MacLean
Title»Implement a better system for assigning a version which is not web installed
Assigned ToGuest»
Notify»Nick Shulman
Currently we have multiple places we need to set a Skyline version in order to ensure the software knows what its version is:
1. Set during installer build in the .csproj file
2. Set in Skyline/app.config - for running a non-web installed Skyline.exe
3. Set in SkylineCmd/App.config - for running SkylineCmd.exe

I am not even sure we ever recognized the need for #3 until just now when I implemented a --version argument for SkylineCmd and it just reported "Skyline-daily (64-bit)" when I ran SkylineCmd.exe, but a full version stamp when I ran Skyline.exe with the version in its app.config. I was able to fix this issue by setting the same version in SkylineCmd/App.config, but this is obviously not ideal to need to set the version manually in 3 separate places.

Matt, you have agreed to look into automating this for the Docker container, and the SkylineCmd.exe issue will certainly get you there.

Once I merge my changes to master, you should at least have the --version argument to check your work more easily than other options like looking inside a saved .sky file or an audit log.

More and more we rely on Skyline knowing its version (especially the audit log). We can't continue releasing Skyline versions which may not know their version number depending on how they get run (e.g. and especially SkylineCmd)

2021-11-30 matt.chambers42
resolve as Fixed
Assigned»Brendan MacLean

2021-11-30 Brendan MacLean
Assigned ToBrendan MacLean»Guest