Bug 5863 - Do not duplicate the same conf notes from oe-core
Summary: Do not duplicate the same conf notes from oe-core
Status: RESOLVED WONTFIX
Alias: None
Product: Meta-yocto
Classification: Build System, Metadata & Runtime
Component: meta-yocto (show other bugs)
Version: unspecified
Hardware: x86 Multiple
: Low normal
Target Milestone: 1.6 M4
Assignee: Richard Purdie
QA Contact:
URL:
Whiteboard:
Depends on:
Blocks:
 
Reported: 2014-02-21 14:43 UTC by Laszlo Papp
Modified: 2014-02-25 10:37 UTC (History)
3 users (show)

See Also:
OS type for building Yocto: ---
Type of Regression: ---
Verified:
Documentation change: Don't know


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Laszlo Papp 2014-02-21 14:43:40 UTC
./meta-yocto/conf/conf-notes.txt and ./meta/conf/conf-notes.txt are identical.

It should be possible to inherit the file from oe-core in my opinion if the desired content is the same.

This can cause sync overhead, et cetera.

If one likes to customize it, it is possible by having that file with a different content, but for the same content, it looks a bit strange a duplication.
Comment 1 Richard Purdie 2014-02-25 10:07:52 UTC
The TEMPLATECONF directory is used to contain conf-notes.txt and things like local.conf.sample. It can only be set to one directory so if you change it, you take copies of the set of files.

We could add a ton of complexity for fallbacks and so on to remove the file duplication however I think the duplicate file isn't a big issue in this case. In most cases you would want to customise *all* the files.

So basically, it doesn't make sense to change this.
Comment 2 Laszlo Papp 2014-02-25 10:11:03 UTC
> We could add a ton of complexity for fallbacks.

It would be literally a few lines in the bash script where it is set up. I am assume it could be done in less than five, eventually.

Please let me know why it would not be that simple.
Comment 3 Richard Purdie 2014-02-25 10:19:11 UTC
Its adding complexity into a location where I don't believe we want or need it. We try to keep the init scripts as simple and clean as possible. The "poky" case is one of the few cases where a copy of that file makes sense, nearly every other user would modify it.

So my answer remains as stated, I don't believe we need or want this.
Comment 4 Laszlo Papp 2014-02-25 10:32:30 UTC
In fact, this "complexity" is already in there:

http://git.yoctoproject.org/cgit/cgit.cgi/poky/tree/scripts/oe-setup-builddir#n121

So, there would be no any need for the meta-yocto layer to keep it. So, you could either remove that "complexity" (IMHO bad idea), or remove the complexity from the meta-yocto layer.
Comment 5 Laszlo Papp 2014-02-25 10:35:42 UTC
I was not probably clear, so most of the complexity is already in there, and adding one line allow to remove a whole file in meta-yocto and elsewhere where the users are happy with the default.

The required change is basically a one-liner in here:

OECORENOTESCONF="$TEMPLATECONF/conf-notes.txt"

to:

if ! (test -r ""$TEMPLATECONF/conf-notes.txt"); then
    OECORENOTESCONF="$TEMPLATECONF/conf-notes.txt"

or something like that.
Comment 6 Laszlo Papp 2014-02-25 10:37:38 UTC
Hmm, I cannot still edit comment on bugzilla, so I made a mistake about the negation, so that would not even be needed, so it would be even simpler than my previous one-liner. :-)