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

Support running the packager on Windows #781

Closed
wants to merge 4 commits into from
Closed

Support running the packager on Windows #781

wants to merge 4 commits into from

Conversation

robarnold
Copy link

These are all commits from issue #468 that I am turning into a pull request (thanks @joewood!). I am seeing different behavior between my laptop and desktop so I don't think this solves all the problems (ex: running dir in a different terminal will cause the packager to crash) but it's a start.

The discovered file paths returned by node have backslash on Windows, so the test was failing - essentially ignoring the blacklist.
When the packager is run from node_modules, there's some code in packager.js that tests for this and sets the project root to ../../.
Watchman doesn't currently work on windows and the fallback path can take longer to initialize, especially on slower machines.
@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Apr 9, 2015
@joewood
Copy link

joewood commented Apr 14, 2015

I've been splitting out the changes and submitting as separate PRs. First one here: #807.
Also, ensuring this is a no-op on Linux, testing for the OS type.

@robarnold
Copy link
Author

Happy to let you do the PRs instead! I'll close this out then when I get back to my work machine.

-----Original Message-----
From: "Joe Wood" notifications@github.com
Sent: ‎4/‎13/‎2015 6:32 PM
To: "facebook/react-native" react-native@noreply.github.com
Cc: "robarnold" robarnold@cs.cmu.edu
Subject: Re: [react-native] Support running the packager on Windows (#781)

I've been splitting out the changes and submitting as separate PRs. First one here: #807.
Also, ensuring this is a no-op on Linux, testing for the OS type.

Reply to this email directly or view it on GitHub.

@robarnold
Copy link
Author

Closing so that @joewood can post these individually and respond to review comments.

@robarnold robarnold closed this Apr 15, 2015
ayushjainrksh referenced this pull request in MLH-Fellowship/react-native Jul 2, 2020
* Explicitly state that `activeOpacity` requires `underlayColor`

It's clear from facebook#11834 that the underlying issue won't be fixed, so we should update the documentation so people understand that for `activeOpacity` to work, they need to set `underlayColor`.

* Update touchablehighlight.md
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants