Page MenuHomePhabricator

Rebuild libphutil library map to remove `sprintf_escape` function
ClosedPublic

Authored by epriestley on Jun 15 2014, 7:14 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Sep 12, 6:05 AM
Unknown Object (File)
Wed, Sep 4, 8:48 PM
Unknown Object (File)
Wed, Aug 28, 1:04 PM
Unknown Object (File)
Aug 14 2024, 12:38 PM
Unknown Object (File)
Aug 9 2024, 10:12 AM
Unknown Object (File)
Aug 5 2024, 10:29 AM
Unknown Object (File)
Jul 25 2024, 2:29 PM
Unknown Object (File)
Jul 16 2024, 4:38 AM
Subscribers

Details

Summary

Minor, fix out-of-date build after D9533.

Test Plan

arc liberate

Diff Detail

Repository
rPHU libphutil
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

epriestley retitled this revision from to Rebuild libphutil library map to remove `sprintf_escape` function.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: joshuaspence.
joshuaspence edited edge metadata.

Whoops, sorry. I didn't notice this because src/__phutil_library_map__.php was shown as collapsed.

This revision is now accepted and ready to land.Jun 15 2014, 7:15 PM

We could detect this sort of thing by running PhutilInfrastructureTestCase. I almost wonder if we should always run this test case?

I think it would take a little effort -- IIRC, we don't look for src/__tests__/ right now -- but I think it's reasonable to make that work and move the src/infrastructure/__tests__/ directory up.

epriestley updated this revision to Diff 22909.

Closed by commit rPHU04712fda4b76 (authored by @epriestley).

I could also imagine writing some kind of arc make which happens during arc land to make sure artifacts like this are generated correctly, but that's a more significant undertaking.

Ideally we probably wouldn't be checking this stuff into the master repository at all, and would just build it for some kind of tarball release kind of thing, but doing that without creating huge barriers for users would require a lot more release/versioning/packaging infrastructure than we currently have.