Return-Path: 
 <dev-return-31113-apmail-subversion-dev-archive=subversion.apache.org@subversion.apache.org>
X-Original-To: apmail-subversion-dev-archive@minotaur.apache.org
Delivered-To: apmail-subversion-dev-archive@minotaur.apache.org
Received: from mail.apache.org (hermes.apache.org [140.211.11.3])
	by minotaur.apache.org (Postfix) with SMTP id 8B93D1133D
	for <apmail-subversion-dev-archive@minotaur.apache.org>;
 Tue,  2 Sep 2014 06:24:45 +0000 (UTC)
Received: (qmail 56479 invoked by uid 500); 2 Sep 2014 06:24:43 -0000
Delivered-To: apmail-subversion-dev-archive@subversion.apache.org
Received: (qmail 56423 invoked by uid 500); 2 Sep 2014 06:24:43 -0000
Mailing-List: contact dev-help@subversion.apache.org; run by ezmlm
Precedence: bulk
List-Help: <mailto:dev-help@subversion.apache.org>
List-Unsubscribe: <mailto:dev-unsubscribe@subversion.apache.org>
List-Post: <mailto:dev@subversion.apache.org>
List-Id: <dev.subversion.apache.org>
Delivered-To: mailing list dev@subversion.apache.org
Received: (qmail 56413 invoked by uid 99); 2 Sep 2014 06:24:42 -0000
Received: from nike.apache.org (HELO nike.apache.org) (192.87.106.230)
    by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 02 Sep 2014 06:24:42 +0000
X-ASF-Spam-Status: No, hits=1.5 required=5.0
	tests=HTML_MESSAGE,RCVD_IN_DNSWL_LOW,SPF_PASS
X-Spam-Check-By: apache.org
Received-SPF: pass (nike.apache.org: domain of brane@wandisco.com designates
 209.85.212.182 as permitted sender)
Received: from [209.85.212.182] (HELO mail-wi0-f182.google.com)
 (209.85.212.182)
    by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 02 Sep 2014 06:24:15 +0000
Received: by mail-wi0-f182.google.com with SMTP id z2so7174221wiv.9
        for <dev@subversion.apache.org>; Mon, 01 Sep 2014 23:24:14 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed;
        d=wandisco.com; s=gapps;
        h=message-id:date:from:organization:user-agent:mime-version:to:cc
         :subject:references:in-reply-to:content-type;
        bh=eKg+jvwbyfDY4ff67A95pmrAjy0qq7Hzmxk3DR6DTGI=;
        b=KR+oINFz/xy8GVVux2gCXJ41ktBUSbUCI9lJO/nEcU2t4yrg0+mrQ7IalthwVe9vgH
         8aBXlHKHMkg8IDgSloV+VKRnoOArgbOMVyQhL5Tu8obZpKF7mxR1GwsLaggoGSHiQ8cU
         u5z4VC1eSLdRExYrzjhAYCVaDplb9HM8zmeQU=
X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed;
        d=1e100.net; s=20130820;
        h=x-gm-message-state:message-id:date:from:organization:user-agent
         :mime-version:to:cc:subject:references:in-reply-to:content-type;
        bh=eKg+jvwbyfDY4ff67A95pmrAjy0qq7Hzmxk3DR6DTGI=;
        b=i4MGvHA7l5rRv38ooZ9XUElfC/BWx+ajRG1LHAntXp2bbfwpWtZ2VdIp6yzN/fm+OX
         eoPMRusSXXYZ9pqWnxBf0P+jlJZ50g4z7K0oLPT77cIv9ZXzNc7shQ6ajdylsy9cV4Je
         n0yFY3m4EkMpM27j0uJjoxmpvLrn52U+63RjF/KTq55E/9B81kW7IB27YC92CUOaW+tS
         74DcS1gf7wMlxzwN+VQgTI9AWZ4k7pElTBKx4on+Bs+kzjtOZaR4MMvuEPpcPwX56RYO
         ZUU8GoUTuqaxQESBBITcPR51POL7j4Jgw9tYltKdTDSA93fm53WOx0L/gNUt1VvrNcU2
         qkxQ==
X-Gm-Message-State: 
 ALoCoQlJgMGQ0yXKuenw/cTx0v2JLfdazq3d1m/NmSqNNymiyNSrMgW3WSgLGfeoakZXGW63LXMO
X-Received: by 10.194.187.241 with SMTP id fv17mr37435195wjc.13.1409639054599;
        Mon, 01 Sep 2014 23:24:14 -0700 (PDT)
Received: from zulu.local (cpe-90-157-246-198.dynamic.amis.net.
 [90.157.246.198])
        by mx.google.com with ESMTPSA id la2sm7242135wjb.5.2014.09.01.23.24.13
        for <multiple recipients>
        (version=TLSv1 cipher=RC4-SHA bits=128/128);
        Mon, 01 Sep 2014 23:24:13 -0700 (PDT)
Received: from zulu.local (localhost [127.0.0.1])
	by zulu.local (Postfix) with ESMTP id 09270C6C3A47;
	Tue,  2 Sep 2014 08:24:12 +0200 (CEST)
Message-ID: <5405628B.3020501@wandisco.com>
Date: Tue, 02 Sep 2014 08:24:11 +0200
From: =?UTF-8?B?QnJhbmtvIMSMaWJlag==?= <brane@wandisco.com>
Organization: WANdisco
User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.9;
 rv:24.0) Gecko/20100101 Thunderbird/24.6.0
MIME-Version: 1.0
To: Stefan Fuhrmann <stefan.fuhrmann@wandisco.com>
CC: Subversion Development <dev@subversion.apache.org>
Subject: Re: svn commit: r1621387 - in
 /subversion/branches/authzperf/subversion/libsvn_repos:
 authz.c authz.h authz_parse.c
References: <20140829210326.32CB82388868@eris.apache.org>
	<54010FE5.3040507@wandisco.com>
	<CA+t0gk1CrXGaB+N+NkK3zYR+=D08mG_7gi8iOcDe-vbv10vRWg@mail.gmail.com>
	<5402371A.7040703@wandisco.com>	<54042BF3.4030804@wandisco.com>
	<CA+t0gk0X23KiMHqKfKk=2aoS1hWswrPUOSHkiChr8qeNgqc0jg@mail.gmail.com>
 <CA+t0gk18J99=Y5KWvPNc7tncN+E6KnCvVEUF61Bi_mHUxmEYUA@mail.gmail.com>
In-Reply-To: 
 <CA+t0gk18J99=Y5KWvPNc7tncN+E6KnCvVEUF61Bi_mHUxmEYUA@mail.gmail.com>
Content-Type: multipart/alternative;
 boundary="------------080807030903000305010801"
X-Virus-Checked: Checked by ClamAV on apache.org

This is a multi-part message in MIME format.
--------------080807030903000305010801
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 7bit

On 01.09.2014 21:48, Stefan Fuhrmann wrote:
>
>         Stefan, I went through the docs (and commits); there are a
>         couple new
>         comments in the doc, one is a minor nit, the other affects the
>         rights
>         lookup workflow.
>
>
>     We already talked about that shortcut.
>
> I'll give it a go tomorrow.

I'm about to commit a new function in authz_info.c, called
svn_authz__get_global_rights; it should do the global access resolving part.

>      IIUC, you already removed the foo/** to foo/*
>
>         normalization from the code?
>
>
>     Yes, at least that was the intent. I might have missed or broken
>     something, though.
>
> Parser states can be non-obvious at times.

authz-test says trailing ** remains unchanged, so I guess you found the
right place.

-- Brane


--------------080807030903000305010801
Content-Type: text/html; charset=UTF-8
Content-Transfer-Encoding: 8bit

<html>
  <head>
    <meta content="text/html; charset=UTF-8" http-equiv="Content-Type">
  </head>
  <body bgcolor="#FFFFFF" text="#000000">
    <div class="moz-cite-prefix">On 01.09.2014 21:48, Stefan Fuhrmann
      wrote:</div>
    <blockquote
cite="mid:CA+t0gk18J99=Y5KWvPNc7tncN+E6KnCvVEUF61Bi_mHUxmEYUA@mail.gmail.com"
      type="cite">
      <div dir="ltr"><br>
        <div class="gmail_extra">
          <div class="gmail_quote">
            <blockquote class="gmail_quote" style="margin:0 0 0
              .8ex;border-left:1px #ccc solid;padding-left:1ex">
              <div dir="ltr">
                <div class="gmail_extra">
                  <div class="gmail_quote">
                    <div class="">
                      <blockquote class="gmail_quote" style="margin:0 0
                        0 .8ex;border-left:1px #ccc
                        solid;padding-left:1ex">Stefan, I went through
                        the docs (and commits); there are a couple new<br>
                        comments in the doc, one is a minor nit, the
                        other affects the rights<br>
                        lookup workflow.</blockquote>
                      <div><br>
                      </div>
                    </div>
                    <div>We already talked about that shortcut.<br>
                    </div>
                  </div>
                </div>
              </div>
            </blockquote>
            <div>I'll give it a go tomorrow. <br>
            </div>
          </div>
        </div>
      </div>
    </blockquote>
    <br>
    I'm about to commit a new function in authz_info.c, called
    svn_authz__get_global_rights; it should do the global access
    resolving part.<br>
    <br>
    <blockquote
cite="mid:CA+t0gk18J99=Y5KWvPNc7tncN+E6KnCvVEUF61Bi_mHUxmEYUA@mail.gmail.com"
      type="cite">
      <div dir="ltr">
        <div class="gmail_extra">
          <div class="gmail_quote">
            <blockquote class="gmail_quote" style="margin:0 0 0
              .8ex;border-left:1px #ccc solid;padding-left:1ex">
              <div dir="ltr">
                <div class="gmail_extra">
                  <div class="gmail_quote">
                    <div class="">
                      <div> IIUC, you already removed the foo/** to
                        foo/*<br>
                      </div>
                      <blockquote class="gmail_quote" style="margin:0 0
                        0 .8ex;border-left:1px #ccc
                        solid;padding-left:1ex">
                        normalization from the code?<br>
                      </blockquote>
                      <div><br>
                      </div>
                    </div>
                    <div>Yes, at least that was the intent. I might have
                      missed or broken something, though.</div>
                  </div>
                </div>
              </div>
            </blockquote>
            <div>Parser states can be non-obvious at times.<br>
            </div>
          </div>
        </div>
      </div>
    </blockquote>
    <br>
    authz-test says trailing ** remains unchanged, so I guess you found
    the right place.<br>
    <br>
    -- Brane<br>
    <br>
  </body>
</html>

--------------080807030903000305010801--