Re: [dev] [sbase][PATCH] install

2016-02-12 Thread Dimitris Papastamos
On Thu, Feb 11, 2016 at 12:06:14PM +0100, Mattias Andrée wrote: > diff --git a/Makefile b/Makefile > index 1c09cac..b34800d 100644 > --- a/Makefile > +++ b/Makefile > @@ -103,6 +103,7 @@ BIN =\ > getconf\ > grep\ > head\ > + install.out\ > join\ > hostname\ >

Re: [dev] [sbase][PATCH] install

2016-02-11 Thread Dimitris Papastamos
On Thu, Feb 11, 2016 at 01:57:00PM +0100, Mattias Andrée wrote: > It cannot be `install` because it conflicts with > install rule. So I decided to append .out (like in a.out) > to make it work. Duh, of course. > > > +static void > > > +make_dir(char *dir, int was_missing) > > > +{ > > > + if (!mk

Re: [dev] [sbase][PATCH] install

2016-02-11 Thread Mattias Andrée
On Thu, 11 Feb 2016 11:45:32 + Dimitris Papastamos wrote: > Looks good, thanks! Some minor comments below. > > On Thu, Feb 11, 2016 at 12:06:14PM +0100, Mattias Andrée > wrote: > > --- > > LICENSE | 1 + > > Makefile | 9 ++- > > README| 1 + > > TODO | 1 - > > install

Re: [dev] [sbase][PATCH] install

2016-02-11 Thread Dimitris Papastamos
Looks good, thanks! Some minor comments below. On Thu, Feb 11, 2016 at 12:06:14PM +0100, Mattias Andrée wrote: > New command. Includes the flags: > > -s strip binary > -d create directory > -D create missing directories > -t DIR target directory > -m MODEpermissio

[dev] [sbase][PATCH] install

2016-02-11 Thread Mattias Andrée
New command. Includes the flags: -s strip binary -d create directory -D create missing directories -t DIR target directory -m MODEpermission bits -o USERset owner -g GROUP set group Installed files are copied, and default mode is 755. Signed-off-by: Mattias