Rely on dart:io to deal with symlinks in pub.

Rely on dart:io to deal with symlinks in pub.

Now that issue 2516 is solved, we don't need these hacks.

Committed: https://code.google.com/p/dart/source/detail?r=7428

[Bob Nystrom, 2012-05-08 16:49:19]

A truly beautiful patch. It warms my heart to see hacks get deleted. LGTM.

[Mads Ager (google), 2012-05-09 10:06:23]

Nice! Happy to see this working without the hacks! :)

https://chromiumcodereview.appspot.com/10378044/diff/1/utils/pub/io.dart
File utils/pub/io.dart (right):

https://chromiumcodereview.appspot.com/10378044/diff/1/utils/pub/io.dart#newc...
utils/pub/io.dart:297: Future<ProcessResult> runProcess(String executable,
List<String> args,
We should see if we can use:

new Process.run(..., (exitCode, stdout, stderr) {
  ...
});

for this as well. That returns strings and not a list of lines though.

[Bob Nystrom, 2012-05-09 15:49:11]

https://chromiumcodereview.appspot.com/10378044/diff/1/utils/pub/io.dart
File utils/pub/io.dart (right):

https://chromiumcodereview.appspot.com/10378044/diff/1/utils/pub/io.dart#newc...
utils/pub/io.dart:297: Future<ProcessResult> runProcess(String executable,
List<String> args,
On 2012/05/09 10:06:23, Mads Ager wrote:
> We should see if we can use:
> 
> new Process.run(..., (exitCode, stdout, stderr) {
>   ...
> });
> 
> for this as well. That returns strings and not a list of lines though.

Good call. I wrote this before Process.run was finished but we will probably
change to use that at some point.