Ticket #2722 (closed: fixed)

Opened 10 years ago

Last modified 5 years ago

Only have version number in one place

Reported by: Russell Taylor Owned by: Russell Taylor
Priority: minor Milestone: Iteration 28
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Peter Peterson

Description

Do away with MantidPlotReleaseDate.h and put everything into MantidVersion.h(.in) in Kernel.

Add a MANTID_REVISION as well. Tidy up the arguments that MantidPlot can take (e.g. there are two places where "-v" is tested for - one obviously hides the other).

To avoid pain, don't do this until Scons builds have gone away.

Change History

comment:1 Changed 9 years ago by Russell Taylor

  • Status changed from new to accepted

comment:2 Changed 9 years ago by Russell Taylor

(In [11075]) Do away with MantidPlotReleaseDate.h and have everything in MantidVersion. Turn MantidVersion into a class with static functions so that the part that changes every time can go in a cpp - this means files that include MantidVersion.h (e.g. the enormous ApplicationWindow.cpp) don't get compiled every commit. Re #2722. Also formats date better, which fixes #2246.

comment:3 Changed 9 years ago by Russell Taylor

(In [11076]) Tidy up. Re #2722.

comment:4 Changed 9 years ago by Russell Taylor

(In [11094]) Add a method to just get the revision number. Re #2722.

comment:5 Changed 9 years ago by Nick Draper

  • Milestone changed from Iteration 28 to Iteration 29

Bulk move of tickets at the end of iteration 28

comment:6 Changed 9 years ago by Russell Taylor

(In [11109]) Fix package version number. Re #2722.

comment:7 Changed 9 years ago by Russell Taylor

  • Milestone changed from Iteration 29 to Iteration 28

comment:8 Changed 9 years ago by Russell Taylor

(In [11129]) Tidy up MantidPlot command line argument options. Re #2722.

comment:9 Changed 9 years ago by Russell Taylor

(In [11130]) Try to fix windows. Re #2722.

comment:10 Changed 9 years ago by Russell Taylor

  • Status changed from accepted to verify
  • Resolution set to fixed

Need to be sure to test the --about argument on Windows.

comment:11 Changed 9 years ago by Peter Peterson

  • Status changed from verify to verifying
  • Tester set to Peter Peterson

comment:12 Changed 9 years ago by Peter Peterson

  • Status changed from verifying to closed

This was done correctly.

comment:13 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 3569

Note: See TracTickets for help on using tickets.