Bug 11546 – string import dependency failure

Status
RESOLVED
Resolution
FIXED
Severity
enhancement
Priority
P2
Component
dmd
Product
D
Version
D2
Platform
All
OS
All
Creation time
2013-11-18T20:29:00Z
Last change time
2013-11-21T18:27:11Z
Keywords
pull
Assigned to
nobody
Creator
turkeyman

Comments

Comment #0 by turkeyman — 2013-11-18T20:29:51Z
vibe.d uses string imports to support it's 'diet' template system. The problem is, changing a .dt should cause a rebuild of the app, but the dependency is not realised and, visuald just claims "up to date". This is very annoying. How much trouble is it to make sure build dependencies properly account for string import?
Comment #1 by r.sagitario — 2013-11-19T00:21:42Z
The problem is that they are not written into the -deps dependency file by dmd.
Comment #2 by turkeyman — 2013-11-19T02:59:32Z
Really? I have a recollection of reading that string imports were added to the deps at some point... Found it. 2.064 seems to have added it: http://dlang.org/changelog.html (search for string import)
Comment #3 by r.sagitario — 2013-11-19T11:05:22Z
(In reply to comment #2) > Really? > I have a recollection of reading that string imports were added to the deps at > some point... > > Found it. 2.064 seems to have added it: http://dlang.org/changelog.html (search > for string import) Unfortunately this is only for the nonsense (IMO) option to print to stdout, not to the dependency file.
Comment #4 by turkeyman — 2013-11-19T17:40:34Z
(In reply to comment #3) > (In reply to comment #2) > > Really? > > I have a recollection of reading that string imports were added to the deps at > > some point... > > > > Found it. 2.064 seems to have added it: http://dlang.org/changelog.html (search > > for string import) > > Unfortunately this is only for the nonsense (IMO) option to print to stdout, > not to the dependency file. ...wow. That just makes no sense. Why the inconsistency?
Comment #5 by r.sagitario — 2013-11-20T00:28:48Z
dmd side of the fix: https://github.com/D-Programming-Language/dmd/pull/2835 >...wow. >That just makes no sense. Why the inconsistency? The stdout version also writes some other stuff, like used version/debug identifiers and pragma(lib) entries.
Comment #6 by github-bugzilla — 2013-11-21T13:54:41Z
Commit pushed to master at https://github.com/D-Programming-Language/dmd https://github.com/D-Programming-Language/dmd/commit/57f34b2abe1693f446d6856d480515c839476b0f Merge pull request #2835 from rainers/string_import_deps fix Issue 11546 - string import dependency failure