diff options
author | Martin Braun <martin.braun@ettus.com> | 2022-03-31 13:43:23 +0200 |
---|---|---|
committer | Aaron Rossetto <aaron.rossetto@ni.com> | 2022-03-31 13:52:54 -0700 |
commit | e4c73e85913b0a635b1eb0ef4e6813e8385aa887 (patch) | |
tree | 7c352088d253340cf83939445727d05a14551962 /tools | |
parent | 002ff9698e09b94c396736613e493f79e9c56442 (diff) | |
download | uhd-e4c73e85913b0a635b1eb0ef4e6813e8385aa887.tar.gz uhd-e4c73e85913b0a635b1eb0ef4e6813e8385aa887.tar.bz2 uhd-e4c73e85913b0a635b1eb0ef4e6813e8385aa887.zip |
tools: uhd_ubuntu_deb: Require --buildpath
Because this script deletes the entire contents of --buildpath, we no
longer provide '..' as the default. First, it is not a suitable default,
because it will contain the directory from which this script is called.
Second, if someone runs this script from a source repo for UHD, and this
repo is checked out into a non-empty directory, the script will still,
by default, delete all the contents from that directory.
Now, the script fails when not providing --buildpath. Note that CI code
is already using this command line argument.
Diffstat (limited to 'tools')
-rw-r--r-- | tools/debs/uhd_ubuntu_deb.py | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/tools/debs/uhd_ubuntu_deb.py b/tools/debs/uhd_ubuntu_deb.py index fb1b988f9..3770b57d2 100644 --- a/tools/debs/uhd_ubuntu_deb.py +++ b/tools/debs/uhd_ubuntu_deb.py @@ -11,7 +11,6 @@ # See here for more on pbuilder: https://wiki.ubuntu.com/PbuilderHowto import argparse -import glob import os import pathlib import re @@ -136,8 +135,9 @@ if __name__ == "__main__": help="Uploads to launchpad. Requires--sign") parser.add_argument("--nobuild", action='store_true', help="Disables building using pbuilder") - parser.add_argument("--buildpath", type=str, - help="Output path for build files", default="..") + parser.add_argument("--buildpath", type=str, required=True, + help="Output path for build files. " + "Will get nuked before creating packages.") parser.add_argument("release", type=str, help="Ubuntu release version. This must match pbuilder create --distribution if building.") args = parser.parse_args() |