On Mon, 2012-08-06 at 16:54 -0700, Christian Hammond wrote:
> This branch has what we need on the djblets side:
> 
> https://github.com/chipx86/djblets/tree/pipeline-fix
> 
> Unless there's just something messed up here, you should hit problems
> with pipeline not finding files in the PipelineCachedStorage. I traced
> the first occurrence to revision
> dd2eed6e51bcfa4352b95a12e8f43c0feaca8819, but even if I revert that on
> master, I still hit some issues. Can you confirm?
> 

I get the following traceback with your bless patch (as expected):

Traceback (most recent call last):
  File "./contrib/internal/build-media.py", line 19, in <module>
    ret = call_command('collectstatic', interactive=False, verbosity=2)
  File
"/usr/lib/python2.7/site-packages/django/core/management/__init__.py",
line 150, in call_command
    return klass.execute(*args, **defaults)
  File
"/usr/lib/python2.7/site-packages/django/core/management/base.py", line
232, in execute
    output = self.handle(*args, **options)
  File
"/usr/lib/python2.7/site-packages/django/core/management/base.py", line
371, in handle
    return self.handle_noargs(**options)
  File
"/usr/lib/python2.7/site-packages/django/contrib/staticfiles/management/commands/collectstatic.py",
 line 163, in handle_noargs
    collected = self.collect()
  File
"/usr/lib/python2.7/site-packages/django/contrib/staticfiles/management/commands/collectstatic.py",
 line 119, in collect
    dry_run=self.dry_run)
  File "/usr/lib/python2.7/site-packages/pipeline/storage.py", line 30,
in post_process
    packager.pack_stylesheets(package)
  File "/usr/lib/python2.7/site-packages/pipeline/packager.py", line 90,
in pack_stylesheets
    variant=package.variant, **kwargs)
  File "/usr/lib/python2.7/site-packages/pipeline/packager.py", line
100, in pack
    content = compress(paths, **kwargs)
  File
"/usr/lib/python2.7/site-packages/pipeline/compressors/__init__.py",
line 72, in compress_css
    css = self.concatenate_and_rewrite(paths, output_filename, variant)
  File
"/usr/lib/python2.7/site-packages/pipeline/compressors/__init__.py",
line 135, in concatenate_and_rewrite
    content = self.read_file(path)
  File
"/usr/lib/python2.7/site-packages/pipeline/compressors/__init__.py",
line 208, in read_file
    file = default_storage.open(path, 'rb')
  File "/usr/lib/python2.7/site-packages/django/core/files/storage.py",
line 33, in open
    return self._open(name, mode)
  File "/usr/lib/python2.7/site-packages/pipeline/storage.py", line 114,
in _open
    storage = self.find_storage(name)
  File "/usr/lib/python2.7/site-packages/pipeline/storage.py", line 111,
in find_storage
    raise ValueError("The file '%s' could not be found with %r." %
(name, self))
ValueError: The file 'rb/css/diffviewer.css' could not be found with
<pipeline.storage.PipelineFinderStorage object at 0x1aeaad0>.
Traceback (most recent call last):
  File "setup.py", line 182, in <module>
    "Topic :: Software Development :: Quality Assurance",
  File "/usr/lib64/python2.7/distutils/core.py", line 152, in setup
    dist.run_commands()
  File "/usr/lib64/python2.7/distutils/dist.py", line 953, in
run_commands
    self.run_command(cmd)
  File "/usr/lib64/python2.7/distutils/dist.py", line 972, in
run_command
    cmd_obj.run()
  File "/usr/lib/python2.7/site-packages/setuptools/command/install.py",
line 53, in run
    return _install.run(self)
  File "/usr/lib64/python2.7/distutils/command/install.py", line 575, in
run
    self.run_command(cmd_name)
  File "/usr/lib64/python2.7/distutils/cmd.py", line 326, in run_command
    self.distribution.run_command(command)
  File "/usr/lib64/python2.7/distutils/dist.py", line 972, in
run_command
    cmd_obj.run()
  File
"/usr/lib/python2.7/site-packages/setuptools/command/install_egg_info.py", line 
29, in run
    self.run_command('egg_info')
  File "/usr/lib64/python2.7/distutils/cmd.py", line 326, in run_command
    self.distribution.run_command(command)
  File "/usr/lib64/python2.7/distutils/dist.py", line 972, in
run_command
    cmd_obj.run()
  File "setup.py", line 63, in run
    self.run_command('build_media')
  File "/usr/lib64/python2.7/distutils/cmd.py", line 326, in run_command
    self.distribution.run_command(command)
  File "/usr/lib64/python2.7/distutils/dist.py", line 972, in
run_command
    cmd_obj.run()
  File "setup.py", line 81, in run
    raise RuntimeError('Failed to build media files')
RuntimeError: Failed to build media files
error: Bad exit status from /var/tmp/rpm-tmp.pmGIf8 (%install)



Looking at the pipeline commit you referenced, I can understand why it's
still having issue. That patch is just an exception-catcher for an
underlying issue that they must have introduced earlier. I'm going to
see if I can spot an earlier patch that may have introduced the issue
with writing to a static root (unless that's an assumption that Review
Board is making that it should not be).





-- 
Want to help the Review Board project? Donate today at 
http://www.reviewboard.org/donate/
Happy user? Let us know at http://www.reviewboard.org/users/
-~----------~----~----~----~------~----~------~--~---
To unsubscribe from this group, send email to 
reviewboard+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/reviewboard?hl=en

Reply via email to