diff options
author | Martin Braun <martin.braun@ettus.com> | 2020-06-16 17:18:16 +0200 |
---|---|---|
committer | Aaron Rossetto <aaron.rossetto@ni.com> | 2020-06-22 14:38:12 -0500 |
commit | 72c9bb367df0561434f4d4b89c404fb817889738 (patch) | |
tree | dd2628c1c0659da5a97cea4e39dc8e2cb081eb5a | |
parent | aad2ccfa7ceebfabde5606893cd4bc7f3a84e40b (diff) | |
download | uhd-72c9bb367df0561434f4d4b89c404fb817889738.tar.gz uhd-72c9bb367df0561434f4d4b89c404fb817889738.tar.bz2 uhd-72c9bb367df0561434f4d4b89c404fb817889738.zip |
utils: update_fbs.py: Minor refactor, fix path bug
- The UHD auto-detection was broken. Now it can find UHD in the same
directory.
- The main() function was split into main() and run(), which would allow
loading this module and calling run() as it's own function.
-rwxr-xr-x | host/utils/update_fbs.py | 24 |
1 files changed, 15 insertions, 9 deletions
diff --git a/host/utils/update_fbs.py b/host/utils/update_fbs.py index d97e120a4..5678d6414 100755 --- a/host/utils/update_fbs.py +++ b/host/utils/update_fbs.py @@ -53,7 +53,7 @@ def find_uhd_source_path(hint): "Invalid UHD source path: {} (does not have subdir: {})" .format(hint, os.path.join(*CAL_SUBDIR))) # If there's no hint, we try our own path as a hint: - return find_uhd_source_path(str(pathlib.Path().absolute().parent)) + return find_uhd_source_path(str(pathlib.Path(__file__).parent.absolute().parent)) def parse_args(): """ Parse args and return args object """ @@ -104,23 +104,29 @@ def verify_fbs(flatc_exe, files): finally: shutil.rmtree(tmp_dir, ignore_errors=True) -def main(): - """ Go, go, go! """ - args = parse_args() - flatc_exe = find_flatc(args.flatc_exe) - print("Found flatc executable: {}".format(flatc_exe)) +def run(flatc_exe, uhd_path=None, verify=False): + """ + The actual code, minus arg parsing + """ try: - cal_path = os.path.join(find_uhd_source_path(args.uhd_path), *CAL_SUBDIR) + cal_path = os.path.join(find_uhd_source_path(uhd_path), *CAL_SUBDIR) except RuntimeError as ex: print("ERROR: {}".format(str(ex))) return False print("Checking UHD cal data in: {}".format(cal_path)) os.chdir(cal_path) files = glob.glob("*.fbs") - if args.verify: - return not verify_fbs(flatc_exe, files) + if verify: + return verify_fbs(flatc_exe, files) subprocess.check_call([flatc_exe, '--cpp'] + files) return True +def main(): + """ Go, go, go! """ + args = parse_args() + flatc_exe = find_flatc(args.flatc_exe) + print("Found flatc executable: {}".format(flatc_exe)) + return run(flatc_exe, uhd_path=args.uhd_path, verify=args.verify) + if __name__ == "__main__": sys.exit(not main()) |