All of lore.kernel.org
 help / color / mirror / Atom feed
From: Richard Purdie <richard.purdie@linuxfoundation.org>
To: Saul Wold <sgw@linux.intel.com>
Cc: dvhart@linux.intel.com, tom.zanussi@linux.intel.com,
	openembedded-core@lists.openembedded.org
Subject: Re: [PATCH 2/2] perf: rename perf_3.4 to perf
Date: Tue, 12 Mar 2013 18:39:20 +0000	[thread overview]
Message-ID: <1363113560.9859.11.camel@ted> (raw)
In-Reply-To: <513E6EC3.7090905@linux.intel.com>

On Mon, 2013-03-11 at 16:54 -0700, Saul Wold wrote:
> On 03/07/2013 04:39 PM, tom.zanussi@linux.intel.com wrote:
> > From: Tom Zanussi <tom.zanussi@linux.intel.com>
> >
> > There's nothing kernel-version-specific about the perf_3.4 recipe, so
> > it's actually misnamed and misleading now that it also gets used with
> > the 3.8 kernel.
> >
> > Since the recipe isn't tied to a specific PV, and simply uses
> > whatever's in STAGING_KERNEL_DIR, there's no reason to add anything
> > else either to the bare PN, so just use that as the recipe name.
> >
> This did not quite work out correctly, since the package name has now 
> gone backwards since the default PV for a recipe is 1.0 when nothing is 
> set on the recipe file name.
> 
> Is there a way to sort this out such that it picks up an incrementing PV 
> from the kernel recipe?
> 
> As it stands now, the PV has gone from 3.4 -> 1.0.
> 
> Thanks for looking after this.

As a hint on how to do this we need to read the kernel version and then
set PKGV just prior to or at the start of do_package time.

The logs will show 1.0 as the version but the final packages will have
the kernel version.

Cheers,

Richard




  reply	other threads:[~2013-03-12 18:56 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-03-08  0:38 [PATCH 0/2] perf recipe update tom.zanussi
2013-03-08  0:39 ` [PATCH 1/2] perf: rename perf.inc to perf-features.inc tom.zanussi
2013-03-08  0:39 ` [PATCH 2/2] perf: rename perf_3.4 to perf tom.zanussi
2013-03-11 23:54   ` Saul Wold
2013-03-12 18:39     ` Richard Purdie [this message]
2013-03-12 20:24     ` Darren Hart
2013-03-08  1:06 ` [PATCH 0/2] perf recipe update Bruce Ashfield
2013-03-08 18:40   ` Darren Hart

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=1363113560.9859.11.camel@ted \
    --to=richard.purdie@linuxfoundation.org \
    --cc=dvhart@linux.intel.com \
    --cc=openembedded-core@lists.openembedded.org \
    --cc=sgw@linux.intel.com \
    --cc=tom.zanussi@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.