This is an archive of the discontinued LLVM Phabricator instance.

[docs] Remove Makefile.sphinx files
ClosedPublic

Authored by tstellar on Oct 18 2021, 1:01 PM.

Details

Reviewers
sylvestre.ledru
pcc
silvas
ldionne
Group Reviewers
Restricted Project
Commits
rGc16655f138b1: [docs] Remove Makefile.sphinx files
Summary

Does anyone still use these? I want to make some changes to the sphinx
html generation and I don't want to have to implement the changes in
two places.

Diff Detail

Event Timeline

tstellar requested review of this revision.Oct 18 2021, 1:01 PM
tstellar created this revision.
Herald added projects: Restricted Project, Restricted Project, Restricted Project. · View Herald TranscriptOct 18 2021, 1:01 PM
Herald added a reviewer: Restricted Project. · View Herald Transcript
ldionne accepted this revision.Oct 18 2021, 1:04 PM
ldionne added a subscriber: ldionne.

We don't use them in libc++, fine by me to remove if the website documentation builders don't use them (IDK how that's set up).

This revision is now accepted and ready to land.Oct 18 2021, 1:04 PM

I am using them but I can move to sphinx calls.

I am using them but I can move to sphinx calls.

How much time do you need to transition?

sylvestre.ledru accepted this revision.Oct 20 2021, 1:53 PM

I don't need any time, please land it when you want, I will adjust later :)

This revision was landed with ongoing or failed builds.Oct 21 2021, 4:09 PM
This revision was automatically updated to reflect the committed changes.

Adjustment made on my side:
https://salsa.debian.org/pkg-llvm-team/llvm-toolchain/-/commit/daf38c1634954d9aad7a1393eef0828c57f41a53

@tstellar Maybe you could add this to the release notes? (I think others are probably using it)

clang/docs/Makefile.sphinx