New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

dcheck failure on static_rewriter test #832

Closed
GoogleCodeExporter opened this Issue Apr 6, 2015 · 2 comments

Comments

Projects
None yet
1 participant
@GoogleCodeExporter
Copy link

GoogleCodeExporter commented Apr 6, 2015

I encountered this when I followed the "Building PSOL From Source" instructions 
at this step
$ cd ~/mod_pagespeed/src/net/instaweb/automatic
$ make CXXFLAGS="-DSERF_HTTPS_FETCHING=0" \
       BUILDTYPE=Debug \
       AR.host="$PWD/../../../build/wrappers/ar.sh" \
       AR.target="$PWD/../../../build/wrappers/ar.sh" \
       all

In more detail, it failed when 
$(MAKE) examples (in Makefile)
with following cmd
static_rewriter --rewriters=collapse_whitespace 
/home/xxx/mod_pagespeed/src/install/mod_pagespeed_example 
/home/xxx/mod_pagespeed/src/net/instaweb/automatic collapse_whitespace.html

Following the backtrace, I found this
StaticRewriter::StaticRewriter(*)
    server_context_(file_rewriter_.CreateServerContext())
        ......
              rewrite_options->ComputeSignature()                     //frozen_ will be set true         
    gflags_.SetOptions(&file_rewriter_,server_context_->global_options())
        RewriteOptions::set_option(*)
            RewriteOptions::Modify()
                         DCHECK(!frozen_)                                     //it failed here 


any help will be appreciated

Original issue reported on code.google.com by jmara...@google.com on 14 Nov 2013 at 2:38

@GoogleCodeExporter

This comment has been minimized.

Copy link

GoogleCodeExporter commented Apr 6, 2015

Fixed in revision 3630

Original comment by jmara...@google.com on 14 Nov 2013 at 4:04

  • Changed state: Fixed
@GoogleCodeExporter

This comment has been minimized.

Copy link

GoogleCodeExporter commented Apr 6, 2015

Original comment by jefftk@google.com on 2 Jan 2014 at 5:38

  • Added labels: Milestone-v31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment