From: Randy MacLeod <randy.macleod@windriver.com>
To: Otavio Salvador <otavio@ossystems.com.br>
Cc: Patches and discussions about the oe-core layer
<openembedded-core@lists.openembedded.org>
Subject: Re: [PATCH 01/20] udev-cache: Compress the cache
Date: Tue, 5 Aug 2014 16:35:31 -0400 [thread overview]
Message-ID: <53E14013.1000301@windriver.com> (raw)
In-Reply-To: <CAP9ODKqt79-1XkYDi01Wn-ZT5f13e4s=BhF08=zj8B==Hcjkpg@mail.gmail.com>
On 14-08-05 04:09 PM, Otavio Salvador wrote:
> On Tue, Aug 5, 2014 at 4:00 PM, Randy MacLeod
> <randy.macleod@windriver.com> wrote:
>> On 14-08-04 02:40 PM, Ben Shelton wrote:
>>>
>>> From: Richard Tollerton <rich.tollerton@ni.com>
>>>
>>> $DEVCACHE is observed to be 100k uncompressed; compressing it reduces
>>> its size to ~5k.
>>>
>>> Natinst-Rally-ID: TA44427
>>> Acked-by: Gratian Crisan <gratian.crisan@ni.com>
>>> Natinst-ReviewBoard-ID: 58620
>>
>>
>>
>> This is a good series of patches; thanks for sending them.
>>
>> It would be nice if you could remove your company's ID tags
>> from the commit log. For example, you create and use could use
>> a YP bugzilla ID ( https://bugzilla.yoctoproject.org/ ) and
>> then match that up in your internal process.
>
> I don't think we should ask for this. I saw several of us to include
> Id tags in past commits (I also did) and I think it is fine. We
> shouldn't ask an extra step for people to be able to track it (open a
> YP bugzilla ID issue for example).
>
Wind River patches used to include a "CQID" tag but we've changed
our process to avoid needing such internal tags. If National
Instruments can do so as well, that'd be best.
I did check my oe-core email list history and this seems like the
first patch from NI that has the tags included so I thought
I'd reply and see if we can get the tags dropped.
Ben, what's the story?
--
# Randy MacLeod. SMTS, Linux, Wind River
Direct: 613.963.1350
next prev parent reply other threads:[~2014-08-05 20:35 UTC|newest]
Thread overview: 55+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-08-04 18:40 [PATCH 00/20] udev-cache related changes Ben Shelton
2014-08-04 18:40 ` [PATCH 01/20] udev-cache: Compress the cache Ben Shelton
2014-08-04 19:29 ` Otavio Salvador
2014-08-04 22:44 ` Khem Raj
2014-08-04 23:45 ` Ben Shelton
2014-08-05 0:06 ` Khem Raj
2014-08-05 19:00 ` Randy MacLeod
2014-08-05 20:09 ` Otavio Salvador
2014-08-05 20:35 ` Randy MacLeod [this message]
2014-08-06 2:08 ` Ben Shelton
2014-08-22 15:24 ` Patch message guidelines and internal/corporate fields Richard Tollerton
2014-08-23 8:40 ` Richard Purdie
2014-08-24 15:33 ` Paul Eggleton
2014-08-25 1:19 ` Khem Raj
2014-08-04 18:40 ` [PATCH 02/20] udev-cache: choose a more descriptive cache filename Ben Shelton
2014-08-04 19:30 ` Otavio Salvador
2014-08-04 18:40 ` [PATCH 03/20] udev-cache: Honor VERBOSE for bootup message Ben Shelton
2014-08-04 19:30 ` Otavio Salvador
2014-08-04 18:40 ` [PATCH 04/20] udev-cache: Don't ignore errors from cache extract Ben Shelton
2014-08-04 19:32 ` Otavio Salvador
2014-08-04 18:40 ` [PATCH 05/20] busybox: tar: enable CONFIG_FEATURE_TAR_LONG_OPTIONS Ben Shelton
2014-08-04 19:33 ` Otavio Salvador
2014-08-04 18:40 ` [PATCH 06/20] udev-cache: Remove superfluous subshell on extract Ben Shelton
2014-08-04 19:33 ` Otavio Salvador
2014-08-04 18:40 ` [PATCH 07/20] udev-cache: Update cache tarball atomically Ben Shelton
2014-08-04 19:35 ` Otavio Salvador
2014-08-04 18:41 ` [PATCH 08/20] udev-cache: Create cache asynchronously Ben Shelton
2014-08-04 19:41 ` Otavio Salvador
2014-08-04 21:18 ` Richard Tollerton
2014-08-04 21:22 ` Otavio Salvador
2014-08-04 21:37 ` Richard Tollerton
2014-08-04 21:41 ` Otavio Salvador
2014-08-04 22:07 ` Richard Tollerton
2014-08-04 18:41 ` [PATCH 09/20] udev-cache.default: documentation update Ben Shelton
2014-08-04 19:42 ` Otavio Salvador
2014-08-04 18:41 ` [PATCH 10/20] udev-cache: parametrize sysconf file paths Ben Shelton
2014-08-04 19:44 ` Otavio Salvador
2014-08-04 21:38 ` Ben Shelton
2014-08-04 21:42 ` Otavio Salvador
2014-08-04 21:48 ` Ben Shelton
2014-08-04 18:41 ` [PATCH 11/20] udev-cache: parametrize tar options Ben Shelton
2014-08-04 19:44 ` Otavio Salvador
2014-08-04 21:40 ` Ben Shelton
2014-08-04 18:41 ` [PATCH 12/20] udev-cache: fix timestamp errors on systems lacking an RTC Ben Shelton
2014-08-04 18:41 ` [PATCH 13/20] udev-cache: Avoid caching udev.cache or non-devfs filesystems Ben Shelton
2014-08-04 18:41 ` [PATCH 14/20] udev-cache: refactor; improve verbosity and error handling Ben Shelton
2014-08-04 18:41 ` [PATCH 15/20] udev-cache: don't attempt to extract cache if it doesn't exist Ben Shelton
2014-08-04 18:41 ` [PATCH 16/20] udev-cache: invalidate on rules.d changes Ben Shelton
2014-08-04 18:41 ` [PATCH 17/20] udev-cache: fix udev-cache changes to work with busybox tar Ben Shelton
2014-08-04 18:41 ` [PATCH 18/20] udev: Make build MACHINE-specific Ben Shelton
2014-08-04 19:43 ` Martin Jansa
2014-08-04 18:41 ` [PATCH 19/20] udev: Disable keymap support on machines lacking keyboard support Ben Shelton
2014-08-04 19:41 ` Martin Jansa
2014-08-04 18:41 ` [PATCH 20/20] udev: don't halt if devtmpfs can't be mounted Ben Shelton
2014-08-04 19:40 ` Martin Jansa
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=53E14013.1000301@windriver.com \
--to=randy.macleod@windriver.com \
--cc=openembedded-core@lists.openembedded.org \
--cc=otavio@ossystems.com.br \
/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.