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

fix(watchman) #7504

Open
wants to merge 21 commits into
base: main
Choose a base branch
from
Open

fix(watchman) #7504

wants to merge 21 commits into from

Conversation

Dunedubby
Copy link

Created a new pull request because my older one was bogged down by a lot of commits to externally fix a problem (CMake prefix) which hopefully is now fixed in the pkgx framework. This pull request has a different scope: getting the WATCHMAN_STATE_DIR variable to be relocatable.

@Dunedubby
Copy link
Author

Dunedubby commented Oct 2, 2024

Just wanted to point out that this seems somewhat relevant: https://aur.archlinux.org/packages/edencommon (the comments).

@jhheider
Copy link
Contributor

jhheider commented Oct 2, 2024

i'm not surprised others are having issues. these package don't follow any versioniong, other than weekly releases, whether or not they're stable, and the versions are all tightly-coupled.

@jhheider
Copy link
Contributor

jhheider commented Oct 2, 2024

Fine to do in a single branch, but the rebuilds won't be available since they're done in parallel.

@jhheider
Copy link
Contributor

jhheider commented Oct 5, 2024

#7539 moves fbthrift to gcc, and fixes the recent build errors. it may help. i'll look at fb303 too.

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

Successfully merging this pull request may close these issues.

2 participants