Re: [PATCH] package.bbclass: handle links in sorted order

["Randle, William C" <william.c.randle@intel.com>]

On Wed, 2016-04-13 at 11:30 -0600, Dan McGregor wrote:

On 12 April 2016 at 10:51, Denys Dmytriyenko <denis@denix.org<mailto:denis@denix.org>> wrote:


On Tue, Apr 12, 2016 at 08:22:21AM -0700, Bill Randle wrote:


When processing links, the directories are processed in unsorted order
which can result in cases like /var/lock -> /run/lock handled before
/var/run -> /run throwing an error for /var/run because /run already exists.
Change the link processing to ensure links are processed in sorted order of
the destination.



Seems to resolve the issue for me.





For me it introduces a new issue. I use two perm files, the OE default
one, and one that changes two links to real directories. The new
packaging code complains that a link target exists as a directory,
even though a subsequent rule sets it up as a directory.

I think during parsing it should remove links as it finds directories
with the same path, and remove directories with the same path in the
order it sees them, that way when it does the in order processing it
uses the last definition in the permissions table.



Dan, can you provide an example permissions file that illustrates this? Preferably via the Bugzilla entry, or if not convenient, then here?







[YOCTO #9430]

Signed-off-by: Bill Randle <william.c.randle@intel.com<mailto:william.c.randle@intel.com>>



Reported-by: Denys Dmytriyenko <denys@ti.com<mailto:denys@ti.com>>
Tested-by: Denys Dmytriyenko <denys@ti.com<mailto:denys@ti.com>>