dep-info includes all transitvely linked files #23089

Activity

1. alexcrichton

   added a commit that references this issue on Mar 5, 2015

   rustc: Don't emit dep-info for imported files

   ...

   This issue will close once commit 9319252 is merged into the 'incoming' branch.

   9319252
2. Manishearth

   added a commit that references this issue on Mar 6, 2015

   Rollup merge of rust-lang#23090 - alexcrichton:dep-info, r=pnkfelix

   ...

   This issue will close once commit 0b7117b is merged into the 'master' branch.

   0b7117b
3. michaelwoerister commented on Mar 6, 2015

   

   michaelwoerister

   on Mar 6, 2015

   Member

   More actions

   Yes, driver::write_out_deps() probably needs to add a FileMap::is_imported() check to filter out files from other crates. I haven't had time to look into this in detail though.

   On March 5, 2015 11:38:23 PM CET, Alex Crichton notifications@github.com wrote:

   I think that the changes in #22235 mean that the dep-info output
   includes too many files now:

   $ touch foo.rs
   $ rustc foo.rs --crate-type lib --emit=dep-info
   $ cat foo.d
   

   output

   cc @michaelwoerister

   ---

   Reply to this email directly or view it on GitHub:
   #23089

   Sent from my Android device with K-9 Mail. Please excuse my brevity.
4. alexcrichton commented on Mar 6, 2015

   

   alexcrichton

   on Mar 6, 2015

   MemberAuthor

   More actions

   No worries! I did precisely that in #23090 :)
5. bors

   closed this as completedin #23090on Mar 7, 2015