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

Remove Path class from the Dart SDK #11666

Closed
DartBot opened this issue Jul 2, 2013 · 7 comments
Closed

Remove Path class from the Dart SDK #11666

DartBot opened this issue Jul 2, 2013 · 7 comments
Assignees
@andersjohnsen
Labels
area-core-library SDK core library issues (core, async, ...); use area-vm or area-web for platform specific libraries. library-io
Milestone
M6

Comments

@DartBot
Copy link

DartBot commented Jul 2, 2013

This issue was originally filed by demis.b...@gmail.com

Introducing an interim Path type to manipulate string paths is an un-necessary friction given the dart:io File and Directory classes only work with strings. It's also a good show-case on how over-typing SDK API's are less re-usable, composable and overall harder to work with.

We should remove the Path type from the SDK given that better and redundant options (also made by the Dart Team) exist including pathos: http://pub.dartlang.org/packages/pathos

Ideally Dart SDK APIs should have a minimal and flexible API surface with special care taken when converting what is commonly else-where a string API behind types which as a side-effect can create friction and reduce interoperability. IOTW there should be a clear and visible benefit to typing string APIs in the SDK. When they exist they should be balanced against the ease-of-use and imposed friction compared with the existing node.js APIs:
http://nodejs.org/api/path.html

It's always an option for external higher-level libs to provide a typed API layer over it, that way their use is isolated to those who prefer and see value in it.
@kevmoo
Copy link
Member

kevmoo commented Jul 2, 2013

Original discussion: https://groups.google.com/a/dartlang.org/d/topic/misc/H7XG_KB6oZQ/discussion

Added Area-IO label.

@andersjohnsen
Copy link

Path is now deprecated, will be removed at the 11th of August.

An email will be sent out later today or tomorrow with instructions.

Cheers,

@DartBot
Copy link
Author

DartBot commented Jul 4, 2013

This comment was originally written by demis.b...@gmail.com

brilliant, thx.

@munificent
Copy link
Member

Cool! If there's any functionality in Path that you find pathos lacks, please don't hesitate to throw patches at me to get that moved over.

@sgjesse
Copy link
Contributor

sgjesse commented Jul 24, 2013

Added this to the M6 milestone.

@sgjesse
Copy link
Contributor

sgjesse commented Aug 15, 2013

The Path class was removed from dart:io in https://code.google.com/p/dart/source/detail?r=26181.

Added Fixed label.

@kevmoo
Copy link
Member

kevmoo commented May 14, 2014

Removed Area-IO label.
Added Area-Library, library-io labels.

@DartBot DartBot added Type-Defect area-core-library SDK core library issues (core, async, ...); use area-vm or area-web for platform specific libraries. library-io labels May 14, 2014
@DartBot DartBot added this to the M6 milestone May 14, 2014
@sgjesse sgjesse mentioned this issue Jun 3, 2015
Closed
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@andersjohnsen andersjohnsen

Labels
area-core-library SDK core library issues (core, async, ...); use area-vm or area-web for platform specific libraries. library-io
Projects
None yet
Milestone
M6
Development

No branches or pull requests
5 participants
@kevmoo @andersjohnsen @munificent @sgjesse @DartBot