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

Allow C/C++ sources in "perl_xs" target. #23

Merged
merged 2 commits into from
Jan 13, 2022

Conversation

PiotrSikora
Copy link
Contributor

Signed-off-by: Piotr Sikora piotrsikora@google.com

Signed-off-by: Piotr Sikora <piotrsikora@google.com>
@PiotrSikora
Copy link
Contributor Author

@skeletonkey note that while this can be used as a workaround for #24, it's also a valid feature on it's own (e.g. see how FCGI module used in examples is built using perl Makefile.PL && make).

@PiotrSikora
Copy link
Contributor Author

@skeletonkey friendly ping.

@skeletonkey
Copy link
Collaborator

@PiotrSikora I apologize about the prolonged silence. I have not gotten anywhere with this issue. I have reached out to the PerlXS mailing list hoping that someone there will have ideas on how to move this forward.

@PiotrSikora
Copy link
Contributor Author

@skeletonkey friendly ping.

@skeletonkey
Copy link
Collaborator

@PiotrSikora I've taken a look and haven't been able to figure out how to fix the merge conflicts for your branch. Can you please make the minor fixes and then I can merge it. Thanks for you patience.

Signed-off-by: Piotr Sikora <piotrsikora@google.com>
@PiotrSikora
Copy link
Contributor Author

@skeletonkey Done.

@skeletonkey skeletonkey merged commit d0f6982 into bazelbuild:master Jan 13, 2022
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.

None yet

2 participants