moving minor branch

CMakeLists.txt

@@ -1,5 +1,5 @@
 cmake_minimum_required(VERSION 3.25)
-project(COSC-4P80-Assignment-1 VERSION 13.0.0)
+project(COSC-4P80-Assignment-1 VERSION 13.7.0)
 
 option(ENABLE_ADDRSAN "Enable the address sanitizer" OFF)
 option(ENABLE_UBSAN "Enable the ub sanitizer" OFF)

commit.py

@@ -24,14 +24,19 @@ USER_HOME = Path.home()
 ENVIRONMENT_DATA_LOCATION = USER_HOME / ".brett_scripts.env"
 
 if sys.platform.startswith("win"):
-	CONFIG_FILE_DIRECTORY = Path(os.getenv('APPDATA') + "\BLT")
+	CONFIG_FILE_DIRECTORY = Path(os.getenv('APPDATA') + "\blt")
-	CONFIG_FILE_LOCATION = Path(CONFIG_FILE_DIRECTORY + "\commit_config.env")
+	CONFIG_FILE_LOCATION = Path(CONFIG_FILE_DIRECTORY + "\commit_config.json")
 else:
-	XDG_CONFIG_HOME = Path(os.environ.get('XDG_CONFIG_HOME'))
+	XDG_CONFIG_HOME = os.environ.get('XDG_CONFIG_HOME')
+	if XDG_CONFIG_HOME is None:
+		XDG_CONFIG_HOME = USER_HOME / ".config"
+	else:
+		XDG_CONFIG_HOME = Path(XDG_CONFIG_HOME)
+	
 	if len(str(XDG_CONFIG_HOME)) == 0:
 		XDG_CONFIG_HOME = USER_HOME
 	CONFIG_FILE_DIRECTORY = XDG_CONFIG_HOME / "blt"
-	CONFIG_FILE_LOCATION = CONFIG_FILE_DIRECTORY / "commit_config.env"
+	CONFIG_FILE_LOCATION = CONFIG_FILE_DIRECTORY / "commit_config.json"
 
 class Config:
 	def __init__(self):
@@ -39,29 +44,32 @@ class Config:
 		self.branch_on_major = True
 		self.branch_on_minor = False
 		self.release_on_major = True
-		self.release_on_minor = False
+		self.release_on_minor = True
 		self.main_branch = "main"
 		self.patch_limit = -1
 	
+	def toJSON(self):
+		return json.dumps(self, default=lambda o: o.__dict__, sort_keys=True, indent=4)
+	
+	def fromJSON(file):
+		with open(file, "r") as f:
+			j = json.load(f)
+			obj = Config()
+			[setattr(obj, key, val) for key, val in j.items() if hasattr(obj, key)]
+			if obj.branch_on_minor:
+				obj.branch_on_major = True
+			return obj
+
 	def from_file(file):
 		values = {}
 		if (not os.path.exists(file)):
 			return Config()
 
-		with open(file, "rt") as f:
+		with open(file, "r") as f:
-			for line in f:
+			j = json.load(f)
-				if line.startswith("export"):
+			obj = Config()
-					content = line.split("=")
+			[setattr(obj, key, val) for key, val in j.items() if hasattr(obj, key)]
-					for idx, c in enumerate(content):
+			return obj
-						content[idx] = c.replace("export", "").strip()
-					values[content[0]] = content[1].replace("\"", "").replace("'", "")
-		config = Config()
-		config.branch_on_major = values["branch_on_major"].lower() == "true"
-		config.release_on_major = values["release_on_major"].lower() == "true"
-		config.main_branch = values["main_branch"]
-		config.patch_limit = int(values["patch_limit"])
-  
-		return config;
 
 	def save_to_file(self, file):
 		dir_index = str(file).rfind("/")
@@ -69,11 +77,8 @@ class Config:
 		if not os.path.exists(dir):
 			print(f"Creating config directory {dir}")
 			os.makedirs(dir)
-		with open(file, 'w') as f:
+		with open(file, "w") as f:
-			f.write("export branch_on_major=" + str(self.branch_on_major) + "\n")
+			json.dump(self, f, default=lambda o: o.__dict__, sort_keys=True, indent=4)
-			f.write("export release_on_major=" + str(self.release_on_major) + "\n")
-			f.write('export main_branch="' + self.main_branch + '"' + "\n")
-			f.write("export patch_limit=" + str(self.patch_limit) + "\n")
 			
 
 class EnvData:
@@ -141,7 +146,6 @@ def recombine(cmake_text, version_parts, begin, end):
 	constructed_text_end = cmake_text[end::]
 	return constructed_text_begin + constructed_version + constructed_text_end
 
-
 def inc_major(cmake_text):
 	version_parts, begin, end = split_version(cmake_text)
 	version_parts[0] = str(int(version_parts[0]) + 1)
@@ -168,6 +172,12 @@ def make_branch(config: Config, name):
 	subprocess.call(["git", "merge", config.main_branch])
 	subprocess.call(["git", "checkout", config.main_branch])
  
