[coreboot] buildrom bug
Jordan Crouse
jordan.crouse at amd.com
Fri Apr 25 22:09:31 CEST 2008
On 25/04/08 16:06 -0400, Marc Karasek wrote:
> I am assuming a update / co from svn will not build until this patch is in
> place? =-O
>
> It doesn't for me.. :(
Um - no. As far as I know, it should work. Myles?
>
> *********************
> Marc Karasek
> MTS
> Sun Microsystems
> mailto:marc.karasek at sun.com
> ph:770.360.6415
> *********************
>
>
>
> Jordan Crouse wrote:
>> On 25/04/08 12:15 -0600, Myles Watson wrote:
>>
>>>
>>>> -----Original Message-----
>>>> From: Jordan Crouse [mailto:jordan.crouse at amd.com]
>>>> Sent: Friday, April 25, 2008 12:12 PM
>>>> To: Myles Watson
>>>> Cc: coreboot at coreboot.org
>>>> Subject: Re: buildrom bug
>>>>
>>>> On 25/04/08 12:01 -0600, Myles Watson wrote:
>>>>
>>>>>> On 25/04/08 11:45 -0600, Myles Watson wrote:
>>>>>>
>>>>>>>> Here's the updated patch. It needs to be preceded by
>>>>>>>> svn mv packages/utils packages/nrv2b
>>>>>>>>
>>>>>>> Sorry. Here's the new patch.
>>>>>>>
>>>>>> So here is the million dollar question. If all the .mk
>>>>>> files are packages/<package>/<package>.mk, is there anything
>>>>>> stopping us from constructing PAYLOAD_AND_DEP_MK like so:
>>>>>>
>>>>>> PAYLOAD_AND_DEP_MK = $(patsubst %,$(PACKAGE_DIR)/packages/%/%.mk,
>>>>>> $(DEPENDS-y) $(PAYLOAD-y) $(HOSTTOOLS-y))
>>>>>>
>>>>> Doesn't that put you back where you were with the "You need ruby to
>>>>>
>>>> compile
>>>>
>>>>> grub2"? The point of the PAYLOAD_AND_DEP_MK variable was supposed to
>>>>> be
>>>>> that it got set (purposefully) in the config/payloads/<payload>.conf
>>>>>
>>>> file.
>>>>
>>>>> I think it's reasonable to ask a developer to set the variable when he
>>>>>
>>>> adds
>>>>
>>>>> the .conf file for a new package.
>>>>>
>>>> DEPENDS-y PAYLOAD-y and HOSTTOOLS-y are the dependencies
>>>> for what you are actually building.
>>>>
>>> Thanks for the clarification. I looked at you code and saw the original
>>> wildcard. Sorry.
>>>
>>>
>>>> Our original problem was that we were doing a $(wildcard */*/.mk), which
>>>> was including the kitchen sink. So unless your package is somehow
>>>> listing
>>>> grub2 as one of its dependencies, then packages/grub2/grub2.mk won't
>>>> make
>>>> it into the list, and you won't have a problem.
>>>>
>>>> I think it is pretty unreasonable to ask the developer to add the
>>>> variable - it includes magic that he shouldn't have to think about,
>>>> namely how packages are referenced and included.
>>>>
>>>> And lets look at it this way - 99.99% of the time, its going to be
>>>> packages/<package>/<package>.mk - we're just automating the process.
>>>> If we want, we can make PAYLOAD_AND_DEP_MK ?= so that the developer
>>>> can override it if they want, but I think most people won't want to.
>>>>
>>> I agree. Do you want to create the patch or send me a patch against
>>> mine?
>>>
>>
>> I'll make a patch based on yours.
>>
>> Jordan
>>
>>
>
--
Jordan Crouse
Systems Software Development Engineer
Advanced Micro Devices, Inc.
More information about the coreboot
mailing list