Function error in dev-branch
Closed this issue · 3 comments
philmmanjaro commented
@fhdk: please recheck our dev-branch. Seems we have a regression:
(47/50) Aktualisiere pacman-mirrors-dev [######################] 100%
Traceback (most recent call last):
File "/usr/bin/pacman-mirrors", line 55, in <module>
pm.run()
File "/usr/lib/python3.6/site-packages/pacman_mirrors/pacman_mirrors.py", line 552, in run
self.command_line_parse()
File "/usr/lib/python3.6/site-packages/pacman_mirrors/pacman_mirrors.py", line 205, in command_line_parse
if args.sync:
AttributeError: 'Namespace' object has no attribute 'sync'
Stable branches are fine.
philmmanjaro commented
Please also check v4.1 series, as some checks failed ...
fhdk commented
I removed the -y --sync arg - I realized it was confusing and also it is not a pacman clone but a mirrorlist helper and the -u --update should not have been there at all.
I must keep the purpose of pacman-mirrors as it is - mirrorlist generator - not a nurse :)
This regression is fixed with 8939ed6
fhdk commented
I have checked - I fixed another bug in 4.1-series.
Status
For 4.1.3-stable
- Fixed comparison of mirror files. The comparison of the newly downloaded and the existing mirror file failed due to a bad comparison by me. Found that the builtin function
filecmp
would do a much better job.
For 4.2.0-dev
- Same as 4.1.2 fix
- Same as 4.1.3 fix
- regression bug fix (described above)
- fixed some sorting and output problems related to protocol selection - partly relating to the 4.1.2 fix
- fixed failing test - the test failed since no access to /tmp - placed the temp file in /var/lib/pacman-mirrors instead