RFR: 8016303: make CONF= isn't working

[Boaz Nahum]

Do you expecte me to test it?I will be glad to, just that I never before
built a patch, only from main branches.
On Jun 27, 2013 12:14 PM, "Erik Joelsson" <erik.joelsson at oracle.com> wrote:

> Would be great if someone could take a look at this, thanks.
>
> /Erik
>
> On 2013-06-11 12:18, Erik Joelsson wrote:
>
>> I managed to fix this with a rather simple patch. The whole thing is a
>> bit of a mess, but this at least restores the advertised functionality.
>>
>> http://cr.openjdk.java.net/~**erikj/8016303/webrev.root.01/<http://cr.openjdk.java.net/~erikj/8016303/webrev.root.01/>
>>
>> /Erik
>>
>> On 2013-06-11 09:57, Erik Joelsson wrote:
>>
>>> There seems to be a bug with that logic. The correct way is supposed to
>>> be:
>>>
>>> make CONF=
>>>
>>> I'm filing a bug for it: 8016303
>>>
>>> /Erik
>>>
>>> On 2013-06-10 14:08, Boaz Nahum wrote:
>>>
>>>> Hi.
>>>>
>>>> I'm successfully building two configurations on windows (lambda/lambda)
>>>>
>>>> make CONF=windows-x86-normal-**server-release images
>>>> make CONF=windows-x86_64-normal-**server-release images
>>>>
>>>> But when I try to build them both:
>>>>
>>>> $ make CONF= images
>>>> Building target 'images' in the following configurations (matching
>>>> CONF=):
>>>> * windows-x86-normal-server-**release
>>>> * windows-x86_64-normal-server-**release
>>>> make: *** common/makefiles: Is a directory.  Stop.
>>>> make: *** No rule to make target `images'.  Stop.
>>>>
>>>> The help says:
>>>>
>>>> Useful make variables
>>>> .  make CONF=             # Build all configurations (note, assignment
>>>> is
>>>> empty)
>>>>
>>>> I also tried (found in README-builds.html):
>>>>    make all-conf  images
>>>>
>>>> Is there other way to build all configurations ?
>>>>
>>>> Thanks
>>>> Boaz
>>>>
>>>