Update install.py #623
Update install.py #623
Conversation
Add Antergos Repo and Staging (Commented) to the proper spot.
|
@lots0logs It would be below testing, don't want that. How this merge looks: (You made a comment note then deleted it...?) |
| @@ -774,10 +774,10 @@ def update_pacman_conf(): | |||
| else: | |||
There was a problem hiding this comment.
Let's do this instead:
elif pacline == '[testing]\n':
pacman_file.write('\n\n#[antergos-staging]\n')
pacman_file.write('#SigLevel = PackageRequired\n')
pacman_file.write('#Include = /etc/pacman.d/antergos-mirrorlist\n\n')
pacman_file.write('[antergos]\n')
pacman_file.write('SigLevel = PackageRequired\n')
pacman_file.write('Include = /etc/pacman.d/antergos-mirrorlist\n\n')
pacman_file.write(pacline)
else:
pacman_file.write(pacline)Its kind of ugly but since I am trying to write this on github as feedback for you instead of doing it myself in the code its the best I can do 😉
There was a problem hiding this comment.
@lots0logs It's still after testing, it shouldn't be since the main priority is Antergos. If testing was used staging would be used at the same time.
There was a problem hiding this comment.
No it shouldnt be. Maybe there is a typo or something. I cant check it now. But the code I provided specifically writes the testing line after writing the antergos lines.
There was a problem hiding this comment.
better but we dont want to have lines longer than 99 characters as that is against Python Coding Standards. That's why I recommended doing it the way that I did above.
There was a problem hiding this comment.
Done.
99 characters as that is against Python Coding Standards
That'll be very hard for me to follow, bad habits catching up with me 👀.
Added to loop
Character per line limit
Added Antergos Repo and Staging (Commented) to the proper spot in
/etc/pacman.conf.