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

Create yummain.py #4

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Create yummain.py #4

wants to merge 1 commit into from

Conversation

krisnova
Copy link
Contributor

@krisnova krisnova commented Feb 3, 2019

Stolen from yum

Stolen from yum
try:
probs = base._rpmdb_warn_checks(out=verbose_logger.info, warn=False)
except Errors.YumBaseError, e:
# This is mainly for PackageSackError from rpmdb.

Choose a reason for hiding this comment

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

Hi, just checking in. Meeps Meeps... this looks weird...

logger.critical("%s", msg)
rpmdb_warn_checks()
return_code = result
if base._ts_save_file:

Choose a reason for hiding this comment

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

Hi...🦄 So I read your PR... Frank is confused... 🌈🐼

if not base.conf.skip_broken:
verbose_logger.info(_(" You could try using --skip-broken to work around the problem"))
rpmdb_warn_checks()
if unlock(): return 200

Choose a reason for hiding this comment

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

Hi...🦄 So I read your PR... Hrmm.. this looks funny...

logger = logging.getLogger("yum.main")
verbose_logger = logging.getLogger("yum.verbose.main")

# Try to open the current directory to see if we have

Choose a reason for hiding this comment

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

Hi...🦄 So I read your PR... Woah... this old bit again... 🦄

# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
# GNU Library General Public License for more details.
#
# You should have received a copy of the GNU General Public License

Choose a reason for hiding this comment

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

Nit... but... Frank is confused... 🌈🐼


# Try to open the current directory to see if we have
# read and execute access. If not, chdir to /
try:

Choose a reason for hiding this comment

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

Hi...🦄 So I read your PR... Something looks weird here. Are you sure about this? 🌈

return 1
elif result == 2:
# Continue on
pass

Choose a reason for hiding this comment

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

Hrmmm... Meeps Meeps... this looks weird...

Copy link
Contributor Author

Choose a reason for hiding this comment

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

thanks franks

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.

2 participants