Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

LibraryMirror does not expose imports #10360

Closed
alan-knight opened this issue May 1, 2013 · 8 comments
Closed

LibraryMirror does not expose imports #10360

alan-knight opened this issue May 1, 2013 · 8 comments
Labels
area-library library-mirrors P2 A bug or feature request we're likely to work on type-bug Incorrect behavior (everything from a crash to more subtle misbehavior)

Comments

@alan-knight
Copy link
Contributor

This is mentioned in dartbug.com/5897 but probably deserves its own separate entry. LibraryMirror does not expose its imports or exports, which limits its usefulness considerably.

@jmesserly
Copy link

fwiw, I was just looking for this too.

@gbracha
Copy link
Contributor

gbracha commented Aug 23, 2013

I don't expect this to make it into 1.0.


Set owner to @gbracha.
Added this to the Later milestone.
Added Accepted label.

@sigmundch
Copy link
Member

I just run into this limitation too.

Any plans to address this now that we are past 1.0?

@sigmundch
Copy link
Member

FYI - it seems that this is partially implemented for dart2js mirrors already. Dart2js exposes these mirrors as LibraryMirror.libraryDependencies:

https://code.google.com/p/dart/source/browse/branches/bleeding_edge/dart/sdk/lib/_internal/compiler/implementation/mirrors/mirrors.dart#­310

and the type LibraryDependencyMirror:
https://code.google.com/p/dart/source/browse/branches/bleeding_edge/dart/sdk/lib/_internal/compiler/implementation/mirrors/mirrors.dart#­314

However, these APIs not exposed in the common API (dart:mirrors), they are not implemented in vm mirrors, and they are not complete. In particular, both part directives and meta-data associated with any directives are missing.

@rmacnak-google
Copy link
Contributor

VM in r33441

@kasperl
Copy link

kasperl commented Jul 10, 2014

Removed this from the Later milestone.
Added Oldschool-Milestone-Later label.

@kasperl
Copy link

kasperl commented Aug 4, 2014

Removed Oldschool-Milestone-Later label.

@kevmoo kevmoo added P2 A bug or feature request we're likely to work on type-bug Incorrect behavior (everything from a crash to more subtle misbehavior) and removed accepted labels Feb 29, 2016
@bkonyi
Copy link
Contributor

bkonyi commented Jun 18, 2018

See here.

@bkonyi bkonyi closed this as completed Jun 18, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-library library-mirrors P2 A bug or feature request we're likely to work on type-bug Incorrect behavior (everything from a crash to more subtle misbehavior)
Projects
None yet
Development

No branches or pull requests

8 participants