On Aug 22, 2013, at 2:07 AM, Paul Eggleton <paul.eggle...@linux.intel.com> 
wrote:

> On Wednesday 21 August 2013 22:58:55 Saul Wold wrote:
>> Paul, Ross:
>> 
>> Please review this set.
>> 
>> It has a number of large changes that I want more EYES on!
>> 
>> Particularly, the python3 patch set, which I may remove before
>> this is final due to the world build issue.
>> 
>> There is also a few patches on poky from Beth and myself which
>> need a look at and ACK
>> The following changes since commit d98f08a7ad95d0b17846276b028a6614f16b6846:
>> 
>>  genext2fs: fix memory corruption on powerpc (2013-08-20 07:11:44 -0700)
>> 
>> are available in the git repository at:
>> 
>>  git://git.openembedded.org/openembedded-core-contrib sgw/stage
>> 
>> http://cgit.openembedded.org/cgit.cgi/openembedded-core-contrib/log/?h=sgw/
>> stage
>> 
>> Cristian Iorga (4):
>>  packagegroup-self-hosted: pcmanfm integration
>>  builder: register directories with pcmanfm
>>  build-appliance-image: changing the icon theme to sato
>>  default-providers: Set the preferred provider for bluez
> 
> I thought we were going to fix the libasound-module-bluez multiple provider 
> issue by renaming the package?
> 
>> Jonathan Liu (3):
>>  xf86-video-intel: remove duplicate xf86driproto from DEPENDS
>>  xf86-video-vesa: remove duplicate xf86driproto from DEPENDS
>>  mesa: enable additional drivers for gallium-llvm x86/x86-64
>> 
>> Kai Kang (3):
>>  bc: fix segmentation fault
>>  libx11: upgrade to 1.6.1
>>  libx11-diet: upgrade to 1.6.1
> 
> "Deleted keysymdef_include patch" - why? Presumably because it was merged 
> upstream, but that needs to be stated.
> 
>> Khem Raj (27):
>>  generate-manifest-3.3.py: Add script to generate python 3.3 manifests
>>  python-3.3-manifest: Add python3 manifest file
>>  python3native.bbclass: Add python3 abstraction class
>>  package_rpm.bbclass:Make the regexp less greedy
>>  classes/distutils: Introduce PYTHON_PN
>>  classes: Add distutils for python3
> 
> I hadn't noticed earlier, but this set of changes introduces a shedload more 
> bbclasses. Are these really all necessary?

If you look at it the classes are to let python3 live along and share the 
common code with python2

> 
>>  setuptools.bbclass: Use python-distribute instead of python-setuptools
>>  distutils3.bbclass: Port the distutils class fix to handle filenames
>>    with spaces
>>  setuptools3.bbclass: Remove useless multiline comment
> 
> These incremental changes to files added in earlier commits need to be 
> squashed 
> into those commits.

I have kept them for history, except the last one other two are worthwhile to 
have separate
commits

> 
>>  distutils: Introduce PYTHON_ABI variable
>>  distutils3: Do build_ext as separate step during compile
>>  python3: Add recipes
>>  python3: Add native recipe
>>  python_2.7.3.bb: Inherit python-dir
>>  python3-nose: Testing tools for python
>>  zeromq: Add recipe
>>  pyzmq: Add recipes using python3
> 
> There's inadequate justification given in the commit messages for why these 
> last two need to be added; I already mentioned this in earlier feedback on 
> these changes. What's more, the second commit makes a tweak to the commit 
> before which should be squashed into there instead.

*mq and *nose are needed for ipython, which is a powerful tool commonly used but
on a second thought actually I will drop ipython
from OE-Core and put it in some other layer like meta-oe and if there are enough
users who then want it, we can make a case of moving it into OE-Core

> 
>>  ipython: Add recipes for ipython2 and ipython3
> 
> Again, the commit message needs to mention why we need these.
> 


see above.


>>  python-distribute: Add recipes for python2 and python3
>>  python3: Upgrade to 3.3.2
>>  distutils3.bbclass: Fix typo and use proper values for target sysroot
>>  python3, python3-native: Consider OE staging installation
>>  python3: Fix host include contamination issue
>>  distutils3.bbclass: Use MACHINE for sysroot when not building for
>>    build host
>>  python3: Fix the compiler invocation and linker flags when cross
>>    compiling
>>  python: Add Patch description and status information
>>  eglibc: Update SRC_URI and fix unpackaged empty dir
> 
> I appreciate the work and testing that has gone into this set of changes but 
> it still needs further cleaning up as discussed above IMO.
> 
>> Lauren Post (2):
>>  directfb: Upgrade to 1.6.3
>>  directfb: add fusion patch for hangs on exit with
> 
> The commit messages seem a little bit malformed on these two.
> 
>> Mark Hatle (11):
>>  image.bbclass: Add basic support for PACKAGE_EXCLUDE
>>  python-smartpm: Add support for excluding package from the install
>>  package_rpm.bbclass:  Add support for PACKAGE_EXCLUDE to RPM installs
>>  python-smartpm: Add support to disable installing recommends
>>  package_rpm.bbclass: NO_RECOMMENDATIONS support
>>  package_deb.bbclass: Use the WORKDIR not SYSROOT for temp files
>>  package_deb: Add support for NO_RECOMMENDATIONS and PACKAGE_EXCLUDE
>>  opkg: Add --no-install-recommends option.
>>  package_ipk: Add support for NO_RECOMMENDATIONS
>>  opkg: Add support for excluding packages from the install
>>  package_ipk: Add support for PACKAGE_EXCLUDE
> 
> As mentioned earlier I'm still unhappy with not persisting these into the 
> rootfs if package-management is enabled, but I'll try to address that myself 
> in a follow-up.
> 
>> Marko Lindqvist (3):
>>  texinfo: correct dont-depend-on-help2man.patch
>>  SDL2: add recipe
>>  boost: fix build when PARALLEL_MAKE is not set
>> 
>> Otavio Salvador (1):
>>  distro_features_check.bbclass: Allow checking of required/conflicting
>>    features
> 
> I should have asked this earlier - why does this need to be a separate 
> bbclass? The code doesn't do anything if the variables aren't set, so it 
> can't 
> be because it could have an impact on existing setups that don't set them.
> 
>> Paul Eggleton (1):
>>  gst-plugins-good: fix orc enabling via PACKAGECONFIG
>> 
>> Roy.Li (1):
>>  shadow: backport a patch to make newgrp work
> 
> The commit message should really have some detail in it for a change like 
> this.
> 
>> Saul Wold (4):
>>  Add Upstream-Status tags to patches
>>  libsdl: Backport xData32 patch for x11 update
>>  valgrind: Backport patch for eglibc 2.18
>>  gnupg: Update to 2.0.21
>> 
>> Tyler Hall (1):
>>  cpan-base: Don't use immediate expansion
> 
> I assume there's no appreciable impact on build performance with this change?
> 
>> Yevhen Kyriukha (1):
>>  connman: fix systemd support for connman-* packages.
> 
> Cheers,
> Paul
> 
> -- 
> 
> Paul Eggleton
> Intel Open Source Technology Centre

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

Reply via email to