replacing version constants with function access

cmd/align.go

@@ -74,7 +74,7 @@ func runSketch() {
 
 	// start the sketch sub command
 	start := time.Now()
-	log.Printf("i am groot (version %s)", version.VERSION)
+	log.Printf("i am groot (version %s)", version.GetVersion)
 	log.Printf("starting the sketch subcommand")
 
 	// check the supplied files and then log some stuff
@@ -91,8 +91,8 @@ func runSketch() {
 	log.Print("loading the index information...")
 	info := new(pipeline.Info)
 	misc.ErrorCheck(info.Load(*indexDir + "/groot.gg"))
-	if info.Version != version.VERSION {
-		misc.ErrorCheck(fmt.Errorf("the groot index was created with a different version of groot (you are currently using version %v)", version.VERSION))
+	if info.Version != version.GetVersion {
+		misc.ErrorCheck(fmt.Errorf("the groot index was created with a different version of groot (you are currently using version %v)", version.GetVersion))
 	}
 	log.Printf("\tk-mer size: %d\n", info.KmerSize)
 	log.Printf("\tsketch size: %d\n", info.SketchSize)

cmd/get.go

@@ -46,7 +46,7 @@ var md5sums = map[string]string{
 }
 
 // dbUrl to download databases from
-var dbURL = fmt.Sprintf("https://github.com/will-rowe/groot/raw/master/db/clustered-ARG-databases/%v/", version.VERSION)
+var dbURL = fmt.Sprintf("https://github.com/will-rowe/groot/raw/master/db/clustered-ARG-databases/%v/", version.GetBaseVersion)
 
 // the command line arguments
 var (

cmd/index.go

@@ -77,7 +77,7 @@ func runIndex() {
 
 	// start the index  sub command
 	start := time.Now()
-	log.Printf("i am groot (version %s)", version.VERSION)
+	log.Printf("i am groot (version %s)", version.GetVersion)
 	log.Printf("starting the index subcommand")
 
 	// check the supplied files and then log some stuff
@@ -92,7 +92,7 @@ func runIndex() {
 
 	// record the runtime information for the index sub command
 	info := &pipeline.Info{
-		Version:    version.VERSION,
+		Version:    version.GetVersion,
 		KmerSize:   *kmerSize,
 		SketchSize: *sketchSize,
 		WindowSize: *windowSize,

cmd/report.go

@@ -110,7 +110,7 @@ func runReport() {
 	logFH := misc.StartLogging(*logFile)
 	defer logFH.Close()
 	log.SetOutput(logFH)
-	log.Printf("i am groot (version %s)", version.VERSION)
+	log.Printf("i am groot (version %s)", version.GetVersion)
 	log.Printf("starting the report subcommand")
 	// check the supplied files and then log some stuff
 	log.Printf("checking parameters...")

cmd/version.go

@@ -33,7 +33,7 @@ var versionCmd = &cobra.Command{
 	Short: "Prints current version number and exits",
 	Long:  `Prints current version number and exits`,
 	Run: func(cmd *cobra.Command, args []string) {
-		fmt.Println(version.VERSION)
+		fmt.Println(version.GetVersion)
 	},
 }
 

src/graph/graphio.go

@@ -20,7 +20,7 @@ func (GrootGraph *GrootGraph) SaveGraphAsGFA(fileName string, totalKmers int) (i
 	// a flag to prevent dumping graphs which had no reads map
 	graphUsed := false
 	t := time.Now()
-	stamp := fmt.Sprintf("variation graph created by groot (version %v) at: %v", version.VERSION, t.Format("Mon Jan _2 15:04:05 2006"))
+	stamp := fmt.Sprintf("variation graph created by groot (version %v) at: %v", version.GetVersion, t.Format("Mon Jan _2 15:04:05 2006"))
 	msg := fmt.Sprintf("this graph is approximately weighted using k-mer frequencies from projected read sketches (total k-mers projected across all graphs: %d)", totalKmers)
 	// create a GFA instance
 	newGFA := gfa.NewGFA()

src/pipeline/1_pipeline_test.go

@@ -31,7 +31,7 @@ TEST PARAMETERS
 */
 var testParameters = &Info{
 	NumProc:              1,
-	Version:              version.VERSION,
+	Version:              version.GetVersion,
 	KmerSize:             31,
 	SketchSize:           30,
 	WindowSize:           100,

src/pipeline/boss.go

@@ -55,7 +55,7 @@ func (theBoss *theBoss) mapReads() error {
 	theBoss.alignments = make(chan *sam.Record, BUFFERSIZE)
 
 	// get program info for SAM header (unique ID, name, command, previous program ID, version)
-	programInfo := sam.NewProgram("1", "groot", "groot align", "", version.VERSION)
+	programInfo := sam.NewProgram("1", "groot", "groot align", "", version.GetVersion)
 
 	// get some readgroup information TODO: set this properly
 	rg, err := sam.NewReadGroup("readsID", "", "", "", "groot align", "illumina", "", "sampleID", "", "", time.Now(), 1000)

src/pipeline/haplotype.go

@@ -165,7 +165,7 @@ func (proc *HaplotypeParser) Run() {
 			log.Printf("\t- [%v (abundance: %.3f)]", path, abundances[i])
 			keptPaths = append(keptPaths, path)
 		}
-		g.GrootVersion = version.VERSION
+		g.GrootVersion = version.GetVersion
 		keptGraphs[g.GraphID] = g
 	}
 	proc.info.Store = keptGraphs

src/version/version.go

@@ -1,4 +1,22 @@
 package version
 
-// VERSION is the current GROOT software version
-const VERSION = "1.0.1"
+import "fmt"
+
+// major is the major version number
+const major = 1
+
+// minor is the minor version number
+const minor = 0
+
+// patch is the patch version number
+const patch = 1
+
+// GetVersion returns the full version string for the current GROOT software
+func GetVersion() string {
+	return fmt.Sprintf("%d.%d.%d", major, minor, patch)
+}
+
+// GetBaseVersion returns the major minor version string for the current GROOT software
+func GetBaseVersion() string {
+	return fmt.Sprintf("%d.%d", major, minor)
+}