Re: [ovs-dev] [PATCH v11 2/8] util: Add a path canonicalizer

2016-04-14 Thread Ben Pfaff
I think I'd prefer to avoid path canonicalization. If it proves not flexible enough, we can add it in later. Thanks, Ben. On Wed, Apr 13, 2016 at 12:08:58PM -0400, Aaron Conole wrote: > Hi Ben, > > I have rebased (due to conflicts) the series and am set to resubmit; > however I'd like to get c

Re: [ovs-dev] [PATCH v11 2/8] util: Add a path canonicalizer

2016-04-13 Thread Aaron Conole
Hi Ben, I have rebased (due to conflicts) the series and am set to resubmit; however I'd like to get closure on this issue before I do. I have no strong feelings either way - I can manually scan the string for .. or use the realpath code I've put here. Since this only happens at initialization ti

Re: [ovs-dev] [PATCH v11 2/8] util: Add a path canonicalizer

2016-04-12 Thread Mooney, Sean K
> -Original Message- > From: Aaron Conole [mailto:acon...@redhat.com] > Sent: Tuesday, April 12, 2016 2:18 PM > To: Mooney, Sean K > Cc: Ben Pfaff ; dev@openvswitch.org; Flavio Leitner > ; Traynor, Kevin ; Panu > Matilainen ; Wojciechowicz, RobertX > ; Andy Zhou ; Daniele Di > Proietto ;

Re: [ovs-dev] [PATCH v11 2/8] util: Add a path canonicalizer

2016-04-12 Thread Aaron Conole
"Mooney, Sean K" writes: >> -Original Message- >> From: Aaron Conole [mailto:acon...@redhat.com] >> Sent: Tuesday, April 12, 2016 1:12 AM >> To: Ben Pfaff >> Cc: dev@openvswitch.org; Flavio Leitner ; Traynor, >> Kevin ; Panu Matilainen ; >> Wojciechowicz, RobertX ; Mooney, Sean K >> ; An

Re: [ovs-dev] [PATCH v11 2/8] util: Add a path canonicalizer

2016-04-12 Thread Mooney, Sean K
> -Original Message- > From: Aaron Conole [mailto:acon...@redhat.com] > Sent: Tuesday, April 12, 2016 1:12 AM > To: Ben Pfaff > Cc: dev@openvswitch.org; Flavio Leitner ; Traynor, > Kevin ; Panu Matilainen ; > Wojciechowicz, RobertX ; Mooney, Sean K > ; Andy Zhou ; Daniele Di > Proietto ;

Re: [ovs-dev] [PATCH v11 2/8] util: Add a path canonicalizer

2016-04-11 Thread Aaron Conole
Hi Ben, Ben Pfaff writes: > (Yow, that's a lot of CCs.) Lots of cooks in the kitchen on this one. > On Fri, Apr 01, 2016 at 11:31:31AM -0400, Aaron Conole wrote: >> This commit adds a new function (ovs_realpath) to perform the role of >> realpath on various operating systems. The purpose is to

Re: [ovs-dev] [PATCH v11 2/8] util: Add a path canonicalizer

2016-04-11 Thread Ben Pfaff
(Yow, that's a lot of CCs.) On Fri, Apr 01, 2016 at 11:31:31AM -0400, Aaron Conole wrote: > This commit adds a new function (ovs_realpath) to perform the role of > realpath on various operating systems. The purpose is to ensure that a > given path to file exists, and to return a completely resolve