This is an archive of the discontinued LLVM Phabricator instance.

Add Option to sphinx-build for custom conf file location
AcceptedPublic

Authored by dibya001 on Aug 31 2020, 3:16 AM.

Details

Summary

1 - Sphinx takes an optional "-c" option where the directory containing the configuration file(conf.py) path is provided.
2 - Before this change, there was no way to provide path for conf.py file which may not be stored in source directory.
3 - With this change, we can provide an option if the file is not present in the source directory but present in another directory.
4 - One usecase for this is the source directory contains a conf.py.in which is converted to conf.py under the build folder after appropriate cmake variable substitution

Diff Detail

Event Timeline

dibya001 created this revision.Aug 31 2020, 3:16 AM
Herald added a project: Restricted Project. · View Herald TranscriptAug 31 2020, 3:16 AM
dibya001 requested review of this revision.Aug 31 2020, 3:16 AM
shankare accepted this revision.Aug 31 2020, 8:50 AM

Ping ?

This revision is now accepted and ready to land.Aug 31 2020, 8:50 AM
labath resigned from this revision.Sep 2 2020, 8:28 AM

I don't know anything about Sphinx. It might be useful to mention in the patch description the motivation for this change.

dibya001 edited the summary of this revision. (Show Details)Sep 3 2020, 4:38 AM
dibya001 added a subscriber: labath.

I don't know anything about Sphinx. It might be useful to mention in the patch description the motivation for this change.

Updated the patch description.

Looks reasonable to me.

xgupta added a subscriber: xgupta.Jul 17 2021, 6:45 AM

ping? need someone to comit?

Do you still want this committed? If so, can you rebase?

Herald added a project: Restricted Project. · View Herald TranscriptOct 25 2022, 4:44 PM