One of the changes between 2015 and 2018 broke the win32 build:

http://d.puremagic.com/test-results/test_data.ghtml?dataid=3355

std\typecons.d(336): Error: cannot evaluate empty("") at compile time
std\typecons.d(371): Error: cannot evaluate injectNamedFields() at compile time
std\typecons.d(371): Error: argument to mixin must be a string, not
(injectNamedFields())

  ??? Works for me.

On 9/17/2010 8:45 PM, Brad Roberts wrote:
> One of the changes between 2015 and 2018 broke the win32 build:
>
> http://d.puremagic.com/test-results/test_data.ghtml?dataid=3355
>
> std\typecons.d(336): Error: cannot evaluate empty("") at compile time
> std\typecons.d(371): Error: cannot evaluate injectNamedFields() at compile time
> std\typecons.d(371): Error: argument to mixin must be a string, not
> (injectNamedFields())
>
> _______________________________________________
> phobos mailing list
> phobos at puremagic.com
> http://lists.puremagic.com/mailman/listinfo/phobos
>

Are you building on win32 or linux32?  It seems to be win32 specific.

On 9/17/2010 5:51 PM, David Simcha wrote:
>  ??? Works for me.
> 
> On 9/17/2010 8:45 PM, Brad Roberts wrote:
>> One of the changes between 2015 and 2018 broke the win32 build:
>>
>> http://d.puremagic.com/test-results/test_data.ghtml?dataid=3355
>>
>> std\typecons.d(336): Error: cannot evaluate empty("") at compile time
>> std\typecons.d(371): Error: cannot evaluate injectNamedFields() at compile time
>> std\typecons.d(371): Error: argument to mixin must be a string, not
>> (injectNamedFields())
>>

  Win32.  I thought that was pretty clearly implied given that you made
it clear before that the issue was Win32 specific.

On 9/17/2010 8:54 PM, Brad Roberts wrote:
> Are you building on win32 or linux32?  It seems to be win32 specific.
>
> On 9/17/2010 5:51 PM, David Simcha wrote:
>>   ??? Works for me.
>>
>> On 9/17/2010 8:45 PM, Brad Roberts wrote:
>>> One of the changes between 2015 and 2018 broke the win32 build:
>>>
>>> http://d.puremagic.com/test-results/test_data.ghtml?dataid=3355
>>>
>>> std\typecons.d(336): Error: cannot evaluate empty("") at compile time
>>> std\typecons.d(371): Error: cannot evaluate injectNamedFields() at compile time
>>> std\typecons.d(371): Error: argument to mixin must be a string, not
>>> (injectNamedFields())
>>>
>
> _______________________________________________
> phobos mailing list
> phobos at puremagic.com
> http://lists.puremagic.com/mailman/listinfo/phobos
>

Doesn't hurt to be sure.  My best guess is that you have local changes that aren't submitted that are helping.

It wasn't a transient failure, it's failed several times now.

I'll narrow down which of those changes it was.

On 9/17/2010 5:55 PM, David Simcha wrote:
>  Win32.  I thought that was pretty clearly implied given that you made it clear
> before that the issue was Win32 specific.
> 
> On 9/17/2010 8:54 PM, Brad Roberts wrote:
>> Are you building on win32 or linux32?  It seems to be win32 specific.
>>
>> On 9/17/2010 5:51 PM, David Simcha wrote:
>>>   ??? Works for me.
>>>
>>> On 9/17/2010 8:45 PM, Brad Roberts wrote:
>>>> One of the changes between 2015 and 2018 broke the win32 build:
>>>>
>>>> http://d.puremagic.com/test-results/test_data.ghtml?dataid=3355
>>>>
>>>> std\typecons.d(336): Error: cannot evaluate empty("") at compile time
>>>> std\typecons.d(371): Error: cannot evaluate injectNamedFields() at compile time
>>>> std\typecons.d(371): Error: argument to mixin must be a string, not
>>>> (injectNamedFields())
>>>>

  (Bangs hand against forehead.)  I never moved std.typecons from my
TortoiseSVN repo directory to my DMD directory since last time I
updated.  Now that I did, yup, it's broken.

On 9/17/2010 8:58 PM, Brad Roberts wrote:
> Doesn't hurt to be sure.  My best guess is that you have local changes that aren't submitted that are helping.
>
> It wasn't a transient failure, it's failed several times now.
>
> I'll narrow down which of those changes it was.
>
> On 9/17/2010 5:55 PM, David Simcha wrote:
>>   Win32.  I thought that was pretty clearly implied given that you made it clear
>> before that the issue was Win32 specific.
>>
>> On 9/17/2010 8:54 PM, Brad Roberts wrote:
>>> Are you building on win32 or linux32?  It seems to be win32 specific.
>>>
>>> On 9/17/2010 5:51 PM, David Simcha wrote:
>>>>    ??? Works for me.
>>>>
>>>> On 9/17/2010 8:45 PM, Brad Roberts wrote:
>>>>> One of the changes between 2015 and 2018 broke the win32 build:
>>>>>
>>>>> http://d.puremagic.com/test-results/test_data.ghtml?dataid=3355
>>>>>
>>>>> std\typecons.d(336): Error: cannot evaluate empty("") at compile time
>>>>> std\typecons.d(371): Error: cannot evaluate injectNamedFields() at compile time
>>>>> std\typecons.d(371): Error: argument to mixin must be a string, not
>>>>> (injectNamedFields())
>>>>>
> _______________________________________________
> phobos mailing list
> phobos at puremagic.com
> http://lists.puremagic.com/mailman/listinfo/phobos
>

