Re: [PATCH] target: change default dbroot to /etc/target

2018-04-06 Thread Christoph Hellwig
On Thu, Apr 05, 2018 at 01:54:25PM -0700, Lee Duncan wrote: > How about if the code looks for /etc/target, then uses /var/target if > /etc/target is not present? Then users can use the current sysfs logic > to set the path to any other directory, if neither of these paths are to > their liking? >

Re: [PATCH] target: change default dbroot to /etc/target

2018-04-05 Thread Lee Duncan
On 04/05/2018 10:57 AM, Christoph Hellwig wrote: > On Thu, Apr 05, 2018 at 10:06:35AM -0700, Lee Duncan wrote: >> Also, I think it's a bit unlikely that anyone will still be using >> /var/target, since targetcli-fb has been setting the target root to >> /etc/target for a while now, and the old targ

Re: [PATCH] target: change default dbroot to /etc/target

2018-04-05 Thread Christoph Hellwig
On Thu, Apr 05, 2018 at 10:06:35AM -0700, Lee Duncan wrote: > Also, I think it's a bit unlikely that anyone will still be using > /var/target, since targetcli-fb has been setting the target root to > /etc/target for a while now, and the old targetcli has been deprecated. > (It's the only app I know

Re: [PATCH] target: change default dbroot to /etc/target

2018-04-05 Thread Lee Duncan
On 04/05/2018 12:17 AM, Christoph Hellwig wrote: > On Wed, Apr 04, 2018 at 12:47:03PM -0700, Lee Duncan wrote: >> The dbroot (target PR database root directory) is >> configurable but default to /var/target, a historic >> value. But the reason for adding configurability >> was to move the target di

Re: [PATCH] target: change default dbroot to /etc/target

2018-04-05 Thread Christoph Hellwig
On Wed, Apr 04, 2018 at 12:47:03PM -0700, Lee Duncan wrote: > The dbroot (target PR database root directory) is > configurable but default to /var/target, a historic > value. But the reason for adding configurability > was to move the target directory out of /var. This > is because the File Hierarc