From: sven.vermeulen@siphos.be (Sven Vermeulen)
Date: Sat, 2 Dec 2017 13:42:22 +0100
Subject: [refpolicy] [PATCH v2 8/8] Fix regexp for templated boolean
	generation
In-Reply-To: <f8fd346b-cafb-69b1-7d0e-fe7aa73ee1db@ieee.org>
References: <20171128173302.16454-1-sven.vermeulen@siphos.be>
	<f8fd346b-cafb-69b1-7d0e-fe7aa73ee1db@ieee.org>
Message-ID: <CAPzO=Nwdhwqr4FjK5atA=vbHvwLvHbiT448CYWVXFUfD6DC2XA@mail.gmail.com>
To: refpolicy@oss.tresys.com
List-Id: refpolicy.oss.tresys.com

On Wed, Nov 29, 2017 at 2:22 AM, Chris PeBenito via refpolicy
<refpolicy@oss.tresys.com> wrote:
> Unfortunately, I'm running into compile errors.  For a monolithic build
> I get:
>
> /usr/bin/checkpolicy:  loading policy configuration from policy.conf
> policy/modules/contrib/cron.te:714:ERROR 'syntax error' at token 'if' on
> line 521548:
>         if (use_nfs_home_dirs) {
> #line 714
> checkpolicy:  error(s) encountered while parsing configuration
>
>
>
>
> I added a commit to insert comments for else and closing brackets of
> tunable blocks, but I couldn't see what the problem was.

I now get the same issue with modular builds as well (the other build
error was due to a typo, my bad for not running sufficient testing,
I'm now doing the various builds beforehand), and would like to get
more deep into this before sending out v3 (which will also include a
fix for a case where not all arguments were processed correctly).

When you say "I added a commit to insert comments", do you mean inside
the policy/support/loadable_module.spt's tunable_policy declaration?

Is there perhaps a maximum depth of tunable_policy() chains that I
might be hitting?

Wkr,
  Sven Vermeulen