Skip to content

Commit

Permalink
Merge branch 'static_build_validate_suite_name' of https://github.com…
Browse files Browse the repository at this point in the history
  • Loading branch information
climbfuji committed Nov 5, 2019
2 parents a1db0aa + 5dd7cc9 commit f3cad9e
Showing 1 changed file with 9 additions and 3 deletions.
12 changes: 9 additions & 3 deletions scripts/mkstatic.py
Original file line number Diff line number Diff line change
Expand Up @@ -183,7 +183,7 @@ class API(object):
{suite_switch}
else
write({ccpp_var_name}%errmsg,'(*(a))'), 'Invalid suite ' // trim(suite_name)
write({ccpp_var_name}%errmsg,'(*(a))') 'Invalid suite ' // trim(suite_name)
ierr = 1
end if
Expand Down Expand Up @@ -303,7 +303,7 @@ def write(self):
arguments=argument_list_group)
group_calls += '''
else
write({ccpp_var_name}%errmsg, '(*(a))') "Group " // trim(group_name) // " not found"
write({ccpp_var_name}%errmsg, '(*(a))') 'Group ' // trim(group_name) // ' not found'
ierr = 1
end if
'''.format(ccpp_var_name=ccpp_var.local_name, group_name=group.name)
Expand Down Expand Up @@ -463,6 +463,12 @@ def parse(self):
tree = ET.parse(self._sdf_name)
suite_xml = tree.getroot()
self._name = suite_xml.get('name')
# Validate name of suite in XML tag against filename; could be moved to common.py
if not (os.path.basename(self._sdf_name) == 'suite_{}.xml'.format(self._name)):
logging.critical("Invalid suite name {0} in suite definition file {1}.".format(
self._name, self._sdf_name))
success = False
return success

# Flattened lists of all schemes and subroutines in SDF
self._all_schemes_called = []
Expand Down Expand Up @@ -938,7 +944,7 @@ def write(self, metadata_request, metadata_define, arguments):
{actions_after}
'''.format(subroutine_name=subroutine_name, args=args, actions_before=actions_before.rstrip('\n'), actions_after=actions_after.rstrip('\n'))
error_check = '''if ({target_name_flag}/=0) then
write({target_name_msg},'(a)') "An error occured in {subroutine_name}"
{target_name_msg} = "An error occured in {subroutine_name}: " // trim({target_name_msg})
ierr={target_name_flag}
return
end if
Expand Down

0 comments on commit f3cad9e

Please sign in to comment.