decathorpe/rust2rpm
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Use just a single field for handling of --build-requires

Browse source 
Having two fields in the args structure is super ugly. Let's just
use one field, and assign True or False to it.

No change in function.
This commit is contained in:
Zbigniew Jędrzejewski-Szmek 2022-07-03 15:55:33 +02:00
parent 1d225f99b1
commit 91907b066f
1 changed files with 5 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
rust2rpm/__main__.py
View file

@ -391,8 +391,10 @@ def main():
parser.add_argument("--all-features", action="store_true", parser.add_argument("--all-features", action="store_true",
help="Activate all available features") help="Activate all available features")
parser.add_argument("--dynamic-buildrequires", action="store_true", parser.add_argument("--dynamic-buildrequires", action="store_true",
default=None,
help="Use dynamic BuildRequires feature") help="Use dynamic BuildRequires feature")
parser.add_argument("--no-dynamic-buildrequires", action="store_true", parser.add_argument("--no-dynamic-buildrequires", action="store_false",
dest="dynamic_buildrequires",
help="Do not use dynamic BuildRequires feature") help="Do not use dynamic BuildRequires feature")
parser.add_argument("--suffix", action="store", parser.add_argument("--suffix", action="store",
help="Package suffix") help="Package suffix")
@ -497,8 +499,8 @@ def main():
else: else:
kwargs["pkg_release"] = "1%{?dist}" kwargs["pkg_release"] = "1%{?dist}"
if args.target == "fedora" and not args.no_dynamic_buildrequires: if args.dynamic_buildrequires is None:
args.dynamic_buildrequires = True args.dynamic_buildrequires = args.target == "fedora"
kwargs["generate_buildrequires"] = args.dynamic_buildrequires kwargs["generate_buildrequires"] = args.dynamic_buildrequires