From: Paul Eggleton <paul.eggleton@linux.intel.com>
To: openembedded-core@lists.openembedded.org
Subject: [PATCH 1/9] classes/buildhistory: remove obsolete flat package history feature
Date: Thu, 2 Aug 2012 10:23:01 +0100 [thread overview]
Message-ID: <0ccadc021a8d55497e329414e2ae8a4486e2fa65.1343899228.git.paul.eggleton@linux.intel.com> (raw)
In-Reply-To: <cover.1343899228.git.paul.eggleton@linux.intel.com>
In-Reply-To: <cover.1343899228.git.paul.eggleton@linux.intel.com>
Remove the ability to set BUILDHISTORY_KEEP_VERSIONS = "1" to save the
package history as flat versioned files rather than relying on git to
keep previous versions of the package information. git has proven to
work quite well in this capacity.
Signed-off-by: Paul Eggleton <paul.eggleton@linux.intel.com>
---
meta/classes/buildhistory.bbclass | 43 +++++++------------------------------
1 file changed, 8 insertions(+), 35 deletions(-)
diff --git a/meta/classes/buildhistory.bbclass b/meta/classes/buildhistory.bbclass
index ddb76e8..f5494cd 100644
--- a/meta/classes/buildhistory.bbclass
+++ b/meta/classes/buildhistory.bbclass
@@ -156,7 +156,6 @@ python buildhistory_emit_pkghistory() {
rcpinfo.depends = sortlist(squashspaces(d.getVar('DEPENDS', True) or ""))
rcpinfo.packages = packages
write_recipehistory(rcpinfo, d)
- write_latestlink(None, pe, pv, pr, d)
# Apparently the version can be different on a per-package basis (see Python)
pkgdest = d.getVar('PKGDEST', True)
@@ -199,8 +198,6 @@ python buildhistory_emit_pkghistory() {
pkginfo.filelist = " ".join(filelist)
write_pkghistory(pkginfo, d)
-
- write_latestlink(pkg, pe, pv, pr, d)
}
@@ -212,8 +209,8 @@ def write_recipehistory(rcpinfo, d):
if not os.path.exists(pkghistdir):
os.makedirs(pkghistdir)
- verfile = os.path.join(pkghistdir, "%s:%s-%s" % (rcpinfo.pe, rcpinfo.pv, rcpinfo.pr))
- f = open(verfile, "w")
+ infofile = os.path.join(pkghistdir, "latest")
+ f = open(infofile, "w")
try:
if rcpinfo.pe != "0":
f.write("PE = %s\n" % rcpinfo.pe)
@@ -226,16 +223,16 @@ def write_recipehistory(rcpinfo, d):
def write_pkghistory(pkginfo, d):
- bb.debug(2, "Writing package history")
+ bb.debug(2, "Writing package history for package %s" % pkginfo.name)
pkghistdir = d.getVar('BUILDHISTORY_DIR_PACKAGE', True)
- verpath = os.path.join(pkghistdir, pkginfo.name)
- if not os.path.exists(verpath):
- os.makedirs(verpath)
+ pkgpath = os.path.join(pkghistdir, pkginfo.name)
+ if not os.path.exists(pkgpath):
+ os.makedirs(pkgpath)
- verfile = os.path.join(verpath, "%s:%s-%s" % (pkginfo.pe, pkginfo.pv, pkginfo.pr))
- f = open(verfile, "w")
+ infofile = os.path.join(pkgpath, "latest")
+ f = open(infofile, "w")
try:
if pkginfo.pe != "0":
f.write("PE = %s\n" % pkginfo.pe)
@@ -250,30 +247,6 @@ def write_pkghistory(pkginfo, d):
f.close()
-def write_latestlink(pkg, pe, pv, pr, d):
- import shutil
-
- pkghistdir = d.getVar('BUILDHISTORY_DIR_PACKAGE', True)
-
- def rm_link(path):
- try:
- os.unlink(path)
- except OSError:
- return
-
- if pkg:
- filedir = os.path.join(pkghistdir, pkg)
- else:
- filedir = pkghistdir
- latest_file = os.path.join(filedir, "latest")
- ver_file = os.path.join(filedir, "%s:%s-%s" % (pe, pv, pr))
- rm_link(latest_file)
- if d.getVar('BUILDHISTORY_KEEP_VERSIONS', True) == '1':
- shutil.copy(ver_file, latest_file)
- else:
- shutil.move(ver_file, latest_file)
-
-
buildhistory_get_image_installed() {
# Anything requiring the use of the packaging system should be done in here
# in case the packaging files are going to be removed for this image
--
1.7.9.5
next prev parent reply other threads:[~2012-08-02 9:34 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-08-02 9:23 [PATCH 0/9] buildhistory improvements Paul Eggleton
2012-08-02 9:23 ` Paul Eggleton [this message]
2012-08-02 9:23 ` [PATCH 2/9] classes/buildhistory: ensure old package info is removed Paul Eggleton
2012-08-02 9:23 ` [PATCH 3/9] classes/buildhistory: remove unused functions Paul Eggleton
2012-08-02 9:23 ` [PATCH 4/9] classes/buildhistory: save preinst/postinst/prerm/postrm Paul Eggleton
2012-08-02 9:23 ` [PATCH 5/9] classes/buildhistory: record PKG/PKGE/PKGV/PKGR Paul Eggleton
2012-08-02 9:23 ` [PATCH 6/9] scripts: add buildhistory-tag script Paul Eggleton
2012-08-02 10:18 ` Burton, Ross
2012-08-02 10:49 ` Paul Eggleton
2012-08-02 11:12 ` Burton, Ross
2012-08-02 9:23 ` [PATCH 7/9] buildhistory_analysis: tidy up duplicate split_version function Paul Eggleton
2012-08-02 9:23 ` [PATCH 8/9] buildhistory_analysis: ignore removal of self-dependencies Paul Eggleton
2012-08-02 9:23 ` [PATCH 9/9] classes/buildhistory: save metadata revisions Paul Eggleton
2012-08-02 11:27 ` [PATCH 0/9] buildhistory improvements Burton, Ross
2012-08-02 12:58 ` Martin Jansa
2012-08-02 16:14 ` Andreas Müller
2012-08-06 14:29 ` Richard Purdie
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=0ccadc021a8d55497e329414e2ae8a4486e2fa65.1343899228.git.paul.eggleton@linux.intel.com \
--to=paul.eggleton@linux.intel.com \
--cc=openembedded-core@lists.openembedded.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox