• Manu (4/4) Nov 20 2013 https://d.puremagic.com/issues/show_bug.cgi?id=11546
• robert (2/7) Nov 20 2013 I fixed that some time ago, see the -deps=filename switch.
• robert (7/15) Nov 20 2013 Ooops, just the other way round: -deps without filename. Reason
• Dicebot (6/8) Nov 20 2013 Is this actually a real concern? There are not that much tools
• Dmitry Olshansky (5/8) Nov 20 2013 Indeed it's not like it's we are shipping an OS kernel update. Why not
• Jacob Carlborg (5/6) Nov 20 2013 If the output wasn't there in the first place wouldn't it be possible to...
• Robert (6/8) Nov 20 2013 No, the format of the file does not allow that. Whether
• Rainer Schuetze (7/16) Nov 20 2013 I created a pull request to add string import dependencies by replacing

Manu <turkeyman gmail.com> writes:

https://d.puremagic.com/issues/show_bug.cgi?id=11546

What's the story here?
Why aren't string imports written to the .deps file? Hard to tooling that
works properly without that info.

"robert" <jfanatiker gmx.at> writes:

On Wednesday, 20 November 2013 at 08:35:37 UTC, Manu wrote:
 https://d.puremagic.com/issues/show_bug.cgi?id=11546

 What's the story here?
 Why aren't string imports written to the .deps file? Hard to 
 tooling that
 works properly without that info.

I fixed that some time ago, see the -deps=filename switch.

"robert" <jfanatiker gmx.at> writes:

On Wednesday, 20 November 2013 at 14:13:42 UTC, robert wrote:
 On Wednesday, 20 November 2013 at 08:35:37 UTC, Manu wrote:
 https://d.puremagic.com/issues/show_bug.cgi?id=11546

 What's the story here?
 Why aren't string imports written to the .deps file? Hard to 
 tooling that
 works properly without that info.

 I fixed that some time ago, see the -deps=filename switch.

Ooops, just the other way round: -deps without filename. Reason 
for stdout: There were some people concerned about adding another 
command line switch, so I just recyclied -deps. The output of 
-deps=filename wasn't changed for backwards compatibility.

Best regards,

Robert

"Dicebot" <public dicebot.lv> writes:

On Wednesday, 20 November 2013 at 14:17:54 UTC, robert wrote:
 The output of -deps=filename wasn't changed for backwards 
 compatibility.

Is this actually a real concern? There are not that much tools 
that use that file output and I can't imagine why they would want 
to keep incorrect dependency handling in presence of string 
imports. Doing an early warning about future change should be 
enough.

Dmitry Olshansky <dmitry.olsh gmail.com> writes:

20-Nov-2013 18:45, Dicebot пишет:
 On Wednesday, 20 November 2013 at 14:17:54 UTC, robert wrote:
 The output of -deps=filename wasn't changed for backwards compatibility.

 Is this actually a real concern?

Indeed it's not like it's we are shipping an OS kernel update. Why not 
just notify/wait a bit/do the change?



-- 
Dmitry Olshansky

Jacob Carlborg <doob me.com> writes:

On 2013-11-20 15:17, robert wrote:

 The output of -deps=filename wasn't changed for backwards compatibility.

If the output wasn't there in the first place wouldn't it be possible to 
add new information in a backwards compatible way?

-- 
/Jacob Carlborg

"Robert" <jfanatiker gmx.at> writes:

 If the output wasn't there in the first place wouldn't it be 
 possible to add new information in a backwards compatible way?

No, the format of the file does not allow that. Whether 
compatibility is a concern or not was not for me to decide. I 
assumed it is not a good idea to break tools this abruptly. I 
thought it would be better to add just a new way of getting 
dependencies (which has an extensible format, btw.), the old one 
can then be deprecated and removed sometime in the future.

Rainer Schuetze <r.sagitario gmx.de> writes:

On 20.11.2013 22:42, Robert wrote:
 If the output wasn't there in the first place wouldn't it be possible
 to add new information in a backwards compatible way?

 No, the format of the file does not allow that. Whether compatibility is
 a concern or not was not for me to decide. I assumed it is not a good
 idea to break tools this abruptly. I thought it would be better to add
 just a new way of getting dependencies (which has an extensible format,
 btw.), the old one can then be deprecated and removed sometime in the
 future.

I created a pull request to add string import dependencies by replacing 
the protection attribute with "string": 
https://github.com/D-Programming-Language/dmd/pull/2835

Visual D just ignores the new lines, but newer versions can support it.

I don't think the other output of -deps (without filename) is needed for 
make-like build tools.