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

Frog broken in SDK #2144

Closed
DartBot opened this issue Mar 14, 2012 · 10 comments
Closed

Frog broken in SDK #2144

DartBot opened this issue Mar 14, 2012 · 10 comments
Assignees

Comments

@DartBot
Copy link

DartBot commented Mar 14, 2012

This issue was originally filed by darren.j....@gmail.com


What steps will reproduce the problem?

  1. Download 64-bit version (listed below) of the editor from dartlang.org
  2. Attempt to compile a sample (such as sunflower)
  3. Observe that the compilation fails because the path to html_frog.dart is wrong.

For example:
C:\Program Files\dart\dart\dart-sdk\lib\html\html_frog.dart
but the compiler is looking for it at:
C:\Program Files\dart\dart\dart-sdk\lib\html\frog\html_frog.dart

What is the expected output? What do you see instead?
Expected successful compilation. Instead confronted with error message regarding path to html_frog.dart.

org.eclipse.core.runtime.CoreException: Failure to launch - unable to generate JavaScript for sunflower.html.

error: File not found: C:/Program Files/dart/dart/dart-sdk/lib/html/frog/html_frog.dart
compilation failed with 1 errors
Compilation failed
Failure to launch - unable to generate JavaScript for sunflower.html.

error: File not found: C:/Program Files/dart/dart/dart-sdk/lib/html/frog/html_frog.dart
compilation failed with 1 errors
Compilation failed

What version of the product are you using? On what operating system?
Dart Editor: Version 0.1.0.201203131146, Build 5427
Dart SDK: Version 5425
Dartium not installed.
Windows 7 Professional, Service Pack 1.

Please provide any additional information below.
Easy fix was to create the frog folder and move the file down into it, but irritating.

@madsager
Copy link
Contributor

Added Triaged label.

@keertip
Copy link
Contributor

keertip commented Mar 14, 2012

Set owner to @keertip.

@keertip
Copy link
Contributor

keertip commented Mar 14, 2012

Issue #2138 has been merged into this issue.

@keertip
Copy link
Contributor

keertip commented Mar 14, 2012

Frog needs to be modified to look at the right place in the dart-sdk for the library files. frog is called from the editor to generate javascript using the following

/pathTo/dart-sdk/bin/dart --new_gen_heap_size=256
/pathTo/dart-sdk/bin/frogc.dart --compile-only --suppress_warnings --no_colors
--libdir=/pathTo/dart-sdk/lib --out=/pathTo/samples/sunflower/Sunflower.dart.js /pathTo/samples/sunflower/Sunflower.dart

Compilation error :

error: File not found: /pathTo/dart-sdk/lib/html/frog/html_frog.dart

Frog is looking for the file at lib/html/frog whereas the file is in /lib/html folder.

The dart-sdk structure is

..dart-sdk/

....bin/

......dart or dart.exe (executable)

......frogc.dart

......frogsh (coming later)

....lib/

......builtin/

........builtin_runtime.dart

......io/

........io_runtime.dart

........runtime/

......core/

........core_{frog, runtime}.dart

........{frog, runtime}/

......coreimpl/

........coreimpl_{frog, runtime}.dart

........{frog, runtime}/

......dartdoc/

......isolate/

........isolate_{frog, runtime}.dart

........{frog, runtime}/

......dom/

........dom.dart

......frog/

......html/

........html_frog.dart

........html_dartium.dart

......json/

........json_frog.dart

#.........json.dart

........{frog}/

......uri/

........uri.dart

......utf/

......(more will come here)

....util/

......(more will come here)


Removed the owner.
Removed Area-Editor label.
Added Area-Frog label.

@danrubel
Copy link

@john:
Could you look into this?
One thing that would help alleviate issues like these in the future would be to run all Frog tests from within the SDK.


Set owner to @jmesserly.
Changed the title to: "Frog broken in SDK".

@jmesserly
Copy link

Set owner to @danrubel.

@danrubel
Copy link

@kasper
This is a fairly high priority bug I'm hoping we can fix. We can't launch on Windows (no Dartium yet) without a fix or workaround. If necessary, we can work around the problem by duplicating the html_frog.dart file in the SDK for now.
Please let me know.


