All of lore.kernel.org
 help / color / mirror / Atom feed
From: Saul Wold <sgw@linux.intel.com>
To: Paul Eggleton <paul.eggleton@linux.intel.com>
Cc: openembedded-core@lists.openembedded.org
Subject: Re: [CONSOLIDATED PULL 00/18] Final Review Pending
Date: Mon, 15 Jul 2013 07:36:30 -0700	[thread overview]
Message-ID: <51E408EE.8080709@linux.intel.com> (raw)
In-Reply-To: <1510613.ezlYfIjzn1@helios>

On 07/15/2013 02:24 AM, Paul Eggleton wrote:
> On Monday 15 July 2013 00:23:38 Saul Wold wrote:
>> Ross, Paul:
>>
>> Please do a review of this set from over the weekend.
>>
>> Hongxu's patch set includes an atom-pc patch that should have
>> been on poky, I removed it from here.
>>
>> I have reviewed the 2 package from the Auto Package Update (my
>> recipes) and they are correct.
>>
>> Thanks
>> 	Sau!
>>
>>
>> The following changes since commit 43c28375f97161e618fa54349c65be2058c33c53:
>>
>>    distrodata.bbaclass: change in git and svn package reporting (2013-07-12
>> 10:12:13 -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
>>
>> Alexandru Palalau (1):
>>    lib/oeqa/runtime: add tests for syslog and df
>>
>> Auto Package Upgrade (2):
>>    libidn: upgrade to 1.28
>>    mc: upgrade to 4.8.9
>
> The author field for these isn't set correctly.
>
Fixed.

>> Chen Qi (1):
>>    core-image-lsb.bb: add 'package-management' to IMAGE_FEATURES
>>
>> Hongxu Jia (5):
>>    pmount: add version 0.9.23
>>    udev-extraconf:fix xuser write to usb storage failed
>>    udev: fix invoking pmount failed based on sysvinit
>>    pmount: allow users in disk group to perform pmount/pumount
>>    xuser-account: add xuser to disk group
>
> I have some reservations about these changes. pmount is obsolete
> according to its author:
>
> https://lists.ubuntu.com/archives/ubuntu-devel-discuss/2011-February/012421.html
>
> I appreciate it may solve the problem we're trying to solve but we need
> to be pretty confident about pulling this in before we do so since we'll
> likely be supporting it ourselves.
>
I was unsure at first about this one, and noone had commented on that 
the last 2 times it was posted, so I took them.

>> Laurentiu Palcu (1):
>>    xinput-calibrator: move it from meta-oe to oe-core
>
> I've just noticed this drops PR (previous value was "r6"). I know it seems
> trivial but we're not increasing PV so if we hope to supersede the recipe
> from meta-oe, this will have to be preserved. Same goes for
> pointercal-xinput which had a PR value of "r4".
>
Dropped for now, I missed that PR change

>> Saul Wold (1):
>>    initscripts: Update to 9.48
>>
>> Stefan Stanacar (7):
>>    lib/oeqa/runtime/dmesg.py: whitelist dmesg message for systemd net
>>      interface name
>>    lib/oeqa/utils/sshcontrol.py: fix passing command to subprocess
>>    testimage.bbclass, lib/oeqa/utils/qemurunner.py: make boot timeout
>>      configurable and increase it
>>    lib/oeqa/runtime/multilib.py: use readelf instead of file
>>    lib/oeqa/oetest.py: provide a ps command for all tests
>>    classes/testimage.bbclass: use a copy of rootfs for tests
>>    testimage.bbclass, lib/oeqa: add headers and comments
>
Stefan has a new set that I will pull and put into the Pull request.

Sau!

> Cheers,
> Paul
>


      parent reply	other threads:[~2013-07-15 14:36 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-07-15  7:23 [CONSOLIDATED PULL 00/18] Final Review Pending Saul Wold
2013-07-15  9:24 ` Paul Eggleton
2013-07-15  9:57   ` Burton, Ross
2013-07-15 10:15     ` Paul Eggleton
2013-07-15 11:23       ` Burton, Ross
2013-07-15 11:42   ` Hongxu Jia
2013-07-15 14:36   ` Saul Wold [this message]

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=51E408EE.8080709@linux.intel.com \
    --to=sgw@linux.intel.com \
    --cc=openembedded-core@lists.openembedded.org \
    --cc=paul.eggleton@linux.intel.com \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.