Page MenuHomeWildfire Games

Remove redundant references to deleted .dtd files
ClosedPublic

Authored by Stan on Aug 2 2022, 5:51 AM.

Details

Summary
issue

The check with xmllint fails during validation. See log output below for more clarity

terminal
❯ xmllint '/Users/paria/0ad/binaries/data/mods/public/gui/common/global.xml' --relaxng '/Users/paria/0ad/binaries/data/mods/mod/gui/gui.rng' --noout
/Users/paria/0ad/binaries/data/mods/public/gui/common/global.xml:3: warning: failed to load external entity "/Users/paria/0ad/binaries/data/mods/public/gui/gui.dtd"
<!DOCTYPE objects SYSTEM "../gui.dtd">
                                      ^
/Users/paria/0ad/binaries/data/mods/public/gui/common/global.xml validates
solution

Remove redundant references to deleted .dtd (Document Type Definition) files.

A DTD defines the structure and the legal elements and attributes of an XML document.

Source: w3schools.com

The problem is partially mentioned in #5556.

Test Plan

Testing with xmllint should work.

Event Timeline

Langbart created this revision.Aug 2 2022, 5:51 AM
Owners added subscribers: Restricted Owners Package, Restricted Owners Package, Restricted Owners Package.Aug 2 2022, 5:51 AM
Vulcan added a comment.Aug 2 2022, 5:56 AM

Successful build - Chance fights ever on the side of the prudent.

Link to build: https://jenkins.wildfiregames.com/job/vs2015-differential/7374/display/redirect

Vulcan added a comment.Aug 2 2022, 6:07 AM

Build failure - The Moirai have given mortals hearts that can endure.

Link to build: https://jenkins.wildfiregames.com/job/macos-differential/6284/display/redirect

Langbart requested review of this revision.Aug 2 2022, 6:20 AM
Langbart edited the summary of this revision. (Show Details)Aug 2 2022, 6:30 AM
Langbart edited the test plan for this revision. (Show Details)
Stan added a subscriber: Stan.Aug 2 2022, 11:13 AM

I'm not sure about the last one since it's an external lib. Otherwise looks good. I wonder if DTDs give more useful messages than RNG.

Langbart updated this revision to Diff 20694.Aug 2 2022, 12:39 PM
Langbart retitled this revision from Remove redundant references to deleted .dtd files to Remove redundant references to deleted .dtd files.

Restore catalog.xml

Stan accepted this revision.Aug 2 2022, 12:42 PM

LGTM.

This revision is now accepted and ready to land.Aug 2 2022, 12:42 PM

Successful build - Chance fights ever on the side of the prudent.

Link to build: https://jenkins.wildfiregames.com/job/vs2015-differential/7375/display/redirect

Build failure - The Moirai have given mortals hearts that can endure.

Link to build: https://jenkins.wildfiregames.com/job/macos-differential/6285/display/redirect

Langbart abandoned this revision.Aug 6 2022, 7:37 PM
Stan commandeered this revision.Oct 23 2022, 7:05 PM
Stan reclaimed this revision.
Stan edited reviewers, added: Langbart; removed: Stan.
This revision was not accepted when it landed; it landed in state Needs Review.Oct 23 2022, 7:07 PM
This revision was landed with ongoing or failed builds.
This revision was automatically updated to reflect the committed changes.