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

Add .gitignore for libllhttp.pc generated by cmake #225

Merged
merged 1 commit into from
Jun 14, 2023

Conversation

halx99
Copy link

@halx99 halx99 commented Jun 13, 2023

No description provided.

@ShogunPanda
Copy link
Contributor

Thanks for your contribution. Why do you think this file should be ignored?

@halx99
Copy link
Author

halx99 commented Jun 14, 2023

Because it's generate file, and when I use https://github.com/cpm-cmake/CPM.cmake to manage llhttp as dependency of my cmake based project, will always failed when run cmake gen second time. I needs to delete libllhttp.pc manually to fix the problem

@ShogunPanda
Copy link
Contributor

Can you please provide me a small repro example so I can play with it?

@halx99
Copy link
Author

halx99 commented Jun 14, 2023

Can you please provide me a small repro example so I can play with it?

This one: https://github.com/halx99/llhttp-pull-225

@ShogunPanda
Copy link
Contributor

Ok. Verified, you are right. Thanks for the fix!

@ShogunPanda ShogunPanda merged commit be96a82 into nodejs:release Jun 14, 2023
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