Skip to content

Commit

Permalink
DM tools scraper: Add new required flags. (#31078)
Browse files Browse the repository at this point in the history
In 1.2.0, the scraper used positional arguments, now it uses
flags. This is better, IMO. Moving the script to the new format
so we can get some of the improvements from the newer scraper
revisions.
  • Loading branch information
cecille authored and samryanuofl committed Jan 12, 2024
1 parent b0ec817 commit 3792981
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions scripts/spec_xml/generate_spec_xml.py
Original file line number Diff line number Diff line change
Expand Up @@ -79,7 +79,7 @@ def scrape_clusters(scraper, spec_root, output_dir, dry_run):

def scrape_cluster(filename: str) -> None:
xml_path = get_xml_path(filename, clusters_output_dir)
cmd = [scraper, 'cluster', filename, xml_path, '-nd']
cmd = [scraper, 'cluster', '-i', filename, '-o', xml_path, '-nd']
if dry_run:
print(cmd)
else:
Expand Down Expand Up @@ -109,7 +109,7 @@ def scrape_device_types(scraper, spec_root, output_dir, dry_run):

def scrape_device_type(filename: str) -> None:
xml_path = get_xml_path(filename, device_types_output_dir)
cmd = [scraper, 'devicetype', '-c', clusters_output_dir, '-nd', filename, xml_path]
cmd = [scraper, 'devicetype', '-c', clusters_output_dir, '-nd', '-i', filename, '-o', xml_path]
if dry_run:
print(cmd)
else:
Expand Down

0 comments on commit 3792981

Please sign in to comment.