problem with: vendorcode/amd/agesa/fam15: Build as a static library

[Scott Duplichan]

While trying to get abuild working from Windows I found a coreboot
makefile problem. File src/vendorcode/amd/agesa/f15/Makefile.inc is
be processed twice. While this doesn't break a Linux build, it does
break the Windows build. The Windows problem is because the ar
command line contains each object file twice, and the Windows 32KB
command line length is exceeded.

As a temporary work around, I bracket the contents if Makefile.inc with:

ifeq ($(F15_MAKE_INCLUDED),)
    export F15_MAKE_INCLUDED=y
        ...
    endif

What is the correct solution?

Thanks,
Scott

[Patrick Georgi]

Am 2014-12-10 07:27, schrieb Scott Duplichan:
> While trying to get abuild working from Windows I found a coreboot
> makefile problem. File src/vendorcode/amd/agesa/f15/Makefile.inc is
> be processed twice.
> What is the correct solution?

Removing the subdirs-y += ../../../../$(AGESA_ROOT) line in the 
mainboard's Makefile.inc

Patrick

problem with: vendorcode/amd/agesa/fam15: Build as a static library

[Marc Jones]

On Thu Dec 11 2014 at 11:40:58 AM Patrick Georgi 
wrote:
> Am 2014-12-10 07:27, schrieb Scott Duplichan:
> > While trying to get abuild working from Windows I found a coreboot
> > makefile problem. File src/vendorcode/amd/agesa/f15/Makefile.inc is
> > be processed twice.
> > What is the correct solution?
>
> Removing the subdirs-y += ../../../../$(AGESA_ROOT) line in the
> mainboard's Makefile.inc
>
> Patrick

problem with: vendorcode/amd/agesa/fam15: Build as a static library

[Scott Duplichan]

Patrick Georgi [mailto:[e-mail]] wrote:

]Sent: Thursday, December 11, 2014 12:40 PM
]To: [e-mail]
]Subject: Re: [coreboot] problem with: vendorcode/amd/agesa/fam15: Build as a static library
]
]Am 2014-12-10 07:27, schrieb Scott Duplichan:
]> While trying to get abuild working from Windows I found a coreboot
]> makefile problem. File src/vendorcode/amd/agesa/f15/Makefile.inc is
]> be processed twice.
]
]> What is the correct solution?
]Removing the subdirs-y += ../../../../$(AGESA_ROOT) line in the 
]mainboard's Makefile.inc

Thanks Patrick. That is perfect. I will try to submit patch soon with
this change and others needed to get abuild working on Windows.

Thanks,
Scott

]
]Patrick