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

Jump to bottom

dart analyzer bat file does not work from within directories containing parenthesis #14795

Closed
ricowind opened this issue Nov 4, 2013 · 12 comments
Closed

dart analyzer bat file does not work from within directories containing parenthesis #14795

ricowind opened this issue Nov 4, 2013 · 12 comments
Labels
area-analyzer Use area-analyzer for Dart analyzer issues, including the analysis server and code completion. P3 A lower priority bug or feature request type-bug Incorrect behavior (everything from a crash to more subtle misbehavior)

Comments

@ricowind
Copy link
Contributor

ricowind commented Nov 4, 2013

To reproduce, extract a build from our builders, rename it to, e.g., foobar(), and run the analyzer on some dart code

Fails with a strange exception

Trying the same simple fix we did for dart2js does not work, there are more escaping to be done.
@clayberg
Copy link

clayberg commented Nov 5, 2013

cc @bwilkerson.
Set owner to @danrubel.
Added this to the Later milestone.
Removed Priority-Unassigned label.
Added Priority-Medium label.

@clayberg
Copy link

clayberg commented Apr 9, 2014

Removed this from the Later milestone.
Added this to the 1.4 milestone.

@kasperl
Copy link

kasperl commented May 8, 2014

Removed this from the 1.4 milestone.
Added this to the 1.5 milestone.

@kasperl
Copy link

kasperl commented Jun 4, 2014

Removed this from the 1.5 milestone.
Added this to the 1.6 milestone.

@kasperl
Copy link

kasperl commented Jul 10, 2014

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

@kasperl
Copy link

kasperl commented Aug 4, 2014

Removed Oldschool-Milestone-1.6 label.

@sethladd
Copy link
Contributor

This sounds like something we want to address.

Now that pub can run bin's, we could consider removing the .sh and .bat support for analyzer.

Removed Priority-Medium, Area-Editor labels.
Added Priority-Low, Area-Analyzer labels.

@ricowind
Copy link
Contributor Author

Now that pub can run bin's, we could consider removing the .sh and .bat support for analyzer.
I am sure that a bunch of people are running the analyzer from the command line as part of testing?

@ricowind ricowind added Type-Defect P3 A lower priority bug or feature request area-analyzer Use area-analyzer for Dart analyzer issues, including the analysis server and code completion. labels Aug 25, 2014
@bwilkerson bwilkerson removed the Triaged label Nov 4, 2015
@kevmoo kevmoo added type-bug Incorrect behavior (everything from a crash to more subtle misbehavior) and removed Type-Defect labels Mar 1, 2016
@pq
Copy link
Member

pq commented Jun 29, 2018

Bumping this one... @danrubel do you have any even vague memories of this issue? Should we keep it open?

@danrubel
Copy link

No idea. Good question.

@pq pq unassigned danrubel Jun 29, 2018
@bwilkerson
Copy link
Member

I'm going to guess that we're not going to fix this (unless it's already been fixed).

@ricowind
Copy link
Contributor Author

you can just test it by doing the suggested repro in the initial bug :-)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
area-analyzer Use area-analyzer for Dart analyzer issues, including the analysis server and code completion. P3 A lower priority bug or feature request type-bug Incorrect behavior (everything from a crash to more subtle misbehavior)
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
8 participants
@sethladd @kevmoo @pq @kasperl @danrubel @ricowind @bwilkerson @clayberg