FAB-10445 configtxgen bad check on channel id

Fixes a bug where configtxgen was checking for an empty channel ID and
printing a warning before the flags had actually been parsed.

Change-Id: Ic62378d7f9245df54e23713d0b86598168b14589
Signed-off-by: Jason Yellick <jyellick@us.ibm.com>

Author: Jason Yellick

common/tools/configtxgen/main.go

@@ -219,15 +219,15 @@ func main() {
 	flag.StringVar(&asOrg, "asOrg", "", "Performs the config generation as a particular organization (by name), only including values in the write set that org (likely) has privilege to set")
 	flag.StringVar(&printOrg, "printOrg", "", "Prints the definition of an organization as JSON. (useful for adding an org to a channel manually)")
 
+	version := flag.Bool("version", false, "Show version information")
+
+	flag.Parse()
+
 	if channelID == "" {
 		channelID = genesisconfig.TestChainID
 		logger.Warningf("Omitting the channel ID for configtxgen is deprecated.  Explicitly passing the channel ID will be required in the future, defaulting to '%s'.", channelID)
 	}
 
-	version := flag.Bool("version", false, "Show version information")
-
-	flag.Parse()
-
 	// show version
 	if *version {
 		printVersion()