On Fri, 2012-03-30 at 00:21 +0200, Martin Jansa wrote: > First 5 patches are taken from oe-core. > > Then there are some cleanups and fixes for issues I've found while looking > for cause of very long package-index calls. > > Some patches uses compatible changes from 2to3 (tested with python2.7), but > python3 support is not 100% complete, but were closer. > > And in the end I've found the cause.. printing long fields (e. g. description) > with opkg.py was producing empty lines in Packages file (which are Packages > entry > separators) so only first entries till first empty line were processed from > old > Packages file and the rest was always extracted from control files in > packaged .ipk. > > And this also fixes Packages.filelist generation. > > I don't use python very much, so please review. > > And the speedup? Almost 60 times :) > before: > real 20m10.484s > user 18m32.335s > sys 0m11.885s > > after > real 0m24.492s > user 0m23.748s > sys 0m0.565s > > The following changes since commit 002d29bc605d7c2d02e4cf20a43c5277c15f5597: > > [opkg-utils] fix install fail problem Thanks for khorben's patch :-) > https://docs.openmoko.org/trac/attachment/ticket/2072/patch-opkg-utils_Makefile_install_path.diff > (2008-11-03 03:59:59 +0000) > > are available in the git repository at: > git://github.com/shr-project/opkg-utils jansa/pull > https://github.com/shr-project/opkg-utils/tree/jansa/pull > > Christopher Larson (1): > Use python via the PATH, rather than hardcoding /usr/bin/python > > Enrico Scholz (1): > opkg-make-index: convert mtime to int before comparing it > > Khem Raj (1): > opkg.py: Add knowledge about License field in ipk headers > > Martin Jansa (17): > opkg.py: use hashlib instead of old md5 module > opkg.py, arfile.py: report which file has wrong format and use the > same test > arfile: decode read lines as ascii string before getting fields from > it > 2to3: use subprocess instead of commands > 2to3: print fixes > 2to3: dictionary fixes > 2to3: exception handling fixes > opkg-make-index: show OSError/IOError > opkg.py: use string funtcions directly on string variable > opkg.py: catch TypeError when reading control file, to show which one > is failing > arfile: fix test > opkg-make-index: don't use stdout for Packages output and use __str__ > instead of __repr__ > opkg.py: cast lines from controlfile as string > opkg.py: computeFileMD5 only when we have fn, otherwise fails to read > None file > opkg.py: fix write_package when called from main test > opkg.py: use textwrap for description writing > opkg.py: improve test so it prints temporary control file with long > description and then reads it back > > Richard Purdie (1): > opkg-make-index: don't error out when some package disappears > > Scott Anderson (1): > arfile.py: handle six digit UIDs
I reviewed these and they all look like good improvements. I've merged it into master, thanks! Richard _______________________________________________ yocto mailing list yocto@yoctoproject.org https://lists.yoctoproject.org/listinfo/yocto
