All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Robert P. J. Day" <rpjday@crashcourse.ca>
To: Joe MacDonald <Joe_MacDonald@mentor.com>
Cc: Yocto discussion list <yocto@yoctoproject.org>
Subject: Re: kernel manual: confusing coverage of FILESEXTRAPATHS_prepend
Date: Thu, 26 Feb 2015 04:12:33 -0500 (EST)	[thread overview]
Message-ID: <alpine.LFD.2.11.1502260402300.27160@localhost> (raw)
In-Reply-To: <20150225134406.GH8949@mentor.com>

On Wed, 25 Feb 2015, Joe MacDonald wrote:

> [[yocto] kernel manual: confusing coverage of FILESEXTRAPATHS_prepend] On 15.02.25 (Wed 03:54) Robert P. J. Day wrote:
>
> >
> >   minor quibble about kernel dev manual -- section 2.2.1, "creating
> > the append file", uses the example of:
> >
> >  FILESEXTRAPATHS_prepend := "${THISDIR}/${PN}:"
> >
> > while section 2.2.3 uses:
> >
> >  FILESEXTRAPATHS_prepend := "${THISDIR}/files:"
> >
> > both sections kind of implying that that's the way to do it, without
> > making it clear that *either* way works as long as the variable
> > prepend matches up with the directory name.
> >
> >   both ways are correct, of course, but the wording is a bit
> > confusing.
>
> It's probably worth changing the latter reference to match the
> former. Both work but with any new recipes (at least in the layers I
> maintain) the preference is to use the former for clarity as well as
> faster lookups.

  sort of related to this, but in a *regular* recipe (not a bbappend),
the default FILESPATH is set in base.bbclass:

FILESPATH = "${@base_set_filespath(["${FILE_DIRNAME}/${BP}",
"${FILE_DIRNAME}/${BPN}", "${FILE_DIRNAME}/files"], d)}"

so that, by default, a regular recipe will look for SRC_URI entries in
*all* of:

1) ${BP}/
2) ${BPN}/
3) "files/"

it's not clear which is the preferred standard (not sure there even
*is* a preferred standard), but in cases where more than one of the
above exists, all of the relevant directories will be searched, but
it's not clear why some recipes insist on breaking up the files over
more than one directory.

  in the case of subversion, i can see the logic:

subversion/
subversion_1.6.15.bb
subversion-1.8.11/
subversion_1.8.11.bb

so that the generic "subversion/" directory will apply to *all*
subversion recipes, but there is also the version-specific
"subversion-1.8.11/", so that's fine.

  busybox, though:

busybox/
busybox_1.23.1.bb
busybox_git.bb
busybox.inc
files/

won't both directories busybox/ and files/ always be consulted for
SRC_URI entries, regardless of the version of busybox? so what is the
rationale for breaking those files over two directories?

  and i'm curious ... is there any recipe that contains all *three*
types of SRC_URI subdirectories?

rday

-- 

========================================================================
Robert P. J. Day                                 Ottawa, Ontario, CANADA
                        http://crashcourse.ca

Twitter:                                       http://twitter.com/rpjday
LinkedIn:                               http://ca.linkedin.com/in/rpjday
========================================================================


  parent reply	other threads:[~2015-02-26  9:12 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-02-25  8:54 kernel manual: confusing coverage of FILESEXTRAPATHS_prepend Robert P. J. Day
2015-02-25 13:44 ` Joe MacDonald
2015-02-25 13:51   ` Gary Thomas
2015-02-25 15:49   ` Rifenbark, Scott M
2015-02-26  9:12   ` Robert P. J. Day [this message]
2015-02-26  9:20     ` Robert P. J. Day
2015-02-26 10:17     ` Paul Eggleton
2015-02-26 20:29       ` Robert P. J. Day
2015-02-25 17:08 ` 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=alpine.LFD.2.11.1502260402300.27160@localhost \
    --to=rpjday@crashcourse.ca \
    --cc=Joe_MacDonald@mentor.com \
    --cc=yocto@yoctoproject.org \
    /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.