Set owner to @kasperl.

@sethladd
Copy link
Contributor

Issue #2170 has been merged into this issue.

@efortuna
Copy link
Contributor

Fixed in revision 5488


Set owner to @efortuna.
Added Fixed label.

@kasperl
Copy link

kasperl commented Mar 15, 2012

Thanks a lot, Emily! Really appreciate it.

copybara-service bot pushed a commit that referenced this issue Jun 20, 2023
… webdev

Revisions updated by `dart tools/rev_sdk_deps.dart`.

bazel_worker (https://github.com/dart-lang/bazel_worker/compare/c14a268..f7b714e):
  f7b714e  2023-06-16  Jacob MacDonald  add firehose publishing workflow (#76)
  0743f11  2023-06-16  Parker Lougheed  Require protobuf 3.0.0 and prepare for 1.0.3 release (#75)

dartdoc (https://github.com/dart-lang/dartdoc/compare/5799424..793d575):
  793d575d  2023-06-14  Sam Rawlins  Remove unused coverage feature (#3444)
  19ad21a4  2023-06-14  Parker Lougheed  Replace `@sealed` meta usage with keyword (#3445)
  8e9d89e3  2023-06-12  Sam Rawlins  Remove all `@deprecated` elements. (#3250)
  2019791c  2023-06-12  dependabot[bot]  Bump actions/checkout from 3.5.2 to 3.5.3 (#3442)
  d34c7067  2023-06-12  dependabot[bot]  Bump github/codeql-action from 2.3.6 to 2.13.4 (#3443)

http (https://github.com/dart-lang/http/compare/ba7eb60..8c25057):
  8c25057  2023-06-16  Nate Bosch  Unskip a skipped test (#966)

protobuf (https://github.com/dart-lang/protobuf/compare/a9bf79f..e76bd74):
  e76bd74  2023-06-19  Ömer Sinan Ağacan  Remove protoc_plugin exe from repo, update .gitignore (#851)
  4b71e60  2023-06-16  Ömer Sinan Ağacan  Fix a change in protoc_plugin changelog (#846)
  f4ffa1d  2023-06-16  Ömer Sinan Ağacan  Fix missing protobuf import in grpc files (#845)

source_maps (https://github.com/dart-lang/source_maps/compare/dd5b5cd..58eef30):
  58eef30  2023-06-12  Kevin Moore  Require Dart 3, update lints (#79)

tools (https://github.com/dart-lang/tools/compare/8d6e8b8..02b5cc5):
  02b5cc5  2023-06-16  Brian Wilkerson  Add documentation for server events (#115)
  698bfe8  2023-06-15  Elias Yishak  Introducing new `Event` class that can be send via `analytics.send(Event event)` (#114)

webdev (https://github.com/dart-lang/webdev/compare/81ae77a..b58edb7):
  b58edb7d  2023-06-16  Elliott Brooks  Fix the DCM workflow (#2147)
  1551fe6c  2023-06-16  Elliott Brooks  Wrap VM service API methods in an error handler to convert any exceptions into type `RPCError`
(#2144)
  06abe901  2023-06-14  Elliott Brooks  Add more tests to daily testing cron job (#2138)
  5cb2dcf6  2023-06-13  Elliott Brooks  Use new error codes from package:vm_service (#2141)

Change-Id: I6779c6d20fbc69e83848f469df3e0b4d21420a9a
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/310322
Auto-Submit: Devon Carew <devoncarew@google.com>
Reviewed-by: Konstantin Shcheglov <scheglov@google.com>
Commit-Queue: Devon Carew <devoncarew@google.com>
copybara-service bot pushed a commit that referenced this issue Dec 11, 2023
…h, protobuf, sync_http, test, tools, web, webdev

Revisions updated by `dart tools/rev_sdk_deps.dart`.

ecosystem (https://github.com/dart-lang/ecosystem/compare/79a1bc5..ce707fb):
  ce707fb  2023-12-06  Moritz  Use Flutter in `health.yaml` (#206)

fixnum (https://github.com/dart-lang/fixnum/compare/6b0888c..3e08c0d):
  3e08c0d  2023-12-04  Kevin Moore  drop outdated lints (#123)

http_parser (https://github.com/dart-lang/http_parser/compare/1cf5b7c..224c1a9):
  224c1a9  2023-12-04  Kevin Moore  drop outdated lints (#82)

leak_tracker (https://github.com/dart-lang/leak_tracker/compare/a618a55..8644f8b):
  8644f8b  2023-12-11  Lucas.Xu  chore: bump leak_tracking_flutter_testing version to 1.0.12-wip (#192)
  cac195d  2023-12-11  Binni Goel  Fix. typos in leak_tracker (#189)
  9fd7932  2023-12-11  Lucas.Xu  fix: add missing on leak callback (#190)
  1ab3140  2023-12-08  Lucas.Xu  chore: refactor code style (#186)

mockito (https://github.com/dart-lang/mockito/compare/fb8a2b5..57a7c82):
  57a7c82  2023-12-06  Copybara-Service  Merge pull request #721 from Sese-Schneider:master
  c5de658  2023-12-06  Sese Schneider  Apply suggestions from code review
  1a652b8  2023-11-21  Sebastian Schneider  Use `posix` style for local imports

path (https://github.com/dart-lang/path/compare/18ec71f..115ea2a):
  115ea2a  2023-12-05  Devon Carew  rev the version of lints used; prep for publishing (#155)

protobuf (https://github.com/dart-lang/protobuf/compare/bb19774..20ec685):
  20ec685  2023-12-05  Ömer Sinan Ağacan  Small PbList improvements (#906)

sync_http (https://github.com/dart-lang/sync_http/compare/d8e9f3d..f96db95):
  f96db95  2023-12-05  Kevin Moore  Latest lints, require Dart 3.0, use mini-libraries (#42)

test (https://github.com/dart-lang/test/compare/9fffb48..43ff5bf):
  43ff5bf9  2023-12-11  Ömer Sinan Ağacan  Fix Dart2Wasm spelling (#2155)
  7d61eef9  2023-12-08  Derek Xu  Update frontend_server_client constraint to allow version 4.0.0 (#2153)
  001c5381  2023-12-07  Jacob MacDonald  fix precompiled vm tests loaded from a package: uri (#2152)
  fb3b5725  2023-12-05  Nate Bosch  Use package: URIs for imports when possible (#2148)
  03cc56ec  2023-12-05  Jacob MacDonald  mark flaky windows test as skipped (#2151)
  6e7e4fa8  2023-12-05  Jacob MacDonald  Add regression test for relative imports in tests under `lib` (#2147)
  84c366a7  2023-12-05  Jacob MacDonald  fix broken tests (#2149)
  e49ae54b  2023-12-05  Jacob MacDonald  Promote dart2wasm compiler support to the stable browser platform. (#2144)

tools (https://github.com/dart-lang/tools/compare/c63dcb5..ed81684):
  ed81684  2023-12-06  Elias Yishak  Fix consent message formatting (#215)
  c4d515a  2023-12-05  Elias Yishak  Enum + event added for `commandUsageValues` (#211)

web (https://github.com/dart-lang/web/compare/865aeaf..acf0beb):
  acf0beb  2023-12-07  Devon Carew  fixes to the return types of several Node helper extension methods (#121)
  9d7b33e  2023-12-06  Srujan Gaddam  Handle Dart SDK change to extension types (#116)
  0ae45ec  2023-12-05  Devon Carew  Update README.md (#117)
  34b3618  2023-12-05  Devon Carew  have web.dart export the helpers; deprecate helpers.dart (#115)

webdev (https://github.com/dart-lang/webdev/compare/63e09e5..8375cd6):
  8375cd66  2023-12-08  Elliott Brooks  Add popup for copying the app ID (#2299)

Change-Id: Ibf5f5eca74c56bf31d50bddabbe9da2a2e6fd716
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/341021
Commit-Queue: Devon Carew <devoncarew@google.com>
Reviewed-by: Konstantin Shcheglov <scheglov@google.com>
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

8 participants