Skip to content

Commit 74bd21c

Browse files
committed
Create purls.yml path without leading segment #660
* leading segment will be the repo itself, not a directory within the repo Signed-off-by: Jono Yang <[email protected]>
1 parent dda2bca commit 74bd21c

File tree

2 files changed

+12
-9
lines changed

2 files changed

+12
-9
lines changed

minecode_pipelines/miners/__init__.py

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,7 @@
1313
from pathlib import Path
1414

1515
from aboutcode import hashid
16+
from packageurl import PackageURL
1617
from scanpipe.pipes import federatedcode
1718

1819

@@ -31,7 +32,9 @@ def write_data_to_file(path, data):
3132

3233
def write_purls_to_repo(repo, package, packages, push_commit=False):
3334
# save purls to yaml
34-
ppath = hashid.get_package_purls_yml_file_path(package)
35+
path_elements = hashid.package_path_elements(package)
36+
_, core_path, _, _ = path_elements
37+
ppath = core_path / hashid.PURLS_FILENAME
3538
purls = [p.purl for p in packages]
3639
federatedcode.write_data_as_yaml(
3740
base_path=repo.working_dir,

minecode_pipelines/pipes/maven.py

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -692,15 +692,15 @@ def get_packages(self, content=None, increments=False):
692692
def collect_packages_from_maven(commits_per_push=10, logger=None):
693693
# download and iterate through maven nexus index
694694
maven_nexus_collector = MavenNexusCollector()
695-
prev_package = None
695+
prev_purl = None
696696
current_packages = []
697-
for i, (current_package, package) in enumerate(maven_nexus_collector.get_packages(), start=1):
698-
if not prev_package:
699-
prev_package = current_package
700-
elif prev_package != current_package:
697+
for i, (current_purl, package) in enumerate(maven_nexus_collector.get_packages(), start=1):
698+
if not prev_purl:
699+
prev_purl = current_purl
700+
elif prev_purl != current_purl:
701701
# check out repo
702702
repo_url, _ = federatedcode.get_package_repository(
703-
project_purl=prev_package,
703+
project_purl=prev_purl,
704704
logger=logger
705705
)
706706
repo = federatedcode.clone_repository(
@@ -712,7 +712,7 @@ def collect_packages_from_maven(commits_per_push=10, logger=None):
712712
# save purls to yaml
713713
write_purls_to_repo(
714714
repo=repo,
715-
package=prev_package,
715+
package=prev_purl,
716716
packages=current_packages,
717717
push_commit=push_commit
718718
)
@@ -721,5 +721,5 @@ def collect_packages_from_maven(commits_per_push=10, logger=None):
721721
federatedcode.delete_local_clone(repo)
722722

723723
current_packages = []
724-
prev_package = current_package
724+
prev_purl = current_purl
725725
current_packages.append(package)

0 commit comments

Comments
 (0)