Re: [dev] fsbm (1) version 1.1 released

2016-02-11 Thread Alba Pompeo
So Calvin version works on Linux and Dimitris version works on OpenBSD and maybe NetBSD. Looks like everyone's covered :) On Thu, Feb 11, 2016 at 3:58 PM, Dimitris Papastamos wrote: > On Thu, Feb 11, 2016 at 12:55:34PM -0500, Calvin Morrison wrote: >> On 11 February 2016 at 12:53, Alba Pompeo

Re: [dev] fsbm (1) version 1.1 released

2016-02-11 Thread Dimitris Papastamos
On Thu, Feb 11, 2016 at 12:55:34PM -0500, Calvin Morrison wrote: > On 11 February 2016 at 12:53, Alba Pompeo wrote: > > It's not supposed to be used on Linux on the first place - right. > > My bad for missing this information. > > > > > > On Thu, Feb 11, 2016 at 3:51 PM, Dimitris Papastamos wrote

Re: [dev] fsbm (1) version 1.1 released

2016-02-11 Thread Calvin Morrison
On 11 February 2016 at 12:53, Alba Pompeo wrote: > It's not supposed to be used on Linux on the first place - right. > My bad for missing this information. > > > On Thu, Feb 11, 2016 at 3:51 PM, Dimitris Papastamos wrote: >> On Thu, Feb 11, 2016 at 03:48:34PM -0200, Alba Pompeo wrote: >>> Both ar

Re: [dev] fsbm (1) version 1.1 released

2016-02-11 Thread Alba Pompeo
It's not supposed to be used on Linux on the first place - right. My bad for missing this information. On Thu, Feb 11, 2016 at 3:51 PM, Dimitris Papastamos wrote: > On Thu, Feb 11, 2016 at 03:48:34PM -0200, Alba Pompeo wrote: >> Both are very cool! >> A nitpick is that Dimitri's version uses the

Re: [dev] fsbm (1) version 1.1 released

2016-02-11 Thread Dimitris Papastamos
On Thu, Feb 11, 2016 at 03:48:34PM -0200, Alba Pompeo wrote: > Both are very cool! > A nitpick is that Dimitri's version uses the nonstandard header sys/sysctl.h. > This prevents compilation on musl. Hm yes, it is only meant to work on OpenBSD. The sample() function is totally non-portable. (It

Re: [dev] fsbm (1) version 1.1 released

2016-02-11 Thread Alba Pompeo
Both are very cool! A nitpick is that Dimitri's version uses the nonstandard header sys/sysctl.h. This prevents compilation on musl. On Thu, Feb 11, 2016 at 3:41 PM, Dimitris Papastamos wrote: > On Thu, Feb 11, 2016 at 11:01:35AM -0500, Calvin Morrison wrote: >> Hi, >> >> I had time to commit my

Re: [dev] fsbm (1) version 1.1 released

2016-02-11 Thread Dimitris Papastamos
On Thu, Feb 11, 2016 at 11:01:35AM -0500, Calvin Morrison wrote: > Hi, > > I had time to commit my last changes into fsbm [0], the minimal > bandwidth meter. Notably the two features added are below: > > -r flag now accepted to show raw byte or bit counts, rather than > squashing into Kb/s Mb/s

Re: [dev] tag new slock release?

2016-02-11 Thread Markus Teich
Anselm R Garbe wrote: > Fine by me, don't forget to update config.mk and LICENSE as well with 1.3 > version info and copyright update. Heyho, I think its ready to ship, even added a man page. Is anything missing? If not, feel free to tag and create the release tar.gz. I already pushed the updated

[dev] fsbm (1) version 1.1 released

2016-02-11 Thread Calvin Morrison
Hi, I had time to commit my last changes into fsbm [0], the minimal bandwidth meter. Notably the two features added are below: -r flag now accepted to show raw byte or bit counts, rather than squashing into Kb/s Mb/s etc. -i flag now accepted to show a single interface that is requested For now

Re: [dev] tag new slock release?

2016-02-11 Thread Anselm R Garbe
Hi Markus, On 10 February 2016 at 16:46, Markus Teich wrote: > I propose to tag a new v1.3 release for slock. I have no open patches or bug > reports on my list and my attention was brought to it a few times already. A > major thing noted by most people was the fix for resizing the cover window o

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

[dev] [sbase][PATCH] diff: fix style

2016-02-11 Thread Mattias Andrée
Signed-off-by: Mattias Andrée --- install.c | 8 +++- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/install.c b/install.c index 0a81101..ec96c05 100644 --- a/install.c +++ b/install.c @@ -45,11 +45,9 @@ static void strip(const char *filename) { pid_t pid = fork(); -

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

Re: [dev] [sbase] install

2016-02-11 Thread Dimitris Papastamos
On Wed, Feb 03, 2016 at 08:10:50PM +0100, Mattias Andrée wrote: > I don't really see the point of including install, > but I assumed there would have been some consensus > on what to features to include. > None of implementations have the same flag set, and > some have incompatible flag sets; and i

Re: [dev] [sbase][PATCH v2] pathchk

2016-02-11 Thread Dimitris Papastamos
On Wed, Feb 03, 2016 at 08:12:26PM +0100, Mattias Andrée wrote: > New command, including man page. > UTF-8 compatible and should be POSIX-compliant. Applied, thanks!