Skip to content
This repository was archived by the owner on Feb 18, 2024. It is now read-only.

Conversation

gfdev
Copy link

@gfdev gfdev commented Jun 24, 2016

No description provided.

@guybedford
Copy link
Member

Thanks for posting. Correct me if I'm missing something but this is a
browser code only, so I'm not sure this is useful in the browser?

In jspm, the code will conditionally switch to using the '@node/fs' module
when running in Node.
On Fri, 24 Jun 2016 at 16:54, Gordon Freeman [email protected]
wrote:


You can view, comment on, or merge this pull request online at:

#5
Commit Summary

  • realpathSync added

File Changes

Patch Links:


You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
#5, or mute the thread
https://github.com/notifications/unsubscribe/AAkiyrJpf1epUg0mmHuTM9syY9549zeyks5qO--XgaJpZM4I917T
.

@gfdev
Copy link
Author

gfdev commented Jun 27, 2016

Thanks for reply! The reason is that I want to use UglifyJS2 in browser and it use realpathSync method of fs module.

@guybedford
Copy link
Member

Are you making progress with that? Sounds like quite a job. Perhaps just returning path.resolve is adequate here then though?

@gfdev
Copy link
Author

gfdev commented Jun 27, 2016

The job is in progress. I think path.resolve will be good.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants