Skip to content

Commit c05a305

Browse files
committed
nix: simplify maintainers handling (repology/repology-rs#343)
We don't need unicalization here
1 parent 499ded6 commit c05a305

File tree

1 file changed

+3
-5
lines changed
  • repology/parsers/parsers

1 file changed

+3
-5
lines changed

repology/parsers/parsers/nix.py

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -219,14 +219,12 @@ def iter_parse(self, path: str, factory: PackageFactory) -> Iterable[PackageMake
219219
pkg.set_summary(meta['description'].replace('\n', ' '))
220220

221221
if 'maintainers' in meta:
222-
# unlike dict, python set does not preserve order which is required here
223-
# so simulate set with dict until we drop python implementation
224-
maintainers = {m: None for m in extract_nix_maintainers(meta['maintainers'])}
222+
maintainers = list(extract_nix_maintainers(meta['maintainers']))
225223

226224
if len(maintainers) > 20:
227-
raise RuntimeError(f'too many maintainers ({len(maintainers)}: {", ".join(sorted(maintainers))}) for a single package')
225+
raise RuntimeError(f'too many maintainers ({len(maintainers)}: {", ".join(maintainers)}) for a single package')
228226

229-
pkg.add_maintainers(maintainers.keys())
227+
pkg.add_maintainers(maintainers)
230228
max_maintainers = max(max_maintainers, len(maintainers))
231229

232230
if 'license' in meta:

0 commit comments

Comments
 (0)