+def sync_branch(config: Config, version_parts, args):
+    if config.branch_on_major:
+        # Branch will be created.
+        if args.minor:
+            return;
+
 def make_release(env: EnvData, name):
 	print(f"Making new release {name}")
 	repos_v = open_process(["git", "remote", "-v"])[0].splitlines()
@@ -176,11 +186,10 @@ def make_release(env: EnvData, name):
 		origin = ''.join(itertools.takewhile(str.isalpha, line.decode('utf8')))
 		urls.append("https://api.github.com/repos/" + open_process(["git", "remote", "get-url", origin], False)[0].decode('utf8').replace("\n", "").replace("https://github.com/", "") + "/releases")
 	urls = set(urls)
-	print(f"Urls: {urls}")
 	data = {
 		'tag_name': name,
 		'name': name,
-		'body': "Automated Release",
+		'body': "Automated Release '" + name + "'",
 		'draft': False,
 		'prerelease': False
 	}
@@ -209,12 +218,19 @@ def main():
 	parser.add_argument("-p", "--patch", action='store_true', default=False, required=False)
 	parser.add_argument("-m", "--minor", action='store_true', default=False, required=False)
 	parser.add_argument("-M", "--major", action='store_true', default=False, required=False)
-	parser.add_argument('-e', "--env", help="environment file", required=False)
+	parser.add_argument('-e', "--env", help="environment file", required=False, default=None)
-	parser.add_argument('-c', "--config", help="config file", required=False)
+	parser.add_argument('-c', "--config", help="config file", required=False, default=None)
-	parser.add_argument("--create_default_config", action="store_true", default=False, required=False)
+	parser.add_argument("--create-default-config", action="store_true", default=False, required=False)
+	parser.add_argument("--no-release", action="store_true", default=False, required=False)
+	parser.add_argument("--no-branch", action="store_true", default=False, required=False)
 	
 	args = parser.parse_args()
  
+	if args.create_default_config:
+		config = Config()
+		config.save_to_file(args.config if args.config is not None else CONFIG_FILE_LOCATION)
+		return
+ 
 	if args.env is not None:
 		env = EnvData.get_env_from_file(args.e)
 	else:
@@ -225,9 +241,6 @@ def main():
 	else:
 		config = Config.from_file(CONFIG_FILE_LOCATION)
 	
-	if args.create_default_config:
-		config.save_to_file(args.config if args.config is not None else CONFIG_FILE_LOCATION)
-	
 	cmake_text = load_cmake()
 	cmake_version = get_version(cmake_text)[0]
 	print(f"Current Version: {cmake_version}")
@@ -259,24 +272,45 @@ def main():
 		print("Selected patch")
 		write_cmake(inc_patch(config, cmake_text))
 
-	
 	subprocess.call(["git", "add", "*"])
 	subprocess.call(["git", "commit"])
  
 	cmake_text = load_cmake()
-	if args.major:
 	version_parts = split_version(cmake_text)[0]
+	if args.major:
 		if config.branch_on_major:
+			if not args.no_branch:
 				make_branch(config, "v" + str(version_parts[0]))
-		if config.branch_on_minor:
-			make_branch(config, "v" + str(version_parts[0]) + "." + str(version_parts[1]))
-		if config.release_on_major:
-			make_release(env, "v" + str(version_parts[0]))
-		if config.release_on_minor:
-			make_release(env, "v" + str(version_parts[0]) + "." + str(version_parts[1]))
 	
+	if args.minor:
+		if config.branch_on_minor:
+			if not args.no_branch:
+				make_branch(config, "v" + str(version_parts[0]) + "." + str(version_parts[1]))
+		elif config.branch_on_major:
+			subprocess.call(["git", "checkout", "v" + str(version_parts[0])])
+			subprocess.call(["git", "rebase", config.main_branch])
+			subprocess.call(["git", "checkout", config.main_branch])
+
+	if args.patch:
+		if config.branch_on_minor:
+			subprocess.call(["git", "checkout", "v" + str(version_parts[0]) + "." + str(version_parts[1])])
+			subprocess.call(["git", "rebase", config.main_branch])
+			subprocess.call(["git", "checkout", config.main_branch])
+		elif config.branch_on_major:
+			subprocess.call(["git", "checkout", "v" + str(version_parts[0])])
+			subprocess.call(["git", "rebase", config.main_branch])
+			subprocess.call(["git", "checkout", config.main_branch])
+  
+	sync_branch(config=config, version_parts=version_parts, args=args)
 		
 	subprocess.call(["sh", "-c", "git remote | xargs -L1 git push --all"])
  
+	if args.major:
+		if not args.no_release and config.release_on_major:
+			make_release(env, "v" + str(version_parts[0]))
+	if args.minor:
+		if not args.no_release and config.release_on_minor:
+			make_release(env, "v" + str(version_parts[0]) + "." + str(version_parts[1]))
+  
 if __name__ == "__main__":
 	main()