New issue
Advanced search Search tips

Issue 604167 link

Starred by 2 users

Issue metadata

Status: Untriaged
Owner: ----
Cc:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 3
Type: Bug



Sign in to add a comment

Limit dependency on libxml, libxslt

Project Member Reported by dominicc@chromium.org, Apr 17 2016

Issue description

libxml is a difficult dependency to manage for various reasons. We should limit our dependency on libxml while we formulate a long term plan for it.
 
Project Member

Comment 1 by bugdroid1@chromium.org, Apr 21 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/546358bef256d70e9bb3a119c2f90c805ac5ff35

commit 546358bef256d70e9bb3a119c2f90c805ac5ff35
Author: dominicc <dominicc@chromium.org>
Date: Thu Apr 21 02:58:31 2016

Remove libxml from google_apis dependencies.

BUG=604167

Review URL: https://codereview.chromium.org/1893143002

Cr-Commit-Position: refs/heads/master@{#388663}

[modify] https://crrev.com/546358bef256d70e9bb3a119c2f90c805ac5ff35/google_apis/google_apis.gyp

Summary: Limit dependency on libxml, libxslt (was: Limit dependency on libxml)
Project Member

Comment 3 by bugdroid1@chromium.org, Jul 28 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/c12034f19b826e8034314c6ae966ed20d473105a

commit c12034f19b826e8034314c6ae966ed20d473105a
Author: dominicc <dominicc@chromium.org>
Date: Thu Jul 28 03:17:13 2016

Add dominicc to libxslt OWNERS.

R=scottmg@chromium.org
BUG=604167

Review-Url: https://codereview.chromium.org/2122763003
Cr-Commit-Position: refs/heads/master@{#408325}

[modify] https://crrev.com/c12034f19b826e8034314c6ae966ed20d473105a/third_party/libxslt/OWNERS

Project Member

Comment 4 by bugdroid1@chromium.org, Apr 3 2017

Project Member

Comment 5 by bugdroid1@chromium.org, Apr 3 2017

Labels: merge-merged-3029
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/c3c1eab11e66eb19a716c1b0bc1f3a3a784d9740

commit c3c1eab11e66eb19a716c1b0bc1f3a3a784d9740
Author: Dominic Cooney <dominicc@chromium.org>
Date: Mon Apr 03 07:46:12 2017

Delete duplicate win32 xmlversion.h header file.

It's unclear which configuration of libxml2 we're using on Windows.

BUG= 692731 ,604167

Review-Url: https://codereview.chromium.org/2792903002
Cr-Commit-Position: refs/heads/master@{#461377}
(cherry picked from commit f1f078d5f56503e5cf8b5490875e1e1ec9c252ba)

Review-Url: https://codereview.chromium.org/2794853003 .
Cr-Commit-Position: refs/branch-heads/3029@{#539}
Cr-Branched-From: 939b32ee5ba05c396eef3fd992822fcca9a2e262-refs/heads/master@{#454471}

[modify] https://crrev.com/c3c1eab11e66eb19a716c1b0bc1f3a3a784d9740/third_party/libxml/README.chromium
[modify] https://crrev.com/c3c1eab11e66eb19a716c1b0bc1f3a3a784d9740/third_party/libxml/win32/include/libxml/xmlversion.h
[delete] https://crrev.com/03873abfe51c7da3dd502d3addd309c1c1a74697/third_party/libxml/win32/xmlversion.h

Project Member

Comment 6 by bugdroid1@chromium.org, Apr 4 2017

Comment 7 by kochi@chromium.org, Oct 23 2017

Cc: joelhockey@chromium.org dcheng@chromium.org scottmg@chromium.org
Owner: ----
Status: Available (was: Started)
Adding OWNERS of libxml in cc.
Project Member

Comment 8 by sheriffbot@chromium.org, Oct 23

Labels: Hotlist-Recharge-Cold
Status: Untriaged (was: Available)
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue.

Sorry for the inconvenience if the bug really should have been left as Available.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot

Sign in to add a comment