Page MenuHomeWildfire Games

svn propset for json files
ClosedPublic

Authored by Nescio on Aug 16 2020, 9:02 PM.

Details

Reviewers
Silier
bb
Group Reviewers
Restricted Owners Package(Owns No Changed Paths)
Commits
rP23994: Fix lineendings for json files
Summary

This patch standardizes the svn propsets for all .json files:

svn propset svn:mime-type text/json *.json
svn propset svn:eol-style native *.json

On my end arcanists complains when a svn propset is not set. Having to set it for individual files whenever a patch includes a json file can be annoying. Setting it for all json files at once is far more efficient.

Test Plan

Verify everything still works.

Diff Detail

Repository
rP 0 A.D. Public Repository
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

Nescio created this revision.Aug 16 2020, 9:02 PM
Owners added subscribers: Restricted Owners Package, Restricted Owners Package, Restricted Owners Package, Restricted Owners Package.Aug 16 2020, 9:02 PM

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

builderr-release-macos.txt
/Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/ranlib: file: ../../../binaries/system/libengine.a(precompiled.o) has no symbols
/Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/ranlib: file: ../../../binaries/system/liblowlevel.a(dbghelp.o) has no symbols
/Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/ranlib: file: ../../../binaries/system/liblowlevel.a(file_stats.o) has no symbols
/Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/ranlib: file: ../../../binaries/system/liblowlevel.a(vfs_path.o) has no symbols

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

Nescio requested review of this revision.Aug 16 2020, 9:07 PM
bb accepted this revision.Aug 25 2020, 4:15 PM
bb added a subscriber: bb.

We should run

svn propset svn:eol-style native $(find . -type f -name "*.json")
svn propset svn:mime-type text/json $(find . -type f -name "*.json")

from time to time in the repo, or let some bot do it

This revision is now accepted and ready to land.Aug 25 2020, 4:15 PM
This revision was automatically updated to reflect the committed changes.