]> code.ossystems Code Review - openembedded-core.git/commitdiff
Pseudo: Update to 1.3
authorPeter Seebach <peter.seebach@windriver.com>
Tue, 27 Mar 2012 21:57:30 +0000 (16:57 -0500)
committerRichard Purdie <richard.purdie@linuxfoundation.org>
Wed, 28 Mar 2012 08:33:01 +0000 (09:33 +0100)
The various local patches have made it into upstream, so we update
the build files and jump to pseudo 1.3.  This also includes a popen()
fix which fixes some edge cases that caused failures trying to check
git branches and the like.

[Yocto bug #2181]

Signed-off-by: Seebs <peter.seebach@windriver.com>
Updated the pseudo_git.bb to match.

Signed-off-by: Mark Hatle <mark.hatle@windriver.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
meta/recipes-devtools/pseudo/pseudo.inc
meta/recipes-devtools/pseudo/pseudo/oe-config.patch [deleted file]
meta/recipes-devtools/pseudo/pseudo/opendir.patch [deleted file]
meta/recipes-devtools/pseudo/pseudo/renameat.patch [deleted file]
meta/recipes-devtools/pseudo/pseudo/static_sqlite.patch [deleted file]
meta/recipes-devtools/pseudo/pseudo_1.2.bb [deleted file]
meta/recipes-devtools/pseudo/pseudo_1.3.bb [new file with mode: 0644]
meta/recipes-devtools/pseudo/pseudo_git.bb

index 4a3ef6f1ce42d66771c3a95832cb8afd0d3cc956..664a9b5937049bbc09089b5309b02bb5d1511265 100644 (file)
@@ -29,9 +29,9 @@ NO32LIBS ??= "0"
 # Compile for the local machine arch...
 do_compile () {
        if [ "${SITEINFO_BITS}" == "64" ]; then
-         ${S}/configure --prefix=${prefix} --libdir=${prefix}/lib/pseudo/lib${SITEINFO_BITS} --with-sqlite=${STAGING_DIR_TARGET}${exec_prefix} --bits=${SITEINFO_BITS}
+         ${S}/configure --prefix=${prefix} --libdir=${prefix}/lib/pseudo/lib${SITEINFO_BITS} --with-sqlite=${STAGING_DIR_TARGET}${exec_prefix} --bits=${SITEINFO_BITS} --enable-static-sqlite
        else
-         ${S}/configure --prefix=${prefix} --libdir=${prefix}/lib/pseudo/lib --with-sqlite=${STAGING_DIR_TARGET}${exec_prefix} --bits=${SITEINFO_BITS}
+         ${S}/configure --prefix=${prefix} --libdir=${prefix}/lib/pseudo/lib --with-sqlite=${STAGING_DIR_TARGET}${exec_prefix} --bits=${SITEINFO_BITS} --enable-static-sqlite
        fi
        oe_runmake ${MAKEOPTS}
 }
diff --git a/meta/recipes-devtools/pseudo/pseudo/oe-config.patch b/meta/recipes-devtools/pseudo/pseudo/oe-config.patch
deleted file mode 100644 (file)
index d368281..0000000
+++ /dev/null
@@ -1,58 +0,0 @@
-configure: Fix configuration to match OE-Core requirements
-
-In OE-Core we need to be able to configure for both 32-bit and 64-bit libpseudo
-libraries.  In order to avoid some complex manipulations, we adjust the configure
-and Makefile to facilitate this.
-
-Upstream-Status: Submitted
-
-Signed-off-by: Mark Hatle <mark.hatle@windriver.com>
-
-diff --git a/Makefile.in b/Makefile.in
-index 6c57d43..342d066 100644
---- a/Makefile.in
-+++ b/Makefile.in
-@@ -27,7 +27,7 @@ MARK64=@MARK64@
- RPATH=@RPATH@
- VERSION=1.1.1
--LIB=$(shell basename $(LIBDIR))
-+LIB=@LIB@
- BIN=bin
- LOCALSTATE=var/pseudo
- BINDIR=$(PREFIX)/$(BIN)
-diff --git a/configure b/configure
-index 9a480fb..eeb19f3 100755
---- a/configure
-+++ b/configure
-@@ -80,14 +80,18 @@ if [ -z "$opt_prefix" ]; then
-   usage
- fi
--if [ -n "$opt_bits" ] && [ -n "$opt_libdir" ]; then
--    echo >&2 "Can not specify both --libdir and --bits"
--    exit 1
--fi
- if [ -z "$opt_libdir" ]; then
-     opt_libdir=$opt_prefix/lib$opt_mark64
- fi
-+# We need to find the libdir relative to the prefix, this is required
-+# by the code in pseudo-utils.c that handles relocation.
-+opt_lib=${opt_libdir#$opt_prefix/}
-+if [ "$opt_lib" = "$opt_libdir" ]; then
-+    echo >&2 "libdir must be relative to prefix."
-+    exit 1
-+fi
-+
- if [ ! -f "${opt_sqlite}/include/sqlite3.h" ]; then
-     echo >&2 "SQLite3 headers not found in at ${opt_sqlite}/include/sqlite3.h. Please check that SQLite3 and SQLite3 headers are installed."
-     exit 1
-@@ -107,6 +111,7 @@ fi
- sed -e '
-   s,@PREFIX@,'"$opt_prefix"',g
-   s,@LIBDIR@,'"$opt_libdir"',g
-+  s,@LIB@,'"$opt_lib"',g
-   s,@SUFFIX@,'"$opt_suffix"',g
-   s,@SQLITE@,'"$opt_sqlite"',g
-   s!@RPATH@!'"$opt_rpath"'!g
diff --git a/meta/recipes-devtools/pseudo/pseudo/opendir.patch b/meta/recipes-devtools/pseudo/pseudo/opendir.patch
deleted file mode 100644 (file)
index bc6cec8..0000000
+++ /dev/null
@@ -1,94 +0,0 @@
-commit 162f2692c399b93311652201a940fdaf9c9e6924
-Author: Peter Seebach <peter.seebach@windriver.com>
-Date:   Thu Feb 2 11:45:42 2012 -0600
-
-    Make opendir/closedir stash and forget directory names.
-    
-    The dirfd(DIR *) interface allows you to get the fd for a DIR *,
-    meaning you can use it with openat(), meaning you can need its
-    path.  This causes a segfault.  Also gonna fix the base_path
-    code not to segfault in that case, but first fix the underlying
-    problem.
-
-Upstream-Status: Backport
-
-diff --git a/ChangeLog.txt b/ChangeLog.txt
-index 4de488c..9625b38 100644
---- a/ChangeLog.txt
-+++ b/ChangeLog.txt
-@@ -1,3 +1,7 @@
-+2012-02-02:
-+      * (seebs) stash dir name for DIR * from opendir using dirfd.
-+      * (seebs) add closedir.
-+
- 2011-11-02:
-       * (seebs) Call this 1.2 because the UNLOAD change is moderately
-         significant, and so's the clone change.
-diff --git a/ports/unix/guts/closedir.c b/ports/unix/guts/closedir.c
-new file mode 100644
-index 0000000..1085361
---- /dev/null
-+++ b/ports/unix/guts/closedir.c
-@@ -0,0 +1,20 @@
-+/*
-+ * Copyright (c) 2012 Wind River Systems; see
-+ * guts/COPYRIGHT for information.
-+ *
-+ * static int
-+ * wrap_closedir(DIR *dirp) {
-+ *    int rc = -1;
-+ */
-+      if (!dirp) {
-+              errno = EFAULT;
-+              return -1;
-+      }
-+
-+      int fd = dirfd(dirp);
-+      pseudo_client_op(OP_CLOSE, 0, fd, -1, 0, 0);
-+      rc = real_closedir(dirp);
-+
-+/*    return rc;
-+ * }
-+ */
-diff --git a/ports/unix/guts/opendir.c b/ports/unix/guts/opendir.c
-index 8eaa71f..e69717e 100644
---- a/ports/unix/guts/opendir.c
-+++ b/ports/unix/guts/opendir.c
-@@ -6,8 +6,25 @@
-  * wrap_opendir(const char *path) {
-  *    DIR * rc = NULL;
-  */
-+      struct stat buf;
-+      int save_errno;
-       rc = real_opendir(path);
-+      if (rc) {
-+              int fd;
-+              save_errno = errno;
-+              fd = dirfd(rc);
-+              if (real_fstat(fd, &buf) == -1) {
-+                      pseudo_debug(1, "diropen (fd %d) succeeded, but fstat failed (%s).\n",
-+                              fd, strerror(errno));
-+                      pseudo_client_op_plain(OP_OPEN, PSA_READ, fd, -1, path, 0);
-+              } else {
-+                      pseudo_client_op_plain(OP_OPEN, PSA_READ, fd, -1, path, &buf);
-+              }
-+
-+
-+              errno = save_errno;
-+      }
- /*    return rc;
-  * }
-diff --git a/ports/unix/wrapfuncs.in b/ports/unix/wrapfuncs.in
-index e06e404..32250c4 100644
---- a/ports/unix/wrapfuncs.in
-+++ b/ports/unix/wrapfuncs.in
-@@ -21,6 +21,7 @@ long pathconf(const char *path, int name);
- char *realpath(const char *name, char *resolved_name); /* version="GLIBC_2.3" */
- int remove(const char *path); /* flags=AT_SYMLINK_NOFOLLOW */
- DIR *opendir(const char *path);
-+int closedir(DIR *dirp);
- char *tempnam(const char *template, const char *pfx);
- char *tmpnam(char *s);
- int truncate(const char *path, off_t length);
diff --git a/meta/recipes-devtools/pseudo/pseudo/renameat.patch b/meta/recipes-devtools/pseudo/pseudo/renameat.patch
deleted file mode 100644 (file)
index a2b2cbc..0000000
+++ /dev/null
@@ -1,229 +0,0 @@
-commit 795f2b44b7f692151556782f142a4a6e7d45d892
-Author: Peter Seebach <peter.seebach@windriver.com>
-Date:   Thu Feb 2 15:49:21 2012 -0600
-
-    Implement renameat()
-    
-    After three long years, someone tried to use this.  This was impossibly
-    hard back when pseudo was written, because there was only one dirfd
-    provided for.  Thing is, now, the canonicalization happens in wrapfuncs,
-    so a small tweak to makewrappers to recognize that oldpath should use
-    olddirfd if it exists is enough to get us fully canonicalized paths
-    when needed.
-    
-    Also fix the crash if base_path gets called with an fd for which we have
-    no path.
-
-Upstream-Status: Backport
-
-diff --git a/ChangeLog.txt b/ChangeLog.txt
-index 9625b38..25bd463 100644
---- a/ChangeLog.txt
-+++ b/ChangeLog.txt
-@@ -1,6 +1,9 @@
- 2012-02-02:
-       * (seebs) stash dir name for DIR * from opendir using dirfd.
-       * (seebs) add closedir.
-+      * (seebs) add initial pass at renameat()
-+      * (seebs) in base_path, don't try to strlen the result if
-+        fd_path() returns NULL.
- 2011-11-02:
-       * (seebs) Call this 1.2 because the UNLOAD change is moderately
-diff --git a/makewrappers b/makewrappers
-index 20bbf2b..bf344d6 100755
---- a/makewrappers
-+++ b/makewrappers
-@@ -211,12 +211,13 @@ class Function:
-         self.flags = '0'
-         self.port = port
-         self.directory = ''
--      self.version = 'NULL'
-+        self.version = 'NULL'
-         # On Darwin, some functions are SECRETLY converted to foo$INODE64
-         # when called.  So we have to look those up for real_*
-         self.inode64 = None
-         self.real_func = None
-         self.paths_to_munge = []
-+        self.dirfds = {}
-         self.hand_wrapped = None
-         # used for the copyright date when creating stub functions
-         self.date = datetime.date.today().year
-@@ -239,6 +240,7 @@ class Function:
-         # * If the arg has a name ending in 'path', we will canonicalize it.
-         # * If the arg is named 'dirfd' or 'flags', it becomes the default
-         #   values for the dirfd and flags arguments when canonicalizing.
-+        # * If the name ends in dirfd, we do the same fancy stuff.
-         # * Note that the "comments" field (/* ... */ after the decl) can
-         #   override the dirfd/flags values.
-         self.args = ArgumentList(bits.group(2))
-@@ -246,7 +248,9 @@ class Function:
-             # ignore varargs, they never get these special treatments
-             if arg.vararg:
-                 pass
--            elif arg.name == 'dirfd':
-+            elif arg.name[-5:] == 'dirfd':
-+                if len(arg.name) > 5:
-+                    self.dirfds[arg.name[:-5]] = True
-                 self.dirfd = 'dirfd'
-             elif arg.name == 'flags':
-                 self.flags = 'flags'
-@@ -325,9 +329,13 @@ class Function:
-         """create/allocate canonical paths"""
-         alloc_paths = []
-         for path in self.paths_to_munge:
-+            prefix = path[:-4]
-+          if not prefix in self.dirfds:
-+                prefix = ''
-+            print "for path %s: prefix <%s>" % ( path, prefix )
-             alloc_paths.append(
--                "%s = pseudo_root_path(__func__, __LINE__, %s, %s, %s);" %
--                (path, self.dirfd, path, self.flags))
-+                "%s = pseudo_root_path(__func__, __LINE__, %s%s, %s, %s);" %
-+                (path, prefix, self.dirfd, path, self.flags))
-         return "\n\t\t\t".join(alloc_paths)
-     def free_paths(self):
-diff --git a/ports/unix/guts/renameat.c b/ports/unix/guts/renameat.c
-index c8203b7..f13cd1e 100644
---- a/ports/unix/guts/renameat.c
-+++ b/ports/unix/guts/renameat.c
-@@ -1,15 +1,111 @@
- /* 
-- * Copyright (c) 2008-2010 Wind River Systems; see
-+ * Copyright (c) 2008-2012 Wind River Systems; see
-  * guts/COPYRIGHT for information.
-  *
-  * static int
-  * wrap_renameat(int olddirfd, const char *oldpath, int newdirfd, const char *newpath) {
-  *    int rc = -1;
-  */
-+      pseudo_msg_t *msg;
-+      struct stat oldbuf, newbuf;
-+      int oldrc, newrc;
-+      int save_errno;
-+      int old_db_entry = 0;
--      pseudo_diag("help! unimplemented renameat [%s -> %s].\n", oldpath, newpath);
-+      pseudo_debug(2, "renameat: %d,%s->%d,%s\n",
-+              olddirfd, oldpath ? oldpath : "<nil>",
-+              newdirfd, newpath ? newpath : "<nil>");
-+
-+#ifdef PSEUDO_NO_REAL_AT_FUNCTIONS
-+      if (olddirfd != AT_FDCWD || newdirfd != AT_FDCWD) {
-+              errno = ENOSYS;
-+              return -1;
-+      }
-+#endif
-+
-+      if (!oldpath || !newpath) {
-+              errno = EFAULT;
-+              return -1;
-+      }
-+
-+      save_errno = errno;
-+
-+#ifdef PSEUDO_NO_REAL_AT_FUNCTIONS
-+      newrc = real_lstat(newpath, &newbuf);
-+      oldrc = real_lstat(oldpath, &oldbuf);
-+#else
-+      oldrc = real___fxstatat(_STAT_VER, olddirfd, oldpath, &oldbuf, AT_SYMLINK_NOFOLLOW);
-+      newrc = real___fxstatat(_STAT_VER, newdirfd, newpath, &newbuf, AT_SYMLINK_NOFOLLOW);
-+#endif
-+
-+      errno = save_errno;
-+
-+      /* newpath must be removed. */
-+      /* as with unlink, we have to mark that the file may get deleted */
-+      msg = pseudo_client_op_plain(OP_MAY_UNLINK, 0, -1, newdirfd, newpath, newrc ? NULL : &newbuf);
-+      if (msg && msg->result == RESULT_SUCCEED)
-+              old_db_entry = 1;
-       rc = real_renameat(olddirfd, oldpath, newdirfd, newpath);
-+      save_errno = errno;
-+      if (old_db_entry) {
-+              if (rc == -1) {
-+                      /* since we failed, that wasn't really unlinked -- put
-+                       * it back.
-+                       */
-+                      pseudo_client_op_plain(OP_CANCEL_UNLINK, 0, -1, newdirfd, newpath, &newbuf);
-+              } else {
-+                      /* confirm that the file was removed */
-+                      pseudo_client_op_plain(OP_DID_UNLINK, 0, -1, newdirfd, newpath, &newbuf);
-+              }
-+      }
-+      if (rc == -1) {
-+              /* and we're done. */
-+              errno = save_errno;
-+              return rc;
-+      }
-+      save_errno = errno;
-+      /* nothing to do for a "rename" of a link to itself */
-+      if (newrc != -1 && oldrc != -1 &&
-+          newbuf.st_dev == oldbuf.st_dev &&
-+          newbuf.st_ino == oldbuf.st_ino) {
-+              return rc;
-+        }
-+
-+      /* rename(3) is not mv(1).  rename(file, dir) fails; you must provide
-+       * the corrected path yourself.  You can rename over a directory only
-+       * if the source is a directory.  Symlinks are simply removed.
-+       *
-+       * If we got here, the real rename call succeeded.  That means newpath
-+       * has been unlinked and oldpath has been linked to it.
-+       *
-+       * There are a ton of special cases to error check.  I don't check
-+       * for any of them, because in every such case, the underlying rename
-+       * failed, and there is nothing to do.
-+       * The only tricky part is that, because we used to ignore symlinks,
-+       * we may have to rename or remove directory trees even though in
-+       * theory rename can never destroy a directory tree.
-+       */
-+      if (!old_db_entry) {
-+              /* create an entry under the old name, which will then be
-+               * renamed; this way, children would get renamed too, if there
-+               * were any.
-+               */
-+              if (newrc == 0) {
-+                      if (newbuf.st_dev != oldbuf.st_dev) {
-+                              oldbuf.st_dev = newbuf.st_dev;
-+                              oldbuf.st_ino = newbuf.st_ino;
-+                      }
-+              }
-+              pseudo_debug(1, "creating new '%s' [%llu] to rename\n",
-+                      oldpath, (unsigned long long) oldbuf.st_ino);
-+              pseudo_client_op_plain(OP_LINK, 0, -1, olddirfd, oldpath, &oldbuf);
-+      }
-+      /* special case: use 'fd' for olddirfd, because
-+       * we know it has no other meaning for RENAME
-+       */
-+      pseudo_client_op_plain(OP_RENAME, 0, olddirfd, newdirfd, newpath, &oldbuf, oldpath);
-+      errno = save_errno;
- /*    return rc;
-  * }
-  */
-diff --git a/pseudo_client.c b/pseudo_client.c
-index 48607c2..4a30420 100644
---- a/pseudo_client.c
-+++ b/pseudo_client.c
-@@ -988,6 +988,8 @@ base_path(int dirfd, const char *path, int leave_last) {
-               if (dirfd != -1 && dirfd != AT_FDCWD) {
-                       if (dirfd >= 0) {
-                               basepath = fd_path(dirfd);
-+                      }
-+                      if (basepath) {
-                               baselen = strlen(basepath);
-                       } else {
-                               pseudo_diag("got *at() syscall for unknown directory, fd %d\n", dirfd);
-@@ -1128,7 +1130,10 @@ pseudo_client_op(pseudo_op_t op, int access, int fd, int dirfd, const char *path
-       if (path) {
-               pseudo_debug(2, " %s", path);
-       }
--      if (fd != -1) {
-+      /* for OP_RENAME in renameat, "fd" is also used for the
-+       * second dirfd.
-+       */
-+      if (fd != -1 && op != OP_RENAME) {
-               pseudo_debug(2, " [fd %d]", fd);
-       }
-       if (buf) {
diff --git a/meta/recipes-devtools/pseudo/pseudo/static_sqlite.patch b/meta/recipes-devtools/pseudo/pseudo/static_sqlite.patch
deleted file mode 100644 (file)
index f9223a8..0000000
+++ /dev/null
@@ -1,17 +0,0 @@
-Due to disabling the LD_LIBRARY_PATH handling, we need to use a static
-libsqlite.
-
-Upstream-Status: Inappropriate [configuration]
-
-diff -ur git.orig/Makefile.in git/Makefile.in
---- git.orig/Makefile.in
-+++ git/Makefile.in
-@@ -55,7 +55,7 @@
- # needed for anything that links with pseduo_client.o, pretty much
- CLIENT_LDFLAGS=-ldl -lpthread
--DB_LDFLAGS=-lsqlite3 -lpthread
-+DB_LDFLAGS=$(SQLITE)/lib/libsqlite3.a -lpthread
- PSEUDO=$(BIN)/pseudo
- PSEUDODB=$(BIN)/pseudodb
diff --git a/meta/recipes-devtools/pseudo/pseudo_1.2.bb b/meta/recipes-devtools/pseudo/pseudo_1.2.bb
deleted file mode 100644 (file)
index 9f95bd7..0000000
+++ /dev/null
@@ -1,12 +0,0 @@
-require pseudo.inc
-
-PR = "r6"
-
-SRC_URI = "http://www.yoctoproject.org/downloads/${BPN}/${BPN}-${PV}.tar.bz2 \
-           file://oe-config.patch \
-           file://static_sqlite.patch \
-           file://opendir.patch \
-           file://renameat.patch"
-
-SRC_URI[md5sum] = "a2819084bab7e991f06626d02cf55048"
-SRC_URI[sha256sum] = "4749a22df687f44d24c26e97170d4781a1bd52d5ee092364a40877e4d96ff058"
diff --git a/meta/recipes-devtools/pseudo/pseudo_1.3.bb b/meta/recipes-devtools/pseudo/pseudo_1.3.bb
new file mode 100644 (file)
index 0000000..e7a329c
--- /dev/null
@@ -0,0 +1,8 @@
+require pseudo.inc
+
+PR = "r7"
+
+SRC_URI = "http://www.yoctoproject.org/downloads/${BPN}/${BPN}-${PV}.tar.bz2"
+
+SRC_URI[md5sum] = "5832bb70e6dce1a17b9b33a9c5c4b923"
+SRC_URI[sha256sum] = "2f30e2e9ec966cec3a321fc2ed80408ee77ad6c0cd73aaf36f88b98e008f8508"
index 4f8c9d619a1b55e21b37608e0053954d3262805a..9414c79b78c4847efb047ab830de7bdd19340583 100644 (file)
@@ -1,14 +1,12 @@
 require pseudo.inc
 
-SRCREV = "17c2233f93692f79684792750001ee6d13e03925"
-PV = "1.2+git${SRCPV}"
-PR = "r21"
+SRCREV = "f0375c9aaefbccfd41aebbf6d332bb4d9e8f980c"
+PV = "1.3+git${SRCPV}"
+PR = "r22"
 
 DEFAULT_PREFERENCE = "-1"
 
-SRC_URI = "git://github.com/wrpseudo/pseudo.git;protocol=git \
-           file://oe-config.patch \
-           file://static_sqlite.patch"
+SRC_URI = "git://github.com/wrpseudo/pseudo.git;protocol=git"
 
 S = "${WORKDIR}/git"