Navigation Menu

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

Dart should have a yaml library #4126

Closed
DartBot opened this issue Jul 17, 2012 · 12 comments
Closed

Dart should have a yaml library #4126

DartBot opened this issue Jul 17, 2012 · 12 comments
Assignees
Labels
area-core-library SDK core library issues (core, async, ...); use area-vm or area-web for platform specific libraries. type-enhancement A request for a change that isn't a bug
Milestone

Comments

@DartBot
Copy link

DartBot commented Jul 17, 2012

This issue was originally filed by jjinux...@google.com


Apparently, Dart already comes with a yaml library. It's part of pub (dart/utils/pub/yaml/yaml.dart). We should make that part of the standard library.

@sethladd
Copy link
Contributor

I imagine this will be a pub package soon.


cc @munificent.
Removed Type-Defect label.
Added Type-Enhancement, Area-Pub, Triaged labels.

@munificent
Copy link
Member

Added this to the Later milestone.

@munificent
Copy link
Member

Issue #4619 has been merged into this issue.

@sethladd
Copy link
Contributor

Removed Area-Pub label.
Added Area-Library, Library-YAML labels.

@jmesserly
Copy link

As soon as this CL is approved, I'll upload it
https://codereview.chromium.org/11622011/


Set owner to @jmesserly.
Removed this from the Later milestone.
Added this to the M3 milestone.

@DartBot
Copy link
Author

DartBot commented Dec 18, 2012

This comment was originally written by ross.m....@gmail.com


awesome!

@DartBot
Copy link
Author

DartBot commented Jan 21, 2013

This comment was originally written by george.moscho...@gmail.com


better make this a pub library!

@DartBot
Copy link
Author

DartBot commented Jan 21, 2013

This comment was originally written by googlegroups...@kaioa.com


Pub itself depends on this library. Without it, pub won't be able to parse the YAML file which references it. So, it really needs to be in the standard lib.

@munificent
Copy link
Member

Without it, pub won't be able to parse the YAML file which references it. So, it really needs to be in the standard lib.

That's not strictly true. You're right that there's a bootstrap problem: pub needs to use it and it obviously can't rely on installing it from pub. However, that's true for other packages that pub uses as well: http, path, and oauth.

The solution is pretty straightforward: pub just includes a "vendored-in" copy of the packages it needs internally.

So YAML likely will become a package when we get around to pulling it out of pub.

@anders-sandholm
Copy link
Contributor

Removed this from the M3 milestone.
Added this to the M4 milestone.

@larsbak
Copy link

larsbak commented May 28, 2013

Removed this from the M4 milestone.
Added this to the M5 milestone.

@jmesserly
Copy link

fixed a long time ago :)


Added Fixed label.

@DartBot DartBot added Type-Enhancement area-core-library SDK core library issues (core, async, ...); use area-vm or area-web for platform specific libraries. labels Aug 28, 2013
@DartBot DartBot added this to the M5 milestone Aug 28, 2013
@kevmoo kevmoo added type-enhancement A request for a change that isn't a bug and removed type-enhancement labels Mar 1, 2016
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-core-library SDK core library issues (core, async, ...); use area-vm or area-web for platform specific libraries. type-enhancement A request for a change that isn't a bug
Projects
None yet
Development

No branches or pull requests

7 participants