On 07/04/2012 03:16 AM, Robert Yang wrote:
when the tmpdir dir is longer than 220, there is no files saved in
tmp/sysroots/x86_64-linux/var/lib/apt/lists/ after run apt-get update,
this is because apt-get uses the path as the file name, but the file
name can't be longer than 255 according to /usr/include/linux/limits.h.

[YOCTO #2688]

Signed-off-by: Robert Yang <liezhi.y...@windriver.com>
---
  .../apt/apt-0.7.14/truncate-filename.patch         |   35 ++++++++++++++++++++
  meta/recipes-devtools/apt/apt-native_0.7.14.bb     |    2 +-
  meta/recipes-devtools/apt/apt.inc                  |    1 +
  meta/recipes-devtools/apt/apt_0.7.14.bb            |    2 +-
  4 files changed, 38 insertions(+), 2 deletions(-)
  create mode 100644 
meta/recipes-devtools/apt/apt-0.7.14/truncate-filename.patch

diff --git a/meta/recipes-devtools/apt/apt-0.7.14/truncate-filename.patch 
b/meta/recipes-devtools/apt/apt-0.7.14/truncate-filename.patch
new file mode 100644
index 0000000..db1c42b
--- /dev/null
+++ b/meta/recipes-devtools/apt/apt-0.7.14/truncate-filename.patch
@@ -0,0 +1,35 @@
+strutl.cc: the filename can't be longer than 255
+
+The URItoFileName translates the path into the filename, but the
+filename can't be longer than 255 according to
+/usr/include/linux/limits.h.
+
+Truncate it when it is longer than 240 (leave some spaces for
+".Packages" and "._Release" suffix)
+
+Upstream-Status: Pending
+Signed-off-by: Robert Yang <liezhi.y...@windriver.com>
+---
+ apt-pkg/contrib/strutl.cc |    7 ++++++-
+ 1 file changed, 6 insertions(+), 1 deletion(-)
+
+diff --git a/apt-pkg/contrib/strutl.cc b/apt-pkg/contrib/strutl.cc
+--- a/apt-pkg/contrib/strutl.cc
++++ b/apt-pkg/contrib/strutl.cc
+@@ -399,7 +399,12 @@ string URItoFileName(const string &URI)
+    // "\x00-\x20{}|\\\\^\\[\\]<>\"\x7F-\xFF";
+    string NewURI = QuoteString(U,"\\|{}[]<>\"^~_=!@#$%^&*");
+    replace(NewURI.begin(),NewURI.end(),'/','_');
+-   return NewURI;
++
++   // Truncate from the head when it is longer than 240
++   if(NewURI.length() > 240)
++       return NewURI.substr(NewURI.length() - 240, NewURI.length() - 1);
++   else
++       return NewURI;

Robert, I am not super familiar with this part of the DPKG code, what uses the URItoFileName() and what effect does the truncation have?

Sau!

+ }
+                                                                       /*}}}*/
+ // Base64Encode - Base64 Encoding routine for short strings           /*{{{*/
+--
+1.7.10.4
+
diff --git a/meta/recipes-devtools/apt/apt-native_0.7.14.bb 
b/meta/recipes-devtools/apt/apt-native_0.7.14.bb
index ca5476b..38e1fe6 100644
--- a/meta/recipes-devtools/apt/apt-native_0.7.14.bb
+++ b/meta/recipes-devtools/apt/apt-native_0.7.14.bb
@@ -1,6 +1,6 @@
  require apt-native.inc

-PR = "r8"
+PR = "r9"

  SRC_URI += "file://nodoc.patch \
              file://noconfigure.patch \
diff --git a/meta/recipes-devtools/apt/apt.inc 
b/meta/recipes-devtools/apt/apt.inc
index 1f6343a..563eda1 100644
--- a/meta/recipes-devtools/apt/apt.inc
+++ b/meta/recipes-devtools/apt/apt.inc
@@ -8,6 +8,7 @@ SRC_URI = "${DEBIAN_MIRROR}/main/a/apt/apt_${PV}.tar.gz \
             file://localefixes.patch \
             file://makerace.patch \
             file://remove-redeclaration.patch \
+           file://truncate-filename.patch \
             "

  inherit autotools gettext
diff --git a/meta/recipes-devtools/apt/apt_0.7.14.bb 
b/meta/recipes-devtools/apt/apt_0.7.14.bb
index a627728..12dd1f2 100644
--- a/meta/recipes-devtools/apt/apt_0.7.14.bb
+++ b/meta/recipes-devtools/apt/apt_0.7.14.bb
@@ -3,7 +3,7 @@ RDEPENDS_${PN} = "dpkg"
  LIC_FILES_CHKSUM = "file://COPYING.GPL;md5=0636e73ff0215e8d672dc4c32c317bb3"
  require apt.inc

-PR = "r13"
+PR = "r14"

  SRC_URI += "file://nodoc.patch \
              file://includes-fix.patch "



_______________________________________________
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core

Reply via email to