From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from lists.gentoo.org (pigeon.gentoo.org [208.92.234.80]) by finch.gentoo.org (Postfix) with ESMTP id 9093E13888F for ; Sun, 11 Oct 2015 06:29:55 +0000 (UTC) Received: from pigeon.gentoo.org (localhost [127.0.0.1]) by pigeon.gentoo.org (Postfix) with SMTP id 6538721C02B; Sun, 11 Oct 2015 06:29:22 +0000 (UTC) Received: from smtp.gentoo.org (smtp.gentoo.org [140.211.166.183]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by pigeon.gentoo.org (Postfix) with ESMTPS id 0717021C02B for ; Sun, 11 Oct 2015 06:29:16 +0000 (UTC) Received: from localhost.localdomain (localhost [127.0.0.1]) by smtp.gentoo.org (Postfix) with ESMTP id BB83D340634 for ; Sun, 11 Oct 2015 06:29:15 +0000 (UTC) From: Mike Frysinger To: gentoo-catalyst@lists.gentoo.org Subject: [gentoo-catalyst] [PATCH 01/10] stage1: convert to log module Date: Sun, 11 Oct 2015 02:29:03 -0400 Message-Id: <1444544952-32408-1-git-send-email-vapier@gentoo.org> X-Mailer: git-send-email 2.5.2 Precedence: bulk List-Post: List-Help: List-Unsubscribe: List-Subscribe: List-Id: Gentoo Linux mail X-BeenThere: gentoo-catalyst@lists.gentoo.org Reply-to: gentoo-catalyst@lists.gentoo.org X-Archives-Salt: 0ef82ed5-d12d-4cf9-881a-b9a501907637 X-Archives-Hash: 329d0ff2eeac7f7104070fcd73076fcc --- catalyst/targets/stage1.py | 38 ++++++++++++++++---------------------- 1 file changed, 16 insertions(+), 22 deletions(-) diff --git a/catalyst/targets/stage1.py b/catalyst/targets/stage1.py index b5dfac7..18ef520 100644 --- a/catalyst/targets/stage1.py +++ b/catalyst/targets/stage1.py @@ -5,6 +5,9 @@ stage1 target import os +from snakeoil import fileutils + +from catalyst import log from catalyst.support import normpath from catalyst.fileops import ensure_dirs from catalyst.base.stagebase import StageBase @@ -22,12 +25,12 @@ class stage1(StageBase): def set_stage_path(self): self.settings["stage_path"]=normpath(self.settings["chroot_path"]+self.settings["root_path"]) - print "stage1 stage path is "+self.settings["stage_path"] + log.notice('stage1 stage path is %s', self.settings['stage_path']) def set_root_path(self): # sets the root path, relative to 'chroot_path', of the stage1 root self.settings["root_path"]=normpath("/tmp/stage1root") - print "stage1 root path is "+self.settings["root_path"] + log.info('stage1 root path is %s', self.settings['root_path']) def set_cleanables(self): StageBase.set_cleanables(self) @@ -55,10 +58,10 @@ class stage1(StageBase): def set_portage_overlay(self): StageBase.set_portage_overlay(self) if "portage_overlay" in self.settings: - print "\nWARNING !!!!!" - print "\tUsing an portage overlay for earlier stages could cause build issues." - print "\tIf you break it, you buy it. Don't complain to us about it." - print "\tDont say we did not warn you\n" + log.warning( + 'Using an overlay for earlier stages could cause build issues.\n' + "If you break it, you buy it. Don't complain to us about it.\n" + "Don't say we did not warn you.") def base_dirs(self): if os.uname()[0] == "FreeBSD": @@ -67,22 +70,13 @@ class stage1(StageBase): # since proc and dev are not writeable, so...create them here ensure_dirs(self.settings["stage_path"]+"/proc") ensure_dirs(self.settings["stage_path"]+"/dev") - if not os.path.isfile(self.settings["stage_path"]+"/proc/.keep"): - try: - proc_keepfile = open(self.settings["stage_path"]+"/proc/.keep","w") - proc_keepfile.write('') - proc_keepfile.close() - except IOError: - print "!!! Failed to create %s" % (self.settings["stage_path"]+"/dev/.keep") - if not os.path.isfile(self.settings["stage_path"]+"/dev/.keep"): - try: - dev_keepfile = open(self.settings["stage_path"]+"/dev/.keep","w") - dev_keepfile.write('') - dev_keepfile.close() - except IOError: - print "!!! Failed to create %s" % (self.settings["stage_path"]+"/dev/.keep") - else: - pass + for f in ('/proc', '/dev'): + f = self.settings['stage_path'] + f + '/.keep' + if not os.path.isfile(f): + try: + fileutils.touch(f) + except IOError: + log.error('Failed to create %s', f) def set_mounts(self): # stage_path/proc probably doesn't exist yet, so create it -- 2.5.2