Ok, 2017 builds fine, 2018 introduces the problem.

http://www.dsource.org/projects/phobos/changeset/2018

I'm going to guess that you haven't updated from svn to pick up rsinfu's changes.


On 9/17/2010 5:58 PM, Brad Roberts wrote:
> Doesn't hurt to be sure.  My best guess is that you have local changes that aren't submitted that are helping.
> 
> It wasn't a transient failure, it's failed several times now.
> 
> I'll narrow down which of those changes it was.
> 
> On 9/17/2010 5:55 PM, David Simcha wrote:
>>  Win32.  I thought that was pretty clearly implied given that you made it clear
>> before that the issue was Win32 specific.
>>
>> On 9/17/2010 8:54 PM, Brad Roberts wrote:
>>> Are you building on win32 or linux32?  It seems to be win32 specific.
>>>
>>> On 9/17/2010 5:51 PM, David Simcha wrote:
>>>>   ??? Works for me.
>>>>
>>>> On 9/17/2010 8:45 PM, Brad Roberts wrote:
>>>>> One of the changes between 2015 and 2018 broke the win32 build:
>>>>>
>>>>> http://d.puremagic.com/test-results/test_data.ghtml?dataid=3355
>>>>>
>>>>> std\typecons.d(336): Error: cannot evaluate empty("") at compile time
>>>>> std\typecons.d(371): Error: cannot evaluate injectNamedFields() at compile time
>>>>> std\typecons.d(371): Error: argument to mixin must be a string, not
>>>>> (injectNamedFields())
>>>>>

On 09/17/2010 08:04 PM, Brad Roberts wrote:
> Ok, 2017 builds fine, 2018 introduces the problem.
>
> http://www.dsource.org/projects/phobos/changeset/2018
>
> I'm going to guess that you haven't updated from svn to pick up rsinfu's changes.

If that happened there is something wrong with the steps taken. svn doesn't allow such overwrites.

Andrei

Ah, ok. Don't do that :o).

Andrei

On 09/17/2010 08:03 PM, David Simcha wrote:
> (Bangs hand against forehead.)  I never moved std.typecons from my TortoiseSVN repo directory to my DMD directory since last time I updated. Now that I did, yup, it's broken.
>
> On 9/17/2010 8:58 PM, Brad Roberts wrote:
>> Doesn't hurt to be sure. My best guess is that you have local changes
>> that
>> aren't submitted that are helping.
>>
>> It wasn't a transient failure, it's failed several times now.
>>
>> I'll narrow down which of those changes it was.
>>
>> On 9/17/2010 5:55 PM, David Simcha wrote:
>>> Win32. I thought that was pretty clearly implied given that you made
>>> it clear
>>> before that the issue was Win32 specific.
>>>
>>> On 9/17/2010 8:54 PM, Brad Roberts wrote:
>>>> Are you building on win32 or linux32? It seems to be win32 specific.
>>>>
>>>> On 9/17/2010 5:51 PM, David Simcha wrote:
>>>>> ??? Works for me.
>>>>>
>>>>> On 9/17/2010 8:45 PM, Brad Roberts wrote:
>>>>>> One of the changes between 2015 and 2018 broke the win32 build:
>>>>>>
>>>>>> http://d.puremagic.com/test-results/test_data.ghtml?dataid=3355
>>>>>>
>>>>>> std\typecons.d(336): Error: cannot evaluate empty("") at compile time
>>>>>> std\typecons.d(371): Error: cannot evaluate injectNamedFields() at
>>>>>> compile time
>>>>>> std\typecons.d(371): Error: argument to mixin must be a string, not
>>>>>> (injectNamedFields())
>>>>>>
>> _______________________________________________
>> phobos mailing list
>> phobos at puremagic.com
>> http://lists.puremagic.com/mailman/listinfo/phobos
>>
>
> _______________________________________________
> phobos mailing list
> phobos at puremagic.com
> http://lists.puremagic.com/mailman/listinfo/phobos

It depends on the order of compilation.  The following succeeds:

% dmd -c std/concurrency.d std/string.d
% _

While the following fails (note the order of arguments):

% dmd -c std/string.d std/concurrency.d
std/typecons.d(336): Error: cannot evaluate empty("") at compile time
std/typecons.d(371): Error: cannot evaluate injectNamedFields() at compile time
std/typecons.d(371): Error: argument to mixin must be a string, not (injectNamedFields())

Anyways, I'll check in a workaround.


Shin