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

RFC: use system pugixml #242

Closed
wants to merge 1 commit into from

Conversation

tripzero
Copy link

@tripzero tripzero commented Sep 9, 2019

Do not use internal fork of pugixml. Require user to install
pugixml from distro provider or upstream.

In general, it's good practice not to use internal forks of projects. This patch may not be the ultimate solution, but it's a quick and dirty way of eliminating the maintenance burden related to internal forks.

@openvino-pushbot
Copy link
Contributor

Can one of the admins verify this patch?

Do not use internal fork of pugixml. Require user to install
pugixml from distro provider or upstream.
@AnastasiaKazantaeva AnastasiaKazantaeva added the ExternalPR External contributor label Jul 22, 2020
@AnastasiaKazantaeva AnastasiaKazantaeva added ExternalIntelPR External contributor from Intel and removed ExternalPR External contributor labels Aug 3, 2020
@ilya-lavrenov
Copy link
Contributor

Now OpenVINO can use system pugixml #2502

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

Successfully merging this pull request may close these issues.

4 participants