Skip to content
This repository has been archived by the owner on Apr 9, 2022. It is now read-only.

Remove Fetchmirrors #895

Merged
merged 5 commits into from
Oct 16, 2019
Merged

Remove Fetchmirrors #895

merged 5 commits into from
Oct 16, 2019

Conversation

herobrauni
Copy link
Contributor

@herobrauni herobrauni commented Oct 16, 2019

Description

Removes packet "fetchmirrors" and changes the variable to "update_mirrors" to avoid confusion.

Affected issues

Closes #842

Pull request checklist

  • I have tested my code
  • I have read the contributing guide
  • I have followed best practices and commented my code well

Copy link
Member

@erazemk erazemk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you rename fetchmirrors to update_mirrors_msg to make it more clear what it does in the code.

…" to avoid duplicate with the function "update_mirrors"
@herobrauni
Copy link
Contributor Author

I updated the variable to "update_mirrors_msg".

I was thinking about adding a new option "anarchy -m" that updates the mirrors before the installation, in the same way fetchmirrors did it before, even though it has very limited usecases.

Would you want something like that?

@erazemk
Copy link
Member

erazemk commented Oct 16, 2019

I don't think we need that right now.
Hopefully we'll figure out how to fix #765 and then it won't matter anyway.

@erazemk erazemk merged commit 819d303 into AnarchyLinux:master Oct 16, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Remove fetchmirrors
2 participants