This is an archive of the discontinued Mercurial Phabricator instance.

clib: simplify include paths in C extensions
ClosedPublic

Authored by simpkins on Aug 24 2017, 4:39 PM.
Tags
None
Subscribers
None

Details

Reviewers
quark
Group Reviewers
Restricted Project
Commits
rFBHGXcd6a0869efe8: clib: simplify include paths in C extensions
Summary

Convert the C and C++ files in cdatapack, clib, cstore, and ctreemanifest
to always include files from the root of fb-hgext. This simplifies the build
process by no long requiring a variety of separate include directories to be
specified on the compiler command line.

This will also make it easier to re-use these extensions in other projects
with different build systems.

This corresponds to the Facebook diff D5588676.

Test Plan

Confirmed that "make local" succeeds from a clean build.

Diff Detail

Repository
rFBHGX Facebook Mercurial Extensions
Lint
Lint Skipped
Unit
Unit Tests Skipped

Event Timeline

simpkins created this revision.Aug 24 2017, 4:39 PM
Herald added a reviewer: Restricted Project. · View Herald TranscriptAug 24 2017, 4:39 PM
quark accepted this revision.Aug 24 2017, 5:42 PM
This revision is now accepted and ready to land.Aug 24 2017, 5:42 PM
This revision was automatically updated to reflect the committed changes.