search

Pylons / pyramid_layout

Pyramid add-on for managing UI layouts.
Other
24 stars 26 forks source link

sphinx_build calls git unconditionally #18

Open frispete opened 11 years ago

frispete commented 11 years ago

which prevents this package from being built for a linux distribution properly.

Sorry for not providing a patch/pull target as I'm completely overcharged ATM..

mcdonc commented 11 years ago

sphinx_build does indeed depend on git availability here. Can you describe what commands you're using that cause an error?

frispete commented 11 years ago

Dear Chris,

I build my packages on openSUSE build service in order to hand them over to the openSUSE python repository.

You can see all my python related packages (but PyQt) here:

https://build.opensuse.org/project/monitor/home:frispete:python

E.g. the pyramid_layout build log is here:

https://build.opensuse.org/package/live_build_log/home:frispete:python/python-pyramid_layout/openSUSE_12.2/x86_64

I created a package called python-pylons_sphinx_theme in order to share the pylon sphinx theming among the related packages, hence the pyramid theme is available system-wide, and there's no need to download it explicitely. BTW, In such a build system, downloads are forbidden for security reasons.

The command leading to the failure is:

python setup.py build_sphinx

In order to build the package successfully, I applied this patch:

Index: b/docs/conf.py

--- a/docs/conf.py +++ b/docs/conf.py @@ -97,24 +97,6 @@ pygments_style = 'sphinx'

-- Options for HTML output ---------------------------------------------------

Add and use Pylons theme

-from subprocess import call, Popen, PIPE

-p = Popen('which git', shell=True, stdout=PIPE) -git = p.stdout.read().strip() -cwd = os.getcwd()

-_themes = os.path.join(cwd, '_themes')

-if not os.path.isdir(_themes):

- os.chdir(cwd)

-sys.path.append(os.path.abspath('_themes')) -html_theme_path = ['_themes'] html_theme = 'pyramid' html_theme_options = dict(github_url='https://github.com/Pylons/pyramid_layout')

All other packages, that depend on pylons_sphinx_theme don't suffer from this.

I know, that virtualenv is the preferred installation method, but the packaging way has some advantages, too.

Cheers, Pete

On Dienstag, 6. August 2013 05:30:08 Chris McDonough wrote:

sphinx_build does indeed depend on git availability here. Can you describe what commands you're using that cause an error?

Reply to this email directly or view it on GitHub: https://github.com/Pylons/pyramid_layout/issues/18#issuecomment-22175215