Skip to content

Remove Path class from the Dart SDK #11666

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

5 participants

@DartBot
Dart member
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.

@skabet
skabet commented Jul 4, 2013

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
Dart member
DartBot commented Jul 4, 2013

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


brilliant, thx.

@munificent
Dart 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
Dart member
sgjesse commented Jul 24, 2013

Added this to the M6 milestone.

@sgjesse
Dart member
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
Dart member
kevmoo commented May 14, 2014

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

@skabet skabet was assigned by DartBot May 14, 2014
@DartBot DartBot added this to the M6 milestone May 14, 2014
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.