Skip to content

Commit a43660f

Browse files
committed
Do not return nameless package with deps #3072
Signed-off-by: Jono Yang <jyang@nexb.com>
1 parent 5ccee5f commit a43660f

File tree

1 file changed

+1
-26
lines changed

1 file changed

+1
-26
lines changed

src/packagedcode/rubygems.py

Lines changed: 1 addition & 26 deletions
Original file line numberDiff line numberDiff line change
@@ -252,32 +252,7 @@ class GemfileLockHandler(BaseGemProjectHandler):
252252
@classmethod
253253
def parse(cls, location):
254254
gemfile_lock = GemfileLockParser(location)
255-
dependencies = []
256255
all_gems = list(gemfile_lock.all_gems.values())
257-
for gem in all_gems:
258-
dependencies.append(
259-
models.DependentPackage(
260-
purl=PackageURL(
261-
type='gem',
262-
name=gem.name,
263-
version=gem.version
264-
).to_string(),
265-
extracted_requirement=', '.join(gem.requirements),
266-
# FIXME: get proper scope... This does not seem right
267-
scope='dependencies',
268-
is_runtime=True,
269-
is_optional=False,
270-
is_resolved=True,
271-
)
272-
)
273-
274-
yield models.PackageData(
275-
datasource_id=cls.datasource_id,
276-
type=cls.default_package_type,
277-
dependencies=dependencies,
278-
primary_language=cls.default_primary_language,
279-
)
280-
281256
if not all_gems:
282257
return
283258

@@ -294,7 +269,7 @@ def parse(cls, location):
294269
is_runtime=True,
295270
is_optional=False,
296271
is_resolved=True,
297-
) for dep in main_gem.dependencies.values()
272+
) for dep in all_gems[1:]
298273
]
299274
urls = get_urls(main_gem.name, main_gem.version)
300275

0 commit comments

Comments
 (0)