Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add dnf option --noclean (RhBug:1361424) #867

Closed
wants to merge 1 commit into from

Conversation

j-mracek
Copy link
Contributor

The option sets clean_requirements_on_remove to false from commandline. The
cleaning is pretty strong behavior that is handy to be easily be switched off.

https://bugzilla.redhat.com/show_bug.cgi?id=1361424

@@ -262,6 +262,9 @@ def _main_parser(self):
metavar='[repo,path]',
help=_("label and path to additional repository,"
" can be specified multiple times."))
main_parser.add_argument("--noclean", action="store_false",
Copy link
Contributor

Choose a reason for hiding this comment

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

I think better option name would be --no-autoremove

@ignatenkobrain
Copy link
Contributor

Cleaning is very vague term

@ignatenkobrain ignatenkobrain self-assigned this Jul 18, 2017
@j-mracek
Copy link
Contributor Author

It could be --noautoremove like --nogpgcheck

@ignatenkobrain
Copy link
Contributor

@j-mracek fine with me

The option sets clean_requirements_on_remove to false from commandline. The
cleaning is pretty strong behavior that is handy to be easily be switched off.

https://bugzilla.redhat.com/show_bug.cgi?id=1361424
@ignatenkobrain
Copy link
Contributor

@dnf-bot r+

@dnf-bot
Copy link
Member

dnf-bot commented Jul 18, 2017

📌 Commit d6111c6 has been approved by ignatenkobrain

@ignatenkobrain
Copy link
Contributor

obviously, having test for new option would be good, but not blocking on it now

@dnf-bot
Copy link
Member

dnf-bot commented Jul 18, 2017

⌛ Testing commit d6111c6 with merge 4329b00...

@dnf-bot
Copy link
Member

dnf-bot commented Jul 18, 2017

☀️ Test successful - status-jenkins
Approved by: ignatenkobrain
Pushing 4329b00 to master...

@dnf-bot dnf-bot closed this in 4329b00 Jul 18, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants