Skip to content

Commit ca387f6

Browse files
authored
Merge pull request #363 from DUNE-DAQ/amogan/issue362_unused_variable
Fix name of branch_name variable
2 parents 4e29c26 + 152e190 commit ca387f6

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

scripts/spack/make-release-repo.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -105,10 +105,10 @@ def copy_release_yaml(self, repo_path, update_hash=False):
105105
yaml.dump(self.rdict, outfile, Dumper=MyDumper, default_flow_style=False, sort_keys=False)
106106
return
107107

108-
def get_cmake_dependencies(self, package_name, branch_name='develop'):
108+
def get_cmake_dependencies(self, package_name, branch_name):
109109
if self.overwrite_branch != '':
110110
if check_branch_exists(package_name, self.overwrite_branch):
111-
branch = self.overwrite_branch
111+
branch_name = self.overwrite_branch
112112
file_name = "CMakeLists.txt"
113113
cmakelists_path = f'https://raw.githubusercontent.com/DUNE-DAQ/{package_name}/{branch_name}/{file_name}'
114114
command = f'curl -o {file_name} --fail {cmakelists_path}'

0 commit comments

Comments
 (0)