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

editor: packages in nested directories confuse pub serve and the debugger #19935

Closed
DartBot opened this issue Jul 9, 2014 · 7 comments
Closed

editor: packages in nested directories confuse pub serve and the debugger #19935

DartBot opened this issue Jul 9, 2014 · 7 comments
Assignees
@keertip
Labels
closed-obsolete Closed as the reported issue is no longer relevant
Milestone
1.9

Comments

@DartBot
Copy link

DartBot commented Jul 9, 2014

This issue was originally filed by @jptrainor

What steps will reproduce the problem?

  1. Untar the attachment and open the "Sandbox" folder in the Dart editor.
  2. Set a break point in Sandbox/somePackages/sub_proj_A/lib/sub/sub_sub_proj_A.dart
  3. Set a break point in Sandbox/somePackages/sub_proj_A/lib/sub_proj_A.dart
  4. Run Sandbox/somePackages/sub_proj_A/test/sub_proj_A_test.html in Dartium using pub serve (it won't work at when you run it).
  5. Run it again but turn on pub serve in the debug configuration (it will run this time but it will miss the breakpoint set in step 2).

What is the expected output? What do you see instead?

I expect it to break at both break points and I expect to see the following in the console after it completes:

    sub proj A
    sub sub project A.

  It doesn't work at all with pub-serve because pub server doesn't seem to resolve the path due to the initial "somePackages" directory.

  With put serve turned off it runs, however it misses the break point in sub_sub_proj_A.dart.

What version of the product are you using? On what operating system?

Dart Editor version 1.5.3.release (STABLE)
Dart SDK version 1.5.3
OSX 10.9.4
java version "1.7.0_40"

Please provide any additional information below.

Attachment:
Sandbox.tar.gz (869 Bytes)
@DartBot
Copy link
Author

DartBot commented Jul 9, 2014

This comment was originally written by @jptrainor

Same happens with Dart Edtior pre-release 1.5.7 (not officially released yet - I picked it up from a link posted on Dart Misc with a request by Kasper Lund for people to test it.)

Here is the failing pub server output

Serving sub_proj_A test on http://127.0.0.1:8080
Build completed successfully
[test] GET /sub_proj_A_test.html => sub_proj_A|test/sub_proj_A_test.html
[test] GET /sub_proj_A_test.dart => sub_proj_A|test/sub_proj_A_test.dart
[test] GET /packages/browser/dart.js => browser|lib/dart.js
[test] GET /packages/sub_proj_A/sub_proj_A.dart => sub_proj_A|lib/sub_proj_A.dart
[test] GET /packages/sub_proj_a/sub/sub_sub_proj_A.dart => Could not find asset sub_proj_a|lib/sub/sub_sub_proj_A.dart.
[test] GET /favicon.ico => Could not find asset sub_proj_A|test/favicon.ico.

@kasperl
Copy link

kasperl commented Jul 10, 2014

Marking this as an editor issue for now, but CC'ing Bob for some pub insights.

cc @munificent.
Added Area-Editor, Triaged labels.

@DartBot
Copy link
Author

DartBot commented Jul 10, 2014

This comment was originally written by @jptrainor

Important correction:
5. Run it again but turn off pub serve in the debug configuration (it will run this time but it will miss the breakpoint set in step 2).

@clayberg
Copy link

cc @devoncarew.
Set owner to @keertip.
Removed Priority-Unassigned label.
Added Priority-Medium label.

@sethladd
Copy link
Contributor

Added Editor-Debugging label.

@keertip
Copy link
Contributor

keertip commented Feb 18, 2015

Pub serve can serve sub directories.

Added AssumedStale label.

@clayberg
Copy link

Added this to the 1.9 milestone.

@DartBot DartBot added Type-Defect closed-obsolete Closed as the reported issue is no longer relevant labels Mar 25, 2015
@DartBot DartBot added this to the 1.9 milestone Mar 25, 2015
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@keertip keertip

Labels
closed-obsolete Closed as the reported issue is no longer relevant
Projects
None yet
Milestone
1.9
Development

No branches or pull requests
5 participants
@sethladd @kasperl @keertip @clayberg @DartBot