Received-SPF: pass (google.com: best guess record for domain of linux-bluetooth-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
Date:   Mon, 6 May 2019 13:43:33 +0200
From:   "michal.lowas-rzechonek@silvair.com" 
        <michal.lowas-rzechonek@silvair.com>
To:     "Stotland, Inga" <inga.stotland@intel.com>
Cc:     "linux-bluetooth@vger.kernel.org" <linux-bluetooth@vger.kernel.org>,
        "Gix, Brian" <brian.gix@intel.com>
Subject: Re: [PATCH] mesh: Use node uuids as storage directory names
Message-ID: <20190506114333.opaumtub4vgzdycw@scytale>
Mail-Followup-To: "Stotland, Inga" <inga.stotland@intel.com>,
        "linux-bluetooth@vger.kernel.org" <linux-bluetooth@vger.kernel.org>,
        "Gix, Brian" <brian.gix@intel.com>
References: <20190502130630.13890-1-michal.lowas-rzechonek@silvair.com>
 <a716514ec5be59e7e177c9a256238e314b4599d9.camel@intel.com>
 <20190502203526.mfglyyzsoudp6mtw@kynes>
 <89be788e4efdd963e39370b0311d8f2a53c2e13c.camel@intel.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
Content-Transfer-Encoding: 8bit
In-Reply-To: <89be788e4efdd963e39370b0311d8f2a53c2e13c.camel@intel.com>
User-Agent: NeoMutt/20180716
Sender: linux-bluetooth-owner@vger.kernel.org
Precedence: bulk

Hi Inga,

On 05/02, Stotland, Inga wrote:
> So, Join() method will have to check if a directory name with the
> supplied UUID value already exists and, if so, fail with
> "AlreadyExists" error.  The mesh-api.txt doc should be updated
> accordingly.
> 
> Also, if the directory name contains the uuid value, then we probably
> don't need the duplicate value in the JSON file since it can be
> derived from the directory name.
Thanks for the review. I'll send an updated patch shortly.

regards
-- 
Michał Lowas-Rzechonek <michal.lowas-rzechonek@silvair.com>
Silvair http://silvair.com
Jasnogórska 44, 31-358 Krakow, POLAND