From: Scott Garman <scott.a.garman@intel.com>
To: yocto@yoctoproject.org
Subject: Re: Master build error caused a small typo by commit today : fetch2: unpack revision
Date: Sat, 12 Feb 2011 17:02:40 -0800 [thread overview]
Message-ID: <4D572DB0.7050805@intel.com> (raw)
In-Reply-To: <4D570BE2.4020900@windriver.com>
On 02/12/2011 02:38 PM, Mark Hatle wrote:
> On 2/12/11 1:17 PM, Gary Thomas wrote:
>> On 02/12/2011 12:08 PM, Koen Kooi wrote:
>>>
>>> Op 12 feb 2011, om 19:09 heeft Mark Hatle het volgende geschreven:
>>>
>>>> On 2/12/11 3:30 AM, Cui, Dexuan wrote:
>>>>> Hi Mark,
>>>>> As Liping said, the following change breaks building xorg-minimal-fonts as destdir is not correct.
>>>>> Can you please explain the actual different between cp and tar here? I think "cp -p" is similar to "tar -x -p" here?
>>>>>
>>>>
>>>> We can revert the change. But the reason for it is tar preserves hard links,
>>>> while cp does not.
>>>
>>> The problem is that people can hit things like this:
>>>
>>> | NOTE: Unpacking /OE/tentacle/sources/angstrom-layers/BSP/TI/recipes-bsp/linux/linux-omap-psp-2.6.32/beagleboard/defconfig to /OE/tentacle/build/tmp-angstrom_2010_x/work/beagleboard-angstrom-linux-gnueabi/linux-omap-psp-2.6.32-r99+gitr5fc29e7b2a76a64a739f857858ef0b98294aa155/
>>> | ERROR: Error executing a python function in /OE/tentacle/sources/angstrom-layers/BSP/TI/recipes-bsp/linux/linux-omap-psp_2.6.32.bb:
>>> | TypeError: %d format: a number is required, not str
>>> |
>>> | ERROR: The stack trace of python calls that resulted in this exception/failure was:
>>> | ERROR: File "base_do_unpack", line 19, in<module>
>>> | ERROR:
>>> | ERROR: File "base_do_unpack", line 14, in base_do_unpack
>>> | ERROR:
>>> | ERROR: File "/OE/tentacle/sources/bitbake/lib/bb/fetch2/__init__.py", line 1015, in unpack
>>> | ERROR: ud.method.unpack(ud, root, self.d)
>>> | ERROR:
>>> | ERROR: File "/OE/tentacle/sources/bitbake/lib/bb/fetch2/__init__.py", line 724, in unpack
>>> | ERROR: cmd = 'tar -cf - -C "%d" -ps . | tar -xf - -C "%s/%s/"' % (file, rootdir, destdir)
>>
>> Does not changing this "%d" to "%s" work?
>
> Yes, this is the correct fix. Nobody has been around today to push the fix into
> the tree however.
>
> It was a simple typo.
I managed to catch Richard on IRC about an hour ago and he pushed a fix
to master.
Scott
--
Scott Garman
Embedded Linux Engineer - Yocto Project
Intel Open Source Technology Center
next prev parent reply other threads:[~2011-02-13 1:02 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-02-12 5:05 Master build error caused a small typo by commit today : fetch2: unpack revision Ke, Liping
2011-02-12 7:53 ` Ke, Liping
2011-02-12 9:30 ` Cui, Dexuan
2011-02-12 18:09 ` Mark Hatle
2011-02-12 19:08 ` Koen Kooi
2011-02-12 19:17 ` Gary Thomas
2011-02-12 22:38 ` Mark Hatle
2011-02-13 1:02 ` Scott Garman [this message]
2011-02-13 7:12 ` Ke, Liping
2011-02-13 19:27 ` Mark Hatle
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=4D572DB0.7050805@intel.com \
--to=scott.a.garman@intel